c言語 キーボード入力で文字列を数値に

Question

前提・実現したいこと

現在C言語でプログラミングを勉強しているのですが、
int num;
while(1){
scanf(%d, &num);
if(num == 1){
printf("yes");
break;
}else{
printf("もう１度入力してください。");
}
}

上のコードに文字を打ち込むとバグが起きてしまいます。
scanfの%dに文字を入力しているため、当然ですが、
これを文字を入力してもelseに処理するようにしたいです。
どのように書けば可能でしょうか？

またatoiを使っても判定されず「1」を入力しても上手く判定されませんでした。

発生している問題・エラーメッセージ

文字を入れてもバグが起こらないようにしたい。

また、
char num;
scanf(%c, &num);
if(num == 'y'){
    printf("yes");
    break;
  }else{
    printf("もう１度入力してください。");
　}
とした場合、「y」1文字を入力すれば問題ないですが、
「yu」など入れたときにもbreak;されてしまいます。
そこを「y」だけ入力したときだけとしたいです。

該当のソースコード

#include<stdio.h>
#include<string.h>
#include<stdlib.h>

int main(){
	int num;
	while(1){
		printf("入力:");
		scanf("%d", &num);
		
		if(num == 1){
			printf("yes");
			break;
		}else{
			printf("もう１度入力してください。\n");
		}
	}
}

試したこと

ほかにも
fgets()とsscanf()を使用して書いていましたが、それを使えば今回のようなバグは起こりませんが、別のバグとして、printfで表示した文がscanf後もう一度表示されるというバグが起きました。
＊上のコードではそれはなりませんでした。

補足情報（FW/ツールのバージョンなど）

Linux環境下で動くものを書いていただきたいです。

Comments

[hayataka2049]

fgetsとsscanfの方のコードもご提示ください。

[fana]

> バグが起きてしまいます

「バグ」とだけ言われても伝わりません．
具体的に現象を説明してください．

> これを文字を入力してもelseに処理するようにしたいです。

とのことですが，こちらで実行してみると，とりあえずelse側には処理が流れます（"もう１度入力してください。"が無限に表示される）ので，
観測している問題点が異なるのかもしれませんので．

[seele]

fgets,sscanfを使用する場合
scanfの部分を
int n;
char buf[128];
fgets(buf, sizeof(128), stdin);
sscanf(buf, "%d", &n);

としてあります。

バグとしては、英字を入力すると無限に表示されるというバグが起こってしまいます。
問題点として、そこであっています。

Answer 1

scanf(%d, &num); はコンパイルエラーになります。
質問をするときは、コンパイル可能なコードでお願いします。

scanf の実行が成功したかどうかは、返却値で判断します。

#include <stdio.h>  // scanf, printf

int main(void)
{
	int num;
	while (1) {
		if (scanf("%d", &num) == 1 && num == 1) {
			printf("yes");
			break;
		} else {
			while ((num = getchar()) != '\n') ; // 改行コードまで読み飛ばす
			printf("もう１度入力してください。");
		}
	}
}

ただし、このコードでは EOF への対応が欠けています。

Comments

[seele]

解答していただきありがとうございます。
参考にさせていただきます。

Answer 2

英字を入力すると無限に表示されるというバグ

については，入力バッファにゴミが残ってしまうために，次回のscanfもそのゴミが入力になってしまい，それがまた残って…

という話かと思うので，再入力させる前に入力バッファのゴミを捨てればよいでしょう．
elseのブロック内に，
while ( getchar() != '\n');
みたいなのを追加すれば良いかと思います．

※なお，
scanf("%d", &num);の入力に失敗した場合，numの値は変化しないので，
int num;の値がたまたま1だった場合にif(num == 1)の条件を満たしてしまう可能性があります．
なので，int num=0; とかして，1でない値で初期化しておくべきでしょう．

Comments

[seele]

解答いただきありがとうございます。
早速試してみます。