ループ中でのscanfおよびcin

あまりに基礎的な質問ですが，ループ中の入力関数が期待する動作になりません．
コードを載せます
while (1) {
int key;
scanf("%d", &key);
if (key == 1) break;
}
期待する動作は1が入力されるまでループし続けるというものですが
scanfは一度しか実行されずループし続けます，cinに変えても同様でした
何が原因か分かる方，ご教示ください

No.4 回答者： chie65535 回答日時： 2009/07/17 12:15

http://www9.plala.or.jp/sgwr-t/c/sec05.html
の「５－４．scanf()の注意事項」を読みましょう。

＞scanfは一度しか実行されずループし続けます

いいえ。scanfは実行されています。

「２度目以降のscanfを呼んだ時、入力バッファに変な物（%dとして受け付けない物）があると、scanfは入力バッファをそのままにして、何もしないで戻ってくる」ので「まるでscanfが実行されていないように見える」だけです。

scanfは「変換できた個数を返す」と言う仕様になっているので
while (1) {
int key;
if (feof(stdin)) break; //EOFの時は終了
if (scanf("%d", &key) != 1) { //予期しない入力なら
scanf("%*[^\n]%*c"); //改行も含めて読み飛ばす
} else { //変換できたなら
if (key == 1) break; //1なら終了
}
}
など、scanfの戻り値を確認すると良いでしょう。

よく「scanfは、初心者が絶対に使ってはいけない関数」などと暴言を吐く人が居ますが、そんな事はありません。

・scanfの戻り値をちゃんと確認すれば予期せぬ入力の判定は可能
・予期せぬ入力を読み飛ばす方法があるので、ちゃんと読み飛ばす

の２つをしっかり行えば、初心者でもちゃんと使える関数です。と言うか、変換の便利さ、書式の豊富さから言えば「超初心者向けの関数」です。

「scanfを使わずに、色々とゴチャゴチャ小細工する」って方が、よっぽど「上級者にしか出来ない高等テクニック」です。

No.3 回答者： notnot 回答日時： 2009/07/16 20:25

scanfは、初心者が絶対に使ってはいけない関数です。

（中上級者も使い道無いですけど）
(1) fgetsとsscanfを組み合わせて使う
(2) sscanfの戻り値を確認する

char line[1000];
while(1){
int key;
if(!fgets(line,sizeof line,stdin)) break;
if(sscanf(line,"%d",&key)==1 && key==1) break;
}


もし、例題にscanf関数が出てくる入門書をお使いなら、別の本を探しましょう。

No.2 回答者： jacta 回答日時： 2009/07/16 19:35

このやり方だと、scanfは整数値として解釈できる入力がないと、何度でも再試行してしまいます。

期待した入力が得られなかった場合、行末まで読み飛ばす処理を入れるべきです。

なお、stdinのような入力ストリームに対してfflushを呼び出すと未定義の動作になりますので、やってはいけません。

参考URL：http://www.kijineko.co.jp/tech/superstitions/ffl …

No.1 回答者： aigaion 回答日時： 2009/07/16 19:23

次の１行をscanfの次の行に追加したらいけるんじゃ？

fflush(stdin);

標準入力(stdin）のバッファを吐き出せということです。
入力というのはすぐにプログラムに渡されるわけではなく一定量が溜まってから渡されます。
これを一定量たまるまで待つのではなくすぐに渡せというのが上の意味です。