scanf関数、数値、文字、エラー処理

Question

N個の整数を入力し、出力した物の合計、平均、標準偏差を出力するプログラムを考えています。
8割方できたのですが、最初にscanf関数で数値以外を入力したときにエラーを表示させて、再度数値の入力を求めるようにしたいです。
よろしくお願いします。

#include <stdio.h>
#include <math.h>
#define N 10

main()
{
        int i, data[N];
	double sum=0;
        
	printf("%d", N);
	printf("個の整数を入力してください\n");
        
	for(i = 0 ; i < N ; i++)
        {
                printf("入力データ = ");
		scanf("%d", &data[i]);
        }
        
	printf("%d", N);
	printf("個の整数を出力します\n");
	
	for(i = 0 ; i < N ; i++)        
	{
		printf("出力データ = ");
		printf("%d\n", data[i]);
		
		sum += data[i];
	}
	
	printf("%d", N);
	printf("個の整数の合計を出力します\n");
	printf("合計 = %.0f\n", sum);

	printf("%d", N);
	printf("個の整数の平均と標準偏差を出力します\n");
	
	double ave = sum/N;
	double sd = 0;
	
	for(i = 0 ; i < N ; i++)
	{
		sd += (data[i]-ave)*(data[i]-ave);
	}
	sd /= N;
	printf("平均 = %.2f\n標準偏差 = %.2f\n", ave, sqrt(sd));
	
	return 0;
}

Answer 1

不正な入力を想定する端末入力にscanfを使うのは間違った考え方です。fgets+sscanfを使います。

char line[1000];
～～～
    for(i = 0; i < N; i++){
        do{
            printf("入力データ %d = ",i);
            if(!fgets(line, sizeof line, stdin)){
                "エラー処理";
            }
        }while(sscanf(line, "%d", &data[i])!=1);
    }

Comments

[yatai]

ありがとうございます。
勉強中なので助かります。
こちらをmain()の前に挿入する形で大丈夫でしょうか？

[otn]

コードの意味がわからないのでしょうか？
どの辺が分かりませんか？

[yatai]

すみません。まだ勉強不足で、曖昧です。
自分でも調べてみます。
charをmain()の中、教えて頂いたforを最初のforに置き換える
と考えたのですがどうでしょうか？

[otn]

入力処理を書き換えたいという質問に対して、入力処理を回答したので、元の入力処理をそれで置き換えるのが当然でしょう。
質問が来たのでびっくりしたのですが、回答のコードが入力処理であることが分からなかったのかな？と思いました。

[yatai]

失礼いたしました。
コードを書き換えたところ、文字を入力しても大丈夫になりました。
しかし、入力処理がループするようになってしまいました。(今はdefineで10個の入力にしているのに)
対策はあるでしょうか？

[otn]

> 入力処理がループするようになってしまいました。
具体的にどういうことでしょうか？

[yatai]

本来ならば10個のデータを入力後、合計などが出力されるのですが、
エラー処理を導入後は延々と入力を求められ、合計などが出力されなくなりました。

[otn]

コードを見ないと何とも。

[yatai]

お願い致します。

#include <stdio.h>
#include <math.h>
#define N 10

main()
{
int i, data[N];
double sum=0;
char line[1000];

printf("%d", N);
printf("個の整数を入力してください\n");

for(i = 0 ; i < N ; i++)
{
do
{
printf("入力データ = ");
scanf("%d", &data[i]);

if(fgets(line, sizeof line, stdin))
{
"エラー処理";
}
}while(sscanf(line, "%d", &data[i])!=1);
}

printf("%d", N);
printf("個の整数を出力します\n");

for(i = 0 ; i < N ; i++)
{
printf("出力データ = ");
printf("%d\n", data[i]);

sum += data[i];
}

printf("%d", N);
printf("個の整数の合計を出力します\n");
printf("合計 = %.0f\n", sum);

printf("%d", N);
printf("個の整数の平均と標準偏差を出力します\n");

double ave = sum/N;
double sd = 0;

for(i = 0 ; i < N ; i++)
{
sd += (data[i]-ave)*(data[i]-ave);
}
sd /= N;
printf("平均 = %.2f\n標準偏差 = %.2f\n", ave, sqrt(sd));

return 0;
}

[otn]

scanfを消し忘れてます。見直しても気づいてないの？？？ぼんやりしすぎです。しっかり見ましょう。

あと、"エラー処理" の中身は書いてないの？？

[yatai]

根本的にまだ理解できていませんでした。
すみません。精進します。

[otn]

まるごとコピペすれば消し忘れミスなど発生しないのに。。。
ミスを防ぐ作業方法を採用しましょう。

[yatai]

心がけていきます。
ありがとうございます。