C言語でループから抜けれない

質問

下記の課題に取り組んでおり、ソートは一回抜かしてファイルの読み込みに取り組んでいる状態です。

電卓アプリケーションの作成
　以下の機能を満たすアプリケーションを作成してください。
　※画面の表示やログの形式は原則として例示されている内容に従ってください。

　1) 四則演算が出来ること
　2) コンソールから計算対象となる数値と演算子を受け取る
　　 例） 5 + 6
　3) 結果をコンソール上に表示する
　　 例） 11
　4) 演算結果と実行した日、時間、秒をログファイルとして保存する
　　 ログの上限は1000行とする（上限を超過するケースはひとまず考慮しなくて良い）
　　 例） log.txt に 以下の内容を記録
　　　　　2015/04/27 14:30:51, 5 + 6, 11
　5) コンソールからパラメータを受け取り、ログを実行時間の昇順/降順に並べ替えて
　　 コンソールに表示する
　　 例） ASC　を入力
　　　　　2015/04/26 10:20:00, 5 + 6, 11
　　　　　2015/04/27 14:30:51, 7 - 6, 1
　　　　　2015/04/27 15:30:00, 7 + 8, 15
　　　　　2015/04/28 14:30:51, 8 + 9, 17
　　 例） DESC　を入力
　　　　　2015/04/28 14:30:51, 8 + 9, 17
　　　　　2015/04/27 15:30:00, 7 + 8, 15
　　　　　2015/04/27 14:30:51, 7 - 6, 1
　　　　　2015/04/26 10:20:00, 5 + 6, 11

しかしVisual studioでのデバックで

for (i = 0;i < 1000;i=i+1) {
fgets(sin[1000], sizeof(sin[0]), fp);
if (strcmp(sin, "EOF") == 0) {
break;
}
この行でループが抜け出せなくなってしまいました。
どこを修正すればいいえしょうか？

#include <stdio.h>
#include <time.h>
#include <string.h>
#include <stdlib.h>
/*
static char ad[10];

int cmp_u(const void* a, const void* d) {
return strcmp((char*)a, (char*)d);
}

int cmp_d(const void* a, const void* d) {
return strcmp((char*)d, (char*)a);
}
*/
int main() {
int num1, num2;
char op;
float answer;
int r,i,n;
FILE* fp;
char c[11];
char sin[1000][1000];
char ad[8];

fp = fopen("log.txt", "a");

if (fp == NULL) {
printf("ファイルオープン失敗\n");
return -1;
}

回答No.1

asciiz · ベストアンサー

＞for (i = 0;i < 1000;i=i+1) {
＞ fgets(sin[1000], sizeof(sin[0]), fp);
＞ if (strcmp(sin, "EOF") == 0) {
＞ break;
＞ }
＞この行でループが抜け出せなくなってしまいました。

もしこの記述でループを抜け出したいなら、ファイルの末尾に文字で"EOF"と書いておくか、手入力なら入力終了の合図に「EOF」と打ち込む、ということが必要です。
でもそんなことは課題の中に書いてないし、あなたも考えていませんよね。

ご質問の記述だと、ファイルのどこにも「EOF」と書いておらず、ファイル末尾に達したなら、それ以上読み込める者は何もないので、入力を読めない→でも"EOF"文字列は出てこない→入力を読めない…(以下略)と、無限ループに陥ってしまうわけです。

なので、入力に「『EOF』と言う文字列があること」を期待するのではなく、「ファイルを最後まで読んだ(本当にこれ以上読めない)」ということを検出するようにしましょう。

それを調べるのは、feof()関数。

＞feof | Programming Place Plus　Ｃ言語編　標準ライブラリのリファレンス
＞https://programming-place.net/ppp/contents/c/appendix/reference/feof.html

なので、上記ページのサンプルにもある

if( feof(fp) ){
break;
}

とすれば抜けられるようになるでしょう。

でも、その前の行
＞ fgets(sin[1000], sizeof(sin[0]), fp);
ここは
＞ fgets(sin[i], sizeof(sin[0]), fp);
こうすべきですよね? 1,000行までのファイル内容を、一行ずつ順にsin[0] sin[1] sin[2] …へと取り込みたいわけです。

で、一行ずつsin[i]に取り込んでいき、ファイルをすべて読み切った時、sin[i] にはヌルストリングが入ります。
ヌルストリングは、長さ0の文字列。
長さ0とは、1文字目のデータが「0」である、ということ。
ですので、そこを判定基準に使うことも可能です。

if (sin[i][0] == 0) {
break;
}

こちらの記述でも、抜けられるでしょう。

お礼

分かりやすい説明、ありがとうございます。