数字を入力する際に文字を弾く方法について

Question

最近、C言語の勉強を始めた初心者です。

数字を入力する際に0~100以外の数字と文字列を弾くようにしたのですが、
一つ困ったことがあります。

	printf("点数を入力してください。\n");	scanf("%d",&ten1);
	tena=getchar();
	while(ten1<0 || ten1>100){
		printf("値に誤りがあります。もう一度入力してください。\n");	scanf("%d",&ten1);
		tena=getchar();
	}

数字やアルファベット一文字を入力すると、1回だけエラー文を出してもう一度入力をさせます。
しかし、容量が大きくなるとその分エラー文が増えてしまいます。
（"aaaaa"なら5回、"あ"なら2回）
この表示を1回だけにするにはどうすれば良いでしょうか？
また、文字を弾く方法は調べながらやりましたがこれは正しいですか？

よろしくお願いします。

Answer 1

scanfで直接特定パターンの文字列を読み込むことはおすすめしません（パターンに合わない入力がストリームに残り続け、あとから別な読み込みで処分する必要が出てきます）。

誤入力があることを前提とするなら、fgetsで1行読み込んで、sscanfやstrtolなどのような関数を使って必要なデータを抽出するという方がすっきり書けます（1行読み込む関数にはgetsもありましたが、最大の長さを指定できないという問題点があったため、使用してはいけません）。

参考ページ

Comments

[kopokopo]

回答ありがとうございます。
なるほど・・・そのような関数もあるのですね。勉強になりました。
次回、プログラムを書く時はそちらの関数も使用してみることにします。

Answer 2

うーん、複数回エラーメッセージが表示されるのはコンパイラ依存のバグかもしれませんが…
バグを引き起こしているのは入力バッファの残りカスに起因していると思います。

たぶん、以下のように記述すればエラーを修正することができます。

int number;
scanf("%d", &number);
while(getchar() != '\n'); // scanf実行後この記述を追加

より良い修正方法としてはmaisumakunさんのおっしゃるようにfgets等を使用することですが、学習に際してscanfを使う場合には上記のように'\n'まで読み捨てを行うといいでしょう。

この理由についてはscanfの動作を知る必要があります。
scanfは文字列から特定パターンを読み込み、引数に読み取った値を代入しますが、以下のように動作しています。
0.パターンにマッチした読み取り可能な文字を見つけるまで読み捨てる ※%dなら0-9を見つけるまで 入力"a123b"なら"a"は捨てられる
0.パターンとして認識できる間読み取りを続ける ※%d 入力"a123b"なら3まで bは0-9では無い
0.パターンの認識から外れたところで読み取りを終了する ※"a123b"なら"b\n"が入力バッファに残る

入力バッファ自体はENTERキーを押下するところまで読み込まれるので、入力バッファの最後には必ず"\n"が含まれます。tena=getchar()はscanf()後に他の関数の動作が不安定なためにつけたものと思いますが、動作が不安定になるのはscanf実行後'\n'が最低１文字分入力バッファに残るためです。逆に言うと、残った入力バッファの終端は必ず'\n'になるので'\n'まで読み飛ばすことで入力バッファに残ったゴミを処理することができます。

Comments

[kopokopo]

回答ありがとうございます。
たしかにエラーが修正され、一度だけの表示になりました。
scanfの仕様とエラーが出る理由がよく理解出来ました。
この先、scanfを使用して動作に不具合を起こした時はこのことを思い出すようにします。