C言語 テキストの場合分け

質問

C言語プログラミングの質問です。
x1, y1, r1, distance1
x2, y2, r2, distance2
・・・
と続くテキストを読み込み、distanceを10刻みで場合分けをするプログラムを作成しています。初心者で苦戦しているところですが、以下のサンプルの間違いをご指摘いただけませんでしょうか。
よろしくお願いいたします。
テキストの行数とdistanceの最大値は不明です。

ーーーーーーーー
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string.h>
#include <locale.h>

int main(int argc, char *argv[])
{

int a,b,c,distance;
int i=0;
int num[200];

FILE* inifile = fopen("file.txt", "r");

if(inifile==NULL)
{
printf("err__file.txt is nothing!!");
return 0;
}

FILE* outfile = fopen("outfile.txt", "w");

if(outfile==NULL)
{
printf("err__outfile.txt is nothing!!");
return 0;
}

while (fscanf(inifile,"%d\t %d\t %d\t %d\n", &a, &b, &c, &distance) != EOF)
fclose(inifile);

for(int i =1; i<101; i++)
{
if((i-1)*10 <= distance && distance < i*10)
{
num[i]+=1;
}
}

for(int i =1; i<101; i++)
{

fprintf(outfile, "%d\t", num[i]);

}

fclose(inifile);
fclose(outfile);
printf("Normal END\n");
return 0;
}

回答No.3

500cii · ベストアンサー

num配列の各要素num[i]の初期値は指定しなければ、最初に何が入っているかわかりません。

初期化の方法ですが、
int num[200] = {0};
で大丈夫です。
この記法は配列の各要素の値を列挙する方法ですが、記述されなかったものに関しては0で初期化されるルールがあるので、これで充分です。

お礼

ありがとうございます。
無事、解決してプログラムを完成させることができました。

回答No.2

500cii

>while (fscanf(inifile,"%d\t %d\t %d\t %d\n", &a, &b, &c, &distance) != EOF)
>fclose(inifile);

この行から意味不明なのですが、どういうことですか？
これでは、最初の一行を読んだ後に、ファイルをクローズするだけにしか見えないのですが。
この行より下にあるdistanseを含むところをwhileが含んでいないといけないのでは？

あと、numは初期化しないといけません

補足

すみません。記載しているサンプルのfclose(infile)の場所間違えていました。
初期化は、最初のタイミングで行うのでしょうか。それとも、iの範囲が変わるたびに初期化を行うということでしょうか。

回答No.1

Tacosan

とりあえず
・「テキスト」と fscanf のフォーマットがあってない
・「distanceの最大値は不明」に対応できていない

ただ, 「間違い」というからには何かあるの?

補足

説明不足で申し訳ありません。

テキストの読み込みはこちらの記述ミスで問題ありません。
distanceの最大値もだいたい40-80くらいなので、100確保しておけば問題ないかと思い、アバウトに100としてあります。

最終的に、
0-10にある数\t　10-20にある数\t　20-30にある数\t.....

というテキストを書き出したいのですが、場合分けするところが間違えているようで、分けられません（書き出されるファイルの中身が0やマイナスの値で）。
勉強不足で申し訳ないですが、コメントをいただけたら幸いです。