segmentation fault - problem z lokalizacją naruszenia pamięci

Question

Witam. Jak w temacie mam problem ze zlokalizowaniem błędu segmentation fault. Wiem tylko tyle, że debugger zatrzymuje mi się w linii oznaczonej "►". Wrzuciłem tylko część kodu w której występuje błąd wszystko co potrzebne jest zawarte w tej części powyżej jest tylko ściągnięcie pewnych danych (część mało istotna nie sprawiająca problemu). Będę wdzięczny za każdą pomoc.

fstream dopis (nazwa.c_str(), ios_base::app);
        string nazwa_zakupionego;
        string cena;
        char pomoc;
        string tmp_str;                                     //tmp_str string powstający w celu zczytania interesujacej nas czesci ceny
        int cena_zl=0, cena_gr=0, dlugosc=0, j=0, tmp=0;    //tmp licba do funkcji atoi w celu zamiany stringa ceny na inta
        cout<<"Podaj nazwę zakupionego przedmiotu:\n";
        cin>>nazwa_zakupionego;
        cout<<"Podaj cenę:\n";
        cin>>cena;
        pomoc=cena[j];
        while (pomoc!='.'||pomoc!=',')
        {
            j++;
►           pomoc=cena[j];
        }
        tmp_str=cena[j+1]+cena[j+2];
        cena_gr = atoi(tmp_str.c_str());
        string tmp_string;
        while(tmp<j)
        {
            tmp_str=tmp_string+cena[j];
            tmp++;
        }
        cena_zl=atoi(tmp_string.c_str());
        dopis<<cena_zl<<"."<<cena_gr<<"\t"<<nazwa_zakupionego<<";"<<endl;
        dopis.close();

 

Comments

A jak dodasz ograniczenie, że j nie może być większe od strlen(cena) ?

Answer 1

Noo, błąd jest dosyć logiczny inkrementujesz zmienną j do momentu, w którym wykracza ona poza tablicę "cena". Nie ma żadnego mechanizmu który by przerywał pętlę gdy osiągnie ostatni znak w "cena".

Lepiej było by użyć kodu:

for (j = 0; j < cena.size(); j++)
{
	if( cena[j] == '.' || cena[j] == ',' )
		break;
}

Poza tym w dalszej części kodu też będą występować te problemy. Powinieneś wprowadzić mechanizmy kontrolne.

I jeżeli chcesz wczytać liczbę z przecinkiem, polecam ten artykuł: https://4programmers.net/C/Konwersje_int_na_string_i_string_na_int#string-na-int

Pozdrawiam i życzę owocnej pracy 

Comments

Dzięki wielkie pomogłeś , a znalazłem też błąd u siebie. Błąd głupi , ale ważne, że znaleziony: warunek do "while" zawsze prawdziwy( powinno być && a nie ||)=>pętla nieskończona i się sypie.