try catch zapętla się na sprawdzaniu czy int jest literą

Question

Witam mam dziwny problem bo w try catch chce sprawdzić czy mój int jest z przedziału od 1 do 6 ale użytkownik może się pomylić i wpisać np A. I tu mam problem bo wszystko działa oprócz sprawdzenia czy przypadkiem ktoś nie napisał litery. Chodzi o to że mam zapętlenie i wyskakuje mi informacja cały czas i do tego wyskakuje mi wyjątek tooSmall zamiast isAlpha

void Manage::setWhatMenuOption() noexcept {
    while (true) {
        try {
            std::cin >> whatMenuOption;
            if (whatMenuOption < 1) {
                throw badWhatMenuOptionTooSmall();
            }
            if (whatMenuOption > 6) {
                throw  badWhatMenuOptionTooBig();
            }
            if (isalpha(whatMenuOption)) {
                throw badWhatMenuOptionIsAlpha();
            }
            if (whatMenuOption >= 1 && whatMenuOption <= 6) {
                break;
            }
        }
        catch (badWhatMenuOptionTooSmall &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionTooBig &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionIsAlpha &ref) {
            ref.message();
        }
    }
}

 

Comments

Jakiego typu jest zmienna whatMenuOption?

---

int

Answer 1

Strumień std::cin "spodziewa się" liczby. Jeżeli wpiszesz jakąś literę to do twojej zmiennej trafi zero i strumień ustawi flagę badbit.
Po wczytaniu danych do zmiennej powinieneś sprawdzić czy operacja się powiodła (możesz użyć std::cin.good()  lub std::cin.fail())

Answer 2

a jakiego typu jest whatMenuOption? bo może problem jest w konwersji, np widzi to nie jako "CHAR" ale jako litere- zapisaną cyfrą z klawiatury.

W dokumentacji zwróć uwagę: 

Parameters
c
Character to be checked, casted to an int, or EOF.

więc tutaj powinieneś jako argument podać stringa- który będzie zrzutowany na inta, a nie podawać int. 

Comments

czyli wystarczy jak tego inta przekonwertuję na string lub char i będzie ok

---

tak podejrzewam, daj znać jak sprawdzisz.... albo znajdziesz rozwiązanie, dziś wieczorkiem mogę się pobawić- jak nadal jest problem.

---

ok mam coś takiego działa ale nie wiem czy tak robi się sprawdzanie błędów

void Manage::setWhatMenuOption() noexcept {
    while (true) {
        try {
            getline(std::cin , whatMenuOptionString);
            if (whatMenuOptionString.empty()) {
                throw badWhatMenuOptionIsEmpty();
            }
            if (isalpha(whatMenuOptionString[0])) {
                throw badWhatMenuOptionIsAlpha();
            }
            if (isspace(whatMenuOptionString[0])) {
                throw badWhatMenuOptionIsWhiteSpace();
            }
            if (ispunct(whatMenuOptionString[0])) {
                throw  badWhatMenuOptionIsPunctuationMarks();
            }
            if (whatMenuOptionString[0] < '1') {
                throw badWhatMenuOptionTooSmall();
            }
            if (whatMenuOptionString[0] > '6') {
                throw badWhatMenuOptionTooBig();
            }
            if (whatMenuOptionString[0] >= '1' && whatMenuOptionString[0] <= '6') {
                whatMenuOption = std::stoi(whatMenuOptionString.c_str());
                std::cout << whatMenuOption << std::endl;
            }
            if (whatMenuOptionString[1]) {
                throw badWhatMenuOption();
            }
            break;
        }
        catch (badWhatMenuOptionIsEmpty &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionTooSmall &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionTooBig &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionIsAlpha &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionIsWhiteSpace &ref) {
            ref.message();
        }
        catch (badWhatMenuOptionIsPunctuationMarks &ref) {
            ref.message();
        }
        catch (badWhatMenuOption &ref) {
            ref.message();
        }
    }
}

 

---

można przed pętlą while dodać 

std::cin.ignore(1000, '\n');

to nie będzie od razu wyrzucany błąd ze string jest pusty.