成功するまで繰り返し、失敗を処理するループを構築する方法

私は独学のプログラマーです。私は約1.5年前にプログラミングを始めました。今、私は学校でプログラミングクラスを始めました。プログラミングクラスは1/2年間ありましたが、今はもう半分あります。

クラスでは、C ++でプログラミングする方法を学びます（これは、始める前に、私がすでにかなりよく使用することを知っていた言語です）。

このクラスでは何の問題もありませんでしたが、明確な解決策を見つけることができなかった問題が繰り返し発生しています。

問題は次のようになります（疑似コード）：

 do something
 if something failed:
     handle the error
     try something (line 1) again
 else:
     we are done!

これはC ++の例です

このコードはユーザーに数値の入力を求め、入力が有効になるまで入力を続けます。cin.fail()入力が無効かどうかを確認するために使用します。ときcin.fail()であるtrue私が呼び出す必要がありますcin.clear()し、cin.ignore()ストリームからの入力を取得し続けることができるように。

このコードはをチェックしないことを知っていますEOF。私たちが書いたプログラムは、そうすることを期待されていません。

これが私が学校での課題の1つでコードを書いた方法です。

for (;;) {
    cout << ": ";
    cin >> input;
    if (cin.fail()) {
        cin.clear();
        cin.ignore(512, '\n');
        continue;
    }
    break;
}

私の先生は、私はこれを使うべきではないbreakとcontinue言っていました。彼は代わりにレギュラーwhileかdo ... whileループを使うべきだと提案しました。

このようなループを表す最も簡単な方法は、breakとを使用しているように思えcontinueます。私は実際にそれについてかなり長い間考えましたが、より明確な解決策を実際には思い付きませんでした。

彼は私に次のようなことをして欲しかったと思います：

do {
    cout << ": ";
    cin >> input;
    bool fail = cin.fail();
    if (fail) {
        cin.clear();
        cin.ignore(512, '\n');
    }
} while (fail);

私にfailは、追跡するために呼び出される変数もあり、入力失敗のチェックが1 回ではなく2回行われるため、このバージョンはかなり複雑に見えます。

また、次のようなコードを記述できることもわかりました（短絡評価を悪用しています）。

do {
    cout << ": ";
    cin >> input;
    if (fail) {
        cin.clear();
        cin.ignore(512, '\n');
    }
} while (cin.fail() && (cin.clear(), cin.ignore(512, '\n', true);

このバージョンは、他のバージョンとまったく同じように機能します。breakまたは使用せずcontinue、cin.fail()テストは1回だけ実行されます。しかし、私がこのような「短絡評価ルール」を悪用することは、私には正しくないと思われます。私の先生もそれを望んでいないと思います。

この問題はcin.fail()チェックだけに当てはまるわけではありません。私はこれを、他の多くの場合に使用breakしcontinue、条件が満たされるまでコードのセットを繰り返すことを含み、条件が満たされない場合は何かを実行する必要があります（呼び出しcin.clear()や例cin.ignore(...)からcin.fail()）。

私はコースをずっと使用breakし続けcontinueましたが、今では私の先生はそれについて不平を言うのをやめました。

これについてのあなたの意見は何ですか？

私の先生は正しいと思いますか？

この種の問題を表すより良い方法を知っていますか？

私はifステートメントを少し異なるように書くので、入力が成功したときにそれが取られます。

for (;;) {
    cout << ": ";
    if (cin >> input)
        break;
    cin.clear();
    cin.ignore(512, '\n');
}

それも短いです。

先生が好むかもしれない短い方法を示唆しています：

cout << ": ";
while (!(cin >> input)) {
    cin.clear();
    cin.ignore(512, '\n');
    cout << ": ";
}

あなたが努力しなければならないのは、生のループを避けることです。

複雑なロジックをヘルパー関数に移動すると、突然物事が非常に明確になります。

bool getValidUserInput(string & input)
{
    cout << ": ";
    cin >> input;
    if (cin.fail()) {
        cin.clear();
        cin.ignore(512, '\n');
        return false;
    }
    return true;
}

int main() {
    string input;
    while (!getValidUserInput(input)) { 
        // We wait for a valid user input...
    }
}

for(;;)悪いのはそれほどではありません。次のようなパターンほど明確ではありません。

while (cin.fail()) {
    ...
}

またはSjoerdが言ったように：

while (!(cin >> input)) {
    ...
}

このようなものの主な対象者は、あなたが仲間のプログラマーであると考えてみましょう。そのようにして、最後にブレークをスタックしたか、継続してブレークをジャンプした理由を覚えていない自分の将来のバージョンを含みます。あなたの例からのこのパターン：

for (;;) {
    ...
    if (blah) {
        continue;
    }
    break;
}

...他のパターンと比較してシミュレーションするには、数秒の追加の思考が必要です。これは難しい規則ではありませんが、continueを使用してbreakステートメントをジャンプするのは面倒で、役に立たないので賢く、ループの最後にbreakステートメントを配置することは、たとえそれが機能したとしても、異常です。通常は両方breakとcontinueするために使用されている途中でその最後のいずれかで彼らの見て、ループまたはループの反復を避難そうでない場合でも、少し奇妙な感じ。

それ以外に、良いもの！

学校のロジックインストラクターはいつも、ループに入る入り口と出口は1つだけであるべきだと言って、それを私の脳に押し込みました。そうでない場合は、スパゲッティコードの取得を開始し、デバッグ中にコードがどこに行くのかを知りません。

現実には、コードの読みやすさは非常に重要であると感じています。そのため、他の誰かがコードの問題を修正またはデバッグする必要がある場合でも、簡単に実行できます。

また、この外観を何百万回も実行しているためパフォーマンスの問題である場合は、forループの方が高速かもしれません。テストはその質問に答えます。

C ++は知りませんが、コード例の最初の2つは完全に理解しました。continueがforループの終わりに行き、それを再びループすると仮定しています。whileの例は完全に理解しましたが、for（;;）の例よりも理解しやすかったです。3つ目は、それを理解するためにいくつかの思考作業を行わなければならないでしょう。

したがって、読みやすく（c ++を知らない）、私のロジックインストラクターがwhileループ1を使用すると言っていたのは、私のためです。

私にとって、疑似コードの最も簡単なC変換は

do
    {
    success = something();
    if (success == FALSE)
        {
        handle_the_error();
        }
    } while (success == FALSE)
\\ moving on ...

この明白な翻訳が問題である理由がわかりません。

おそらくこれ：

while (!something())
    {
    handle_the_error();
    }

シンプルに見えます。