C++ Sprawdzanie liczby całkowitej.

Question

Nowość w C++. Problemy z poprawnie zapętlaniem podczas obsługi błędów. Próbuję sprawdzić, czy dane wejściowe użytkownika są liczbą całkowitą i są dodatnie.

do{ 
    cout << "Please enter an integer."; 
    cin >> n; 

    if (cin.good()) 
    { 
     if (n < 0) {cout << "Negative.";} 
     else {cout << "Positive.";} 
    } 
    else 
    { 
     cout << "Not an integer."; 
     cin.clear(); 
     cin.ignore(); 
    } 
}while (!cin.good() || n < 0); 

cout << "\ndone."; 

Po wpisaniu liczby niecałkowitej, pętla ulega złamaniu. Czuję, że nie rozumiem nieodłącznego użycia cin.clear() i cin.ignore() i statusu cin podczas tej pętli. Jeśli usuniemy cin.ignore(), pętla stanie się nieskończona. Dlaczego to? Co mogę zrobić, aby przekształcić go w elegancko funkcjonującą pętlę? Dziękuję Ci.

Answer 1

W Twojej gałęzi niecałkownikowej wywołujesz dalsze metody cin, dzięki czemu cin.good() zostaje zresetowany do wartości true.

Można zmienić swój kod na coś takiego:

while(1) { // <<< loop "forever" 
    cout << "Please enter an integer."; 
    cin >> n; 

    if (cin.good()) 
    { 
     if (n < 0) {cout << "Negative.";} 
     else { cout << "Positive."; break; } 
    }       // ^^^^^ break out of loop only if valid +ve integer 
    else 
    { 
     cout << "Not an integer."; 
     cin.clear(); 
     cin.ignore(INT_MAX, '\n'); // NB: preferred method for flushing cin 
    } 
} 

cout << "\ndone."; 

czy można uprościć go jeszcze tak:

while (!(cin >> n) || n < 0) // <<< note use of "short circuit" logical operation here 
{ 
    cout << "Bad input - try again: "; 
    cin.clear(); 
    cin.ignore(INT_MAX, '\n'); // NB: preferred method for flushing cin 
} 

cout << "\ndone."; 

Answer 2

int n; 

while (!(cin >> n)||n<0)//as long as the number entered is not an int or negative, keep checking 
{ 
cout << "Wrong input. Please, try again: "; 
cin.clear();//clear input buffer 

} 
//only gets executed when you've broken out of the while loop, so n must be an int 
cout << "Positive."; 

cout << "\ndone.";//finished! 

Powinno robić, co chcesz.