コンソールからのC++入力、整数の比較

Question

bool choose() { 
    int answer = 0; 
    while(answer != 1 || answer != 2) { 
     cout << endl << "Do you want to encrypt(enter 1) or decrypt(enter 2)?" << endl; 
     cin >> answer; 
    } 
    if(answer == 1) return true; 
    return false; 
} 

入力から行を読み込み、入力を整数と比較する最良の方法は何ですか？ 私は方法が分かりますcin、gets、getline(cin, answer)です。 私はどちらを使うべきですか？なぜですか？

現時点では、私は1または2を入力すると、まだwhileにとどまっているので、この方法は機能しません。

Answer 1

このコードはdo-whileループのための非常に良い候補である、とあなたは変数answerを初期化するために必要としていないことを使用している場合。

int answer; //= 0; no need to initialize! 
do { 

    cout<<"Do you want to encrypt(enter 1) or decrypt(enter 2)?"<<endl; 
    cin >> answer; 

}while(answer != 1 && answer != 2); 

そしてもちろん、あなたが他の人がすでに指摘したように&&を使用する必要があります。 :-)

ところで、ユーザーがhgjkhgなどの非整数入力を入力した場合はどうなりますか？ cin>>answerは入力を読み込むことができず、それは永遠にそこに残り、cinのフラグは失敗に設定され、cinはそれ以上の入力を読み取ることができません。つまり、ループは決して終了しません。

これを回避するために、そして、コードをより堅牢にし、あなたのようにそれを書くべきで完了させる：

int answer = 0; 
while(answer != 1 && answer != 2) { 

    cout<<"Do you want to encrypt(enter 1) or decrypt(enter 2)?"<<endl; 
    if (!(cin >> answer)) 
    { 
    cin.clear(); //clear the failure flag if there is an error when reading! 
    std::string garbage; 
    std::getline(cin, garbage); //read the garbage from the stream and throw it away 
    } 
} 

はい。 do-whileの代わりにwhileループを使用できます。あなたのコードにif(!(cin>>answer)) { ... }を追加するだけです。 :-)

Answer 2

値を正しく読み取っていますか。しかし、ループの状態は間違っています。それは||を使用して

while(answer != 1 && answer != 2) 

する必要があります何の数があなたの条件が間違って1と2の両方

Answer 3

に等しくないので、条件は常にtrueになります。 De Morgan's lawsすると、代わりに&&を使用する必要があります。

while(answer != 1 && answer != 2) {