C++の入力検証中にループが終了しない

Question

私はイントロC++コンピュータサイエンスコースの学生です。ここに初めて投稿しました。私たちはちょうどwhileループについて学んだことがありますが、割り当てには必要がありませんが、この割り当てに関する入力検証を試みています。このプログラムは、数字のリストを読み、最初と最後の8つがあるリスト内の位置を把握することを目的としています。したがって、私が4つの数字（1,8,42,8）のリストを持っていれば、最初と最後の8つの位置は2と4です。セットのサイズはユーザーによって決まります。

私はwhileループを作ろうとしていましたが、ユーザが実際に入力したものが実際に数字だったのを確認しましたが、 "。"または "a"ループは無限に進み、終了しません。私は自分の誤りを見つけることができず、私の教科書に書かれているものとまったく同じ構文を使っていると私には分かります。誰かが私のwhileループで間違っていることを私に見せてもらえますか？

int numbers,   //How large the set will be 
    num,    //What the user enters for each number 
    first8position = 0, //The first position in the set that has an 8 
    last8position = 0; //The last position in the set that has an 8 

//Prompt the user to get set size 
cout << "How many numbers will be entered? "; 
cin >> numbers; 

//Loop to get all the numbers of the set and figure out 
//which position the first and last 8 are in 
for (int position = 1; position <= numbers; position++) 
{ 
    cout << "Enter num: "; 
    cin >> num; 

    //If num isn't a digit, prompt the user to enter a digit 
    while (!isdigit(num)) 
    { 
     cout << "Please enter a decimal number: "; 
     cin >> num; 
    } 

    //If num is 8, and first8position still isn't filled, 
    //set first8position to the current position. 
    //Otherwise, set last8position to the current position. 
    if (num == 8) 
    { 
     if (first8position == 0) 
      first8position = position; 
     else 
      last8position = position; 
    } 


} 

//If the set had an 8, print what its position was 
if (first8position != 0) 
    cout << "The first 8 was in position " << first8position << endl; 

//If there was more than one 8, print the last 8 position. 
//Otherwise, the first and last 8 position are the same. 
if (last8position != 0) 
    cout << "The last 8 was in position " << last8position << endl; 
else 
    cout << "The last 8 was in position " << first8position << endl; 

//If there were no 8s, say so. 
if (first8position == 0) 
    cout << "Sorry, no eights were entered."; 

return 0; 

}

Answer 1

2つの問題は、あなたの無限ループにつながるされています

まず、cin >> numで、あなたは整数値を読み込むようにしてください。ユーザがaまたは.のようなものを入力した場合、これは整数値の開始にはなりません。何も読み込まれず、aまたは.が入力バッファに残っています。したがって、それ以降のすべてのcin >> numはすぐに（ユーザーに何かを入力する機会を与えずに）失敗します。なぜなら、aまたは.は依然として入力バッファーに残っているからです。そのような場合は、これらの文字をcinから消費する必要があります。 cin.ignoreを使用して、この場合に設定されているfailbitをリセットする必要があります。

次に、ASCII -value c、すなわちc >= 48 && c <= 57場合、数字である場合にisdigit(int c)チェックに注意してください。したがって、ユーザが48との間の数字を入力するまで、isdigit(num)のチェックは失敗します。

入力エラーを処理する方法を示す次のコードを参照してください。それが役に立てば幸い。

int main() { 

    int num; 
    cin >> num; 
    while (!cin.eof() && cin.fail()) { // failure when extracting an integral value? 
     cout << "not an integral value." << endl; 

     // clear failbit 
     cin.clear(); 

     // remove characters that are still in the input buffer (until next end of line) 
     cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n'); 

     // next try to read in an integer 
     cin >> num; 
    } 
    if (!cin.eof()) { 
     cout << "juu:" << num << endl; 
    } 
}