while文がうまくいきません。

Question

入力が正しくない場合に、続けるを選んだ後に、再度入力をうながしているのですが、なぜか入力の処理をさせてもらえずに、”入力値が正しくありません”と表示されてしまい、入力ができなくなってしまいました。
結構な時間悩んでもわからなかったので質問させていただいています。
分かる方助けてください。

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

int inputCheck(char myHand);
int main()
{
	int flg = 0, keep;
	char myHand, youHand;
	puts("じゃんけんゲーム！");

	while(flg == 0)
	{
		printf("手を入力してください【グー:1 チョキ:2 パー:3】:");
		scanf("%c", &myHand);
		if(inputCheck(myHand) == 1) flg = 1;
		else
		{
			flg = 0;
			puts("入力値が正しくありません。\n");
			printf("終了しますか？【終了:0 続ける:1】：");
			scanf("%d", &keep);
			if(keep == 0) exit(0);
		}
	}

	return 0;	
}

// 入力をチェックする関数　引数：入力値　戻り値：数値1 数値ではない0
int inputCheck(char myHand)
{
	if(myHand <= '3' && myHand >= '1')
	{
		return 1;	
	}

	return 0;
}

Comments

[jimbe]

> 入力ができなくなってしまいました
具体的に, 実行から終了(?)までの画面の状態を画像なりコピペなりでご質問に追加してくださいますか.

[L4zy]

scanf("%c", &myHand);の部分を%sにするのでも大丈夫ですか？

[jimbe]

大丈夫かと言われれば大丈夫ではありません.
scanf は比較的入門時に扱う関数ですが, 実は使い方は結構難しく, それを説明するのも(入門時ということもあって)難しいです.
どのような入力がされた時にどのような動作をするのか/させたいのかを調べながら試してみるのが良いかと思います.

Answer 1

分かる方助けてください。

おそらく、「続ける」を入力するときに「1（エンター）」のように入力したかと思いますが、この改行が入力として残って、次の「手の入力」として解釈されてしまいます。

簡易的には1文字読み捨てるような方法もありますが、ある程度安定して入力したい場合は、「fgetsで1行取ってくる→sscanfで行ないから必要な値をピックアップ」のような方法が取られることもあります。

Comments

[maisumakun]

それだけでは正常に動きません（char1文字の領域を超えて書き込む危険があります）。

[L4zy]

ということは、もし3文字入力したら、2文字捨てなければならないということですか？

Answer 2

scanfの"%c"指定は結構厄介な動きをします。
"%s"で文字列として受け取って一部を読み取るか、
" %c"とすると、回避できるかも。