ファイルの読み込みでエラー？

質問

以前アドレス帳を自分で作成したのですが、その仕様は個人情報を１次元配列に読み込み、操作するものでした。
これではいろいろと効率が悪いように思えるので、情報をヒープ（２分木）に格納するよう改良を加えましたが、いきなり躓いてしまいました。

ファイルの形式は改良中なので簡単な内容にしています

総データ数
名前
住所
名前
住所

のように連続して書き込まれています。

これを読み込み、かつ同時に２分木に格納するよう組んだのですが、２人目の情報からうまく読み取れません。　いかに大まかなソースを載せます。

fp = fopen("MembersFile.txt", "r");
　fscanf(fp,　"%d\n",　&Member);

　if((fgets(src->Name,　NameLength, fp))　!=　NULL)
　　　　my_strtrim(src->Name);
　else
　　　　return -1;//エラー
　if((fgets(src->Address,　AddressLength,　fp))　!=　NULL)
　　　　my_strtrim(src->Address);
　else
　　　　return -1;//エラー

　Parent = src;

　while(num < Member - 1){
　　　　　if((Child = Make_Person()) == 0){
　　　　　　　　　return -1;
　　　　　}

　　　　　if((fgets(Child->Name, NameLength, fp)) != NULL)
　　　　　　　　　my_strtrim(Child->Name);
　　　　　else{
　　　　　　　　　return -1;
　　　　　}
　　　　　if((fgets(Child->Address, AddressLength, fp)) != NULL)
　　　　　　　　　my_strtrim(Child->Address);
　　　　　else{
　　　　　　　　　return -1;
　　　　　}

大雑把に記述してしまいましたが、途中途中自作関数があるので説明します。
my_strtrim関数は改行コードを終端文字に変換する関数です。
void my_strtrim(char *buff){
while(*buff != '\n')
buff++;

*buff = '\0';
return;
}

Make_Person関数は一人の情報のメモリ確保を行っています。

一応いろいろと確認してみたのですが、真ん中にあるwhile文以降読み込むことができません。

ひょっとしてファイルポインタは宣言されたスコープの外では使用することができないのでしょうか？

回答No.1

tekebon · ベストアンサー

まず、ここが気になります。
＞fscanf(fp,　"%d\n",　&Member);
scanf関数、fscanf関数では"%変換文字"を指定します。\nは不要です。
これによりMemberにデータ数が正しく読み取れていない可能性があります。
fscanf(fp,　"%d",　&Member);
としてみてください。

そして、while文の条件に使われているnumは0で初期化してありますか？
numがローカル変数だった場合には、0で初期化しないと値が不定ですので
やたら大きな数になっていると、条件が最初から成り立たなくて繰り返しに入らない可能性もあります。

質問からはこの2点があやしいです。確認してみてください。

また、Make_Person()関数は確保したメモリのアドレスが返却値になっていると
思われるので、if文の条件は==0ではなく==NULLにしたほうがいいです。

ファイルポインタはfcloseするまで有効ですから、値が正しくやり取りできていれば問題ないはずです。

お礼

回答ありがとうございました。
fscanf(fp,　"%d",　&Member);
このようにした場合、バッファに改行文字が残ってしまい、この後の動作がうまく動かなかったので、あえて\nをつけて記述していましたが、どうやらこれも正常に動かなかった原因のひとつだったらしいです。
的確に処理するためこの文字もfgetsにして読み込んだところ正常に動作しました。よかったぁ・・・・

色々なアドバイスをもらいましたが、上の疑問以外は全て正常に行っていますが、０をNULLにしたほうが良いというアドバイスはごもっともです。

ありがとうございました！