ファイル読み込みが上手くできません

質問

C言語初心者です。

現在、ファイルの情報を構造体に読込んで実行するプログラムを作成しております。
読込むファイルは700MBほど(行数は39900000行)となっており、1行ずつfscanfで読込んでいます。
400000行ごとに構造体に読込んで、処理を実行し、構造体に読込んだ全ての処理が完了したら、再度400000行読込んで・・・を繰り返すものになっております。

400000行ごとにしているためforループで100回まわすようにしているのですが、最後の1回(100回目のループ)で、99回目と同じ場所を読んできてしまいます。
つまり、最後の1回だけはファイルポインタが進んでいない状況になります。

コードは下記になります。
#define MAX_SIZE 400000
uint64_t i = 0;
uint64_t fc = 0;

int main(int argc, char *argv[]){
char w[] = "$write";
char r[] = "$read";

// ファイルを構造体へ格納
char filename[] = "TEST_READ_WRITE.txt";

struct TEST_DATA{
char cmd[10];
int addr;
int bsize;
};

FILE *fp;

if ((fp = fopen(filename, "r")) == NULL){
printf("%s open error !\n", filename);
exit(1);
}

for (int lp = 0; lp < 100; lp++){
struct TEST_DATA TD[MAX_SIZE]; // 構造体配列の宣言

for (fc = 0; fc < MAX_SIZE; fc++) {
if (feof(fp)){
break;
}
else{
fscanf(fp, "%s %d %d\n", TD[fc].cmd, &TD[fc].addr, &TD[fc].bsize);
}
}

while (i < MAX_SIZE - 1){
//ファイルから取得したデータによって処理を実行
if (strcmp(TD[i].cmd, w) == 0){
//書込み処理
}
if (strcmp(TD[i].cmd, r) == 0){
//読み込み処理
}
}
printf("Finish!! \n");
}
fclose(fp);

return 0;
}

読込むファイルはテキストデータで、
＄write 25651496 152
＄write 135878112 8
＄read 1244848 16
のような感じのものが39900000行並んでいるものになります(数字はランダムです)。

最後の1回のみ上手くファイルポインタが進まない原因が分からずに困っています。
お気づきの方がいらっしゃいましたら、アドバイスをよろしくお願いします。

回答No.2

m-take0220 · ベストアンサー

iの初期化とカウントアップは正しく処理されているという前提で。

ファイルポインタが進まないのではなくて、ファイル終端に達した場合の処理が正しくないのでは?

if (feof(fp)){
break;
}

でfcをループカウンタとしたforループから抜けていますが、iを使ったwhileループでは、どこまでデータが読み込まれたか確認せずにMAX_SIZE-1までループしてますよね。
たとえばfc=0の時にfeof(fp)がtrueになったとして、そのあとのwhileループはMAX_SIZE-1回処理されてしまうので、一回前の処理で読み込まれたTDに対して処理を行い、結果ファイルポインタが進んでないように見えるということになりませんか?
fcの上限がMAX_SIZEで、iの上限がMAX_SIZE-1というのもおかしいと思います。

お礼

m-take0220さん

返信が遅くなり、申し訳ございません。
ご回答、ありがとうございました。

前の回答でmaiko0318さんもおっしゃっていたようにカウント処理が間違っていました。
最後のループだけ実行されていないように見えていましたが、実際は構造体に入れている値の
MAX値が400000としていたため、ファイルの最大行数を越えた部分だけが前のデータで
使用されているようでした。

カウンタの値を変更したところ、目的の動作を行っています。
アドバイス、ありがとうございました。

回答No.1

maiko0318

while (i < MAX_SIZE - 1){

読んだ件数をカウントして処理しないと、
構造体に残っている前のデータが処理されてしまっている。

fcを使いましょう。

補足

maiko0318さん

ご回答ありがとうございます。
今更ですが、貼り付けたコードにミスがありました。
while ループを抜けたときにi=0をしているのですが、記載したコードでは抜けていました。
申し訳ございません。

読んだ件数をカウントして処理するというのは、MAX＿SIZEの代わりにfcを使用するということでしょうか？
現状、構造体の中身はwhileループで処理をした後のforループ内で上書きしていると思っていました。
実際に最後の1回以外は予定通り構造体の中身が更新されているように見えていたのですが、fcを使わないと上手く処理できないようですので、修正してみます。