pętla która wywołuje złe wyniki

Question

Witam ma problem ze sprawdzanie błędu. Mianowicie jak za pierwszym razem wpisuje 800.000 to wywala mi błąd że przecinek jest w błędnym miejscu. Ale za drugim razem program akceptuje taki wynik i nie wiem dlaczego. Powinno być 800.00 po taki jest format pieniędzy.

void Budget::convertBudgetStringToInt() {
//    bool test = false;
//    int indexStep = 0;
//    int loopCount = 0;
    while (true) {
        try {
            std::cout << "MMM" << std::endl;
            for (int j = 0 ; j < budgetString.size() ; ++j) {
                ++loopCount;
                if (budgetString[j] == '.') {
                    indexStep = j;
                    ++index;
                    loopCount = 0;
                }
            }
                for (int i = 0 ; i < budgetString.size() ; ++i) {
                    if (test == true) {
                        test = false;
                        indexStep = 0;
                        loopCount = 0;
                        std::cout << "Podaj budzet: " << std::endl;
                        getline(std::cin, budgetString);
                        continue;
                    }
                    else if (loopCount > 2 && test == false) {
                        test = true;
                        throw badBudgetStringPlaceDot();
                    }
                    else if (budgetString[i] == '-' && i == 0 && test == false) {
                        test = true;
                        throw badBudgetStringIsNegativeNumber();
                    }
                    else if (isalpha(budgetString[i]) && test == false) {
                        test = true;
                        throw badBudgetStringIsLetter();
                    }
                    else if (ispunct(budgetString[i]) && i == indexStep && test == false) {
                        test = true;
                        throw  badBudgetStringIsPunctuationMarks();
                    }
                    else if (isspace(budgetString[i]) && test == false) {
                        test = true;
                        throw badBudgetStringIsWhiteSpace();
                    }
                }
            budget = std::stoi(budgetString.c_str())*100;
            break;
        }
        catch (badBudgetStringPlaceDot &ref) {
            ref.message();
        }
        catch (badBudgetStringIsNegativeNumber &ref) {
            ref.message();
        }
        catch (badBudgetStringIsLetter &ref) {
            ref.message();
        }
        catch (badBudgetStringIsPunctuationMarks &ref) {
            ref.message();
        }
        catch (badBudgetStringIsWhiteSpace &ref) {
            ref.message();
        }
    }
}

 

Answer 1

Nie znając całego kodu trudno mi doradzić. Problem może leżeć albo w pre-inkrementacjach "++zmienna", albo w iteracji pentli. Dla przykładu size() zwraca ilosc elementów więc np dla słowa MaMa ilosc bedzie 4, natomiast uwzględniając to jako zmienne tablicowe będzie 3. bo iteracja idzie od 0, stąd tu gdzieś może wynikać ten problem.

np tu 

++j

 upewnij się czy aby 1 element ;] nie indeksował się od 1, a nie od 0 jak chcesz.