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! 

必要な操作を行う必要があります。