for文による連続scanf(%c)で空の引数がそのままループしてしまう

Question

連続して一文字を入力。数字以外を入力するとエラーメッセージを表示し、次の入力に移行する。Eを入力すると終了。それまで出てきた数字らのSumとAverageを出力。

for文でscanfで連続入力するシステムです。if文でEとそれ以外の文字、次のifで数字かそれ以外と分けました。elseに"Illegal input:"を表示するところがあるのですが、入力し判別されたあとに、常に"Illegal input:が表示されてしまいます。引数の状態を調べたところ、判別されたあとに空の引数になって、もう一度if文に入っていました。調べてもどうしてなのかわかりません。

現状の表示＊Linaxのgccで実行。引数aの状態をprintfで表示している。

５
a=5
a=

Illegal input:

t
a=t
Illegal input:t
a=

Illegal input:

3
a=3
a=

Illegal input:

E
a=E
Sum is:8.000000
Average is:4.000000

該当のソースコード

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

int average = 0;
float count = 0;

void judge(char x)
{
      if(x!='E'){
          if(x=='1'){
            average += 1;
            count += 1;
          }else if(x=='2'){
            average += 2;
            count += 1;
            .
            .
            .
          }else if(x=='9'){
            average += 9;
            count += 1;
          }else{
            printf("Illegal input:%c\n", x);
          }
      }
}

int main()
{
      char a;
      int i;
      for (i=0;a!='E';i++){
           scanf("%c", &a);
           printf("a=%c\n", a);
           judge(a);
      }
      printf("Sum is:%d\n", average);
      printf("Average is:%f\n",average/count);
      return 0;
}

試したこと

int main(void)にしてみた　→　変わらず
scanf(%*c%c)にしてみた　→　変わらず
if(a!='E' && a!=' ')にしてみた　→　変わらず
judge()にreturn　→　変わらず

Comments

[cateye]

参考：http://nanoappli.com/blog/archives/3550

Answer 1

%cは改行コードも読み込み対象なので、そのままでは変数aに改行コードが渡されて意図しない処理となってしまいます。
%cの手前に半角スペースを１個入れてください。空白と改行コードを無視するようになります。
→ scanf(" %c", &a);

あと、ご質問のコードでは変数'a'が初期化されないままfor文の条件判定で使われています。char a = 0;のように必ず初期化してください。

Comments

[shiki_willow]

丁寧なご説明ありがとうございます。勉強になりました。

Answer 2

3 Enterの入力で、最初のscanf("%c", &a);では'3'が取得されますが、次のscanf("%c", &a);では、改行の'\n'が取得されます。
'\n'をjudgeに渡すと、printf("Illegal input:%c\n", x);に行きます。

isspace(a)が真の時は、何もせずに繰り返すといいでしょう。

Comments

[shiki_willow]

\nが取得されることを授業で言われたような気がします。ご説明ありがとうございます。

Answer 3

scanf の "%c" を " %c" に変えてみてください。
% の前にスペースを置くということです。
どうなりますか？
なぜそうなるか分かりますか？

それから、変数 a が未初期化のまま 'E' と比較されています。
また、変数 i が無意味になっています。

Answer 4

scanfというのは、Enter押して初めて解釈が行われますんで、あなたの期待するような動作にはならないかと。

Comments

[kazuma-s]

scanf の %c に Enter を特別扱いする機能はありません。

キーボード入力の場合 Enter を押すまでプログラムに文字が
渡されないのは、キーボードドライバ(端末ドライバ)が
行バッファリングを行っているからです。

Linux あるいは、Windows で Cygwin を使っているなら、mainの
最初で system("stty cbreak"); を実行し、main の終了直前に
system("stty -cbreak"); を実行して、端末ドライバにバッファ
リングをやめさせれば、Enter にかかわらず、入力文字が 1文字
ずつプログラムに渡されます。

[y_waiwai]

そう、特別扱いしないからこそ、％ｃにリターンコードが入って返され、期待するような動作にはなりません
提示のコードには、バッファリングを無効にしてる様子もないですしね