ファイル出力結果がダブってしまう

Question

前提・実現したいこと

既存のファイルからデータを読み取って、合計点と合否を出し、その結果を別のファイルに出力したいです。

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

既存のファイル（kison.data）に入っているデータから合計・合否を追加したものを別のファイル（betu.data）に出力しようとしたのですが、うまくいきません。
$ cat kison.data
1 22 23
2 55 78
3 89 90

$ cat betu.data
1 22 23 45 不合格
2 55 78 133 合格
3 89 90 179 合格
3 89 90 179 合格

のように最後がおかしくなってしまいます。。。

該当のソースコード

include <stdio.h>

int main(void){

FILE *fp_r,*fp_w;
int eng,math,num,sum;

if((fp_r=fopen("kison.data","r"))==NULL){
printf("ファイルを開けません\n");
}
else if((fp_w=fopen("betu.data","w"))==NULL){
}else{
while( !feof(fp_r)){
fscanf(fp_r,"%d%d%d",&num,&eng,&math);
sum=eng+math;
if(sum>=120){
fprintf(fp_w,"%d %d %d %d 合格 \n",num,eng,math,sum);
}else{
fprintf(fp_w,"%d %d %d %d 不合格 \n",num,eng,math,sum);
}
}
fclose(fp_w);
fclose(fp_r);
}

return 0;
}

試したこと

1つ目のif文でNULLが出た時にプログラムが終了していないことに気づいて、betu.dataを開くif文をelse if文に書き換えたとたんにこの現象が現れました。
なので、タップミスか何かでループ本体を触ってしまったのかと思い、書き換える前に戻してみたりもしましたが、一向に直りませんでした。
自分でもどうしてこうなるのか本当にわからなくて、行き詰っています。

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

ここにより詳細な情報を記載してください。

Answer 1

feof の使い方を間違っているからです。

feof はこれより前の入力関数が、
ファイルの終わりに達してエラーになったことを示すものです。

feof(fp_r) が 0、すなわち !feof(fp_r) が 1だからといって
次の fscanf がエラーにならないことを保証するものではありません。

fscanf で 3行目のデータまで読めても、まだ feof は 0 のままです。
4回目の fscanf は失敗するのですが、それはチェックしていないので
3回目のデータをそのまま出力しています。

修正方法は

		while (fscanf(fp_r, "%d%d%d", &num, &eng, &math) == 3) {

または、

		while (1) {
			fscanf(fp_r, "%d%d%d", &num, &eng, &math);
			if (feof(fp_r)) break;

feof の間違った使い方の例を挙げます。

	int c;
	while (!feof(fp)) {
		c = fgetc(fp);
		putchar(c);
	}

1文字余計に出力されてしまいます。
たいていの場合最後の文字は改行文字なので気が付かないことが多い。

	char buf[1024];
	while (!feof(fp)) {
		fgets(buf, sizeof buf, fp);
		fputs(buf, stdout);
	}

最後の行が 2つ出てしまいます。

Comments

[mn6262]

回答ありがとうございました。
C言語を習いはじめたばかりで、勉強不足でした。。
いただいたアドバイスを参考にしながら精進していきたいと思います。

Answer 2

1 22 23
2 55 78
3 89 90

の後改行を入れているんじゃないですか？
改行後の行を読み込んでも EOF にはならないので、
fscanf(fp_r,"%d%d%d",&num,&eng,&math);
の返却値が 0 になっているはずです。

きちんと scanf の返却値をチェックして、
それの対処できるように書いてないから、
そんなことになるんですよ。

まあ、
1 22 23
2 55 78
3 89 90[EOF]
みたいにしてもいいとは思いますけど。

Comments

[mn6262]

回答ありがとうございました。

Answer 3

再現しません。
こんな出力が得られました。

1 22 23 45 不合格 
2 55 78 133 合格 
3 89 90 179 合格 
3 89 90 179 合格 

Comments

[mn6262]

すみません、合否を書き忘れていました。。。
実際は私もそのように合否が出ます。
最後の３番が二回続いてしまうのはなぜなのか、よろしければご教授いただければ幸いです。

[episteme]

ソッチか、ごめん。
他の回答にもあるとおり、while ( ファイルの終わりじゃない ) { 読んでなんかする } じゃダメなんです。
while ( 読むことができた ) { なんかする } じゃないと。