C++ Проверка целого числа.

Question

Новое на C++. Имеет проблемы с правильным циклом при обработке ошибок. Я пытаюсь проверить, является ли пользовательский ввод целым, и является положительным.

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."; 

Когда вводится нецелое число, цикл прерывается. Я чувствую, что я не понимаю, как использовать cin.clear() и cin.ignore() и статус cin во время этого цикла. Если я удалю cin.ignore(), цикл станет бесконечным. Почему это? Что я могу сделать, чтобы сделать это в элегантно функционирующий цикл? Спасибо.

Answer 1

В вашей нецелой ветке вы вызываете дополнительные методы cin, поэтому cin.good() получает значение true.

Вы можете изменить свой код, чтобы что-то вроде этого:

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."; 

или вы можете упростить его еще больше, как это:

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! 

Должен делать то, что вы хотите.