Dlaczego ten program nie działa?

Question

Witam, piszę program, który mam zadanie skracać podany ciąg liter od wybranego znaku, np. gdy podam aaa i chcę skracać od pierwszego miejsca, to program powinien wypisać a3. Jednak mój kod nie działa, nie wiem co jest nie tak...

#include <iostream>
#include <sstream>
#include <fstream>
#include <string>

using namespace std;

string konwersjaIntNaString (int liczba)
{
    ostringstream ss;
    ss << liczba;
    string str = ss.str();

    return str;
}

int obliczIleTakichSamychZnakowWystepujeObokSiebie(string wyraz, int pozycja)
{
    int iloscTakichSamychZnakow = 1;
    char znak;
    if(wyraz.length() > pozycja)
    {
        while((wyraz[iloscTakichSamychZnakow] == wyraz[iloscTakichSamychZnakow + 1]) && (! isdigit(wyraz[iloscTakichSamychZnakow])))
        {
            iloscTakichSamychZnakow++;
            pozycja++;
        }
   }
    else
        iloscTakichSamychZnakow = 0;


    return iloscTakichSamychZnakow;
}


string skracanieWyrazow(string wyrazDoSkrocenia, int minimalnaIloscZnakowOdKtorychNalezySkracac)
{
    int dlugosc = wyrazDoSkrocenia.length();
    int iloscWystapien = 1;
    char powtarzajacySieZnak;
    int pozycja;


    for (int i = 0; i < dlugosc-2; i++)
    {
        if ( !isdigit(wyrazDoSkrocenia[pozycja]) && obliczIleTakichSamychZnakowWystepujeObokSiebie(wyrazDoSkrocenia, pozycja) >= minimalnaIloscZnakowOdKtorychNalezySkracac)
        {
            iloscWystapien = obliczIleTakichSamychZnakowWystepujeObokSiebie(wyrazDoSkrocenia, i);
            powtarzajacySieZnak = wyrazDoSkrocenia[i];
            wyrazDoSkrocenia.replace(i, iloscWystapien, powtarzajacySieZnak + konwersjaIntNaString(iloscWystapien));
            dlugosc = wyrazDoSkrocenia.length();
        }

    }
    return wyrazDoSkrocenia;
}


int main()
{
    string wyraz;
    int minimalnaIloscZnakowOdKtorychNalezySkracac;

    cin >> wyraz;
    cin >> minimalnaIloscZnakowOdKtorychNalezySkracac;
    cout << skracanieWyrazow(wyraz, minimalnaIloscZnakowOdKtorychNalezySkracac) << endl;

    return 0;
}

 

Answer 1

Um, trochę rzeczy w tym kodzie jest dziwnych:

int obliczIleTakichSamychZnakowWystepujeObokSiebie(string wyraz, int pozycja)
{
    int iloscTakichSamychZnakow = 1;
    char znak;
    if(wyraz.length() > pozycja)
    {
        while((wyraz[iloscTakichSamychZnakow] == wyraz[iloscTakichSamychZnakow + 1]) && (! isdigit(wyraz[iloscTakichSamychZnakow])))
        {
            iloscTakichSamychZnakow++;
            pozycja++;
        }
   }
    else
        iloscTakichSamychZnakow = 0;
 
 
    return iloscTakichSamychZnakow;
}

Na pewno w tej pętli nigdzie nie chcesz użyć `pozycja`? Trochę dziwne, że używasz go tylko do porównania z rozmiarem.

    int pozycja;
 
 
    for (int i = 0; i < dlugosc-2; i++)
    {
        if ( !isdigit(wyrazDoSkrocenia[pozycja]) && ...

Nigdzie nie inicjalizujesz tej zmiennej?

BTW:

string konwersjaIntNaString (int liczba)
{
    ostringstream ss;
    ss << liczba;
    string str = ss.str();
 
    return str;
}

dokładnie to robi std::to_string.

Comments

https://forum.pasja-informatyki.pl/443571/dlaczego-nie-dziala-zamiana-i-skracanie-liter

https://forum.pasja-informatyki.pl/443264/skracanie-wyrazu-od-wybranego-znaku

oba pytania o ten sam kod w innym postaciah, kazdy z nich zawiera ta sama funkcje konwertujaca, ale w zadnych z nich nie zostala ona wykorzystana. Wiec nie dokladnie rozumiem po co ona jest w tym kodzie. 

---

(konwersjaIntNaString jest wykorzystane w tym kodzie)

---

 @magda_19,

Może pisz programy po jednej funkcji, i jak działa to przerzucasz do głównego programu.

A jak nie działa to cout<< po każdej zmiennej lub kontrolne linie kodu np

.... // jakiś kod 
cout<< "sprawdzenie 1 *****************************************************" <<endl;
.... // jakiś kod 
cout<< "sprawdzenie 2 *****************************************************" <<endl;
.... // jakiś kod 
cout<< "sprawdzenie 3 *****************************************************" <<endl;
// i wiesz w którym miejscu się wywala

albo w nagłówku
 #define spr(x) std::cout<<"spwadzenie"<<#x<<"***************************************"<<std::endl;

i w kodzie 

.... // jakiś kod 
spr(1)
.... // jakiś kod 
spr(2)
.... // jakiś kod 
spr(3) // nawet średnika nie musisz wstawiać bo już tam jest

a jeszcze lepiej debuger.

---

To o czym piszesz, nazwa się testami jednostkowymi.

---

ok, Dzięki nawet nie wiedziałem. Bez klas jak się uczyłem to zawsze zdawało egzamin. Ale teraz staram się tak nie robić, bo jak się zagnieździ klasa w klasie to nie wiadomo co wyciąć, by sprawdzić staram się słuchać kompilatora i uważnie przejrzeć kod. I czasami naprawdę się udaje wyłapać, nawet gdy kompilator waruje.

---

Poczytaj o SOLID.