do ... whileループは私が欲しいものを表示していません

Question

これは基本的に私はこのループを今日では約7時間働いています。それは私がそれがどのようにすべきか振る舞いません。 cin（行はcharですが、列はintです。したがって、たとえばa1の代わりにaa1を入力すると、新しいcoordを要求する前にinvalidCoordが2回呼び出されます）。

do 
{ 
    confirmShot = false; 
    validShot = false; 
    failShot = false; 
    cout << enterCoord; 
    cin >> row >> column; 
    if (cin.fail()) 
    { 
     cin.clear(); 
     cout << invalidCoord << endl; 
     failShot = false; 
     continue; 
    } 
    if (failShot == true) 
    { 
     row = toupper(row); 
     xCoord = column - 1; 
     yCoord = static_cast<int>(row - 'A'); 
     if(xCoord > 9 || xCoord < 0 || yCoord > 9 || yCoord < 0) 
     { 
      cout << invalidCoord << endl; 
      validShot = false; 
     } 
     else 
     { 
      validShot = true; 
     } 
    } 
    else 
    { 
     continue; 
    } 
    currentTile = tileToSymbol(computerBoard[xCoord][yCoord]); 
    if (validShot == true) 
    { 
     switch(currentTile) 
     { 
      case '~': 
      case 'a': 
      case 'b': 
      case 's': 
      case 'd': 
      case 'p': 
       cout << "You have already shot at " << row << column << "." << '\n' 
        << "Please pick a new coordinate!" << endl; 
       validShot = false; 
       confirmShot = false; 
       break; 
      default: 
       confirmShot = true; 
     } 
    } 
    else 
    { 
     continue; 
    } 
} 
while(!confirmShot); 

出力：

Your shot: Enter the coordinates for a shot (e.g. B2) cc2 
Invalid coordinates! 
Pick a row value between A and J 
and a column value between 1 and 10. 
Enter the coordinates for a shot (e.g. B2) Enter the coordinates for a shot (e.g. B2) cc2 
Invalid coordinates! 
Pick a row value between A and J 
and a column value between 1 and 10. 
Enter the coordinates for a shot (e.g. B2) Enter the coordinates for a shot (e.g. B2) c2 
Enter the coordinates for a shot (e.g. B2) c2 

また、私はループはもう終了している間...やる取得する方法を見つけ出すことはできません。それは、誰かが2番目の文字として非整数を入力した場合に、フェイルステートを処理しなければならないことに気付く前に、正常終了していました。

私が話していた誰かが次のセクションにスキップする必要があると言い、プログラムをロックすると、プログラムがロックアップします。そこにコンティニューコードがあり、うまくいきました。

私は本当にこのループを正しく動作させる必要がありますが、私は自分の能力を使い果たしました。 failstateを処理してinvalidCoordを2回表示させてもいいですか（正直なところ、それはなぜ起こっているのかわかりません）、有効なエントリを受け取った後にループを終了してその日を保存しますか？

Answer 1

私はそれを単純化しすぎている可能性があります。私が間違っている場合はコメントしてください。単にif文に誤りがあると思います。

failShotがfalseに初期化されたため、if (failShot == true)が呼び出されないように見え、if (cin.fail())をチェックするたびに、再びfalseとして更新されます。これを正しく理解している場合は、cinが失敗しない場合はif (failShot == true)が呼び出されます。この場合、failShotをTrueに初期化するとエラーが修正されます。

は変更してみてください：

failShot = false; 

へ：

failShot = true; 

Answer 2

OK、あなたの問題はstd::cinオブジェクトの理解です。それは、キーボードバッファから文字を取り出すように入力した文字を解析中にエラーが発生したときに、この位置を増分位置pos_type p = -1に

std::cin開始する仕組み

これは、そのように、eofの位置を設定します次回実行時にはeofを返し、何もしません。消費するstd::cinニーズを意味している

// when you enter for expel: c1 - which is correct input, it prints 
p = 2; // advanced two position in the buffer until EOF 
// when you enter wrong input: cc1 - it prints 
p =-1; 
// next run just after the error message, it will print 
p = 3;//this number is the number of character read last time and it coincide with EOF 
// right after, it will prompt you for another input 

（唯一興味深い部分がここに印刷された）出力を以下になるcin<<row<<column;

cin << row << column ; 
cout << "p =" << cin.tellg(); 

：下のこのスニペットを挿入し、あなたのコードでこれを説明するため

バッファ内のすべての文字が-1に設定され、文字の抽出が再開されます。

これは、メッセージが重複している理由です。どのようにそれを修正するには？

これは醜いハックです：

//inside the loop under 
cin.clear() 
cin.seekg(0); // rewind...uuurrrgg, ugly!! but it works 

私が提案する何をあなたのコードを再構築することです。ブール論理を使用して、美しいコードフローチャートを設計します。このようなコードベースのエラーは、コードがよく書かれたコードではなくスパゲッティコードであることを意味します。

頑張ってください！