末尾の行が二回読み込まれてしまいます。

質問

末尾の行が二回読み込まれてしまいます。
以下のコードで
#include <stdio.h>
#include <stdlib.h>

//ファイルに書き込まれた24:00:00の形式の時刻を秒単位で出力する。

int translate(char* str)
{

int a,b,c;
int time;
sscanf(str,"%d:%d:%d",&a,&b,&c);

time = 60*60*a + 60*b + c;

return time;
}

int main(int argc,char *argv[])
{
FILE *fp;
char str[100];
int s;

fp = fopen("file.time","r");

if(fp == NULL){
printf("can not open the file.\n");
return 1;
}

while(!feof(fp)){

fscanf(fp,"%s",str);
s = translate(str);
printf("%d\n",s);

}

fclose(fp);

return 0;
}
末尾の行が二回処理されてしまいます。なにがまずいのでしょうか？よろしくお願いいたします。

回答No.3

bigcat1969 · ベストアンサー

...
fscanf(fp,"%s",str);

while(!feof(fp)){
s = translate(str);
printf("%d\n",s);

fscanf(fp,"%s",str);
}
...

お礼

回答ありがとうございます。きちんと直りました。

回答No.5

Tacosan

ちと規格を確認してみましたが, 「scanf系で読み込みに失敗した場合, 読み込めなかった変数の内容は変化しない」と読めます＞#2.
「feof をやめて fscanf の返り値をチェックする」のが正解だと思う.

お礼

回答ありがとうございました。知らないことがおおく勉強になりました。

回答No.4

SaKaKashi

http://www.kiso.tsukuba.ac.jp/~makimura/text/node128.html

お礼

回答ありがとうございます。いろいろ勉強になります。

回答No.2

magicalpass

最終行を実行した段階ではまだfpがファイルエンドに達していません。そこでfeof(fp)の結果が偽なのでもう１回ループが繰り返されます。
　次にfscanfを実行したところfpがファイルエンドに達したので、strの内容は変更されずに帰ってきてるようです。strの内容が書きかえられていないので中身は前回の内容と同じまま関数translateが実行されているのです。
　fscanfが失敗した時の格納先の内容はとくに動作が定義されていないようなので処理系依存かと思います。何も入っていないことを期待するなら呼び出しの前に空にしておくことをおすすめします。あるいはちゃんとfscanfの返り値を判断して処理を書くかですね。

お礼

回答ありがとうございます。勉強になりました。

回答No.1

SaKaKashi

while (feof(fp)==0) {
fscanf(fp,"%s",str);
s = translate(str);
printf("%d\n",s);
}
ではいかが？while条件を変えてみた。

お礼

回答ありがとうございます。残念ながらだめでした。