C言語で、ファイルを読み込んで数字と名前に分けて配列に格納に関する質問

C言語で、ファイルを読み込んで数字と名前に分けて配列に格納に関する質問です！
ファイルを開いた後でエラーとなるのですが、何が足りないのでしょうか？

ファイル内容
20
田中
10
鈴木

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main(int argc,char *argv[])
{
FILE *fp;
char str[256];
char *tp;
int k,i=0;
int num[10];
char na[10][20];

fp=fopen(argv[1],"r");
if(fp==NULL){
printf("ファイルを開けません\n");
return 1;
}else{
printf("開けた\n");
}

while(fgets(str,sizeof str,fp)!=NULL){

tp=strtok(str," ");
num[i]=atoi(tp);
tp=strtok(NULL," ");
strcpy(na[i],tp);
i++;
}
printf("%d\n%s\n",num[0],na[0]);
printf("%d\n%s\n",num[1],na[1]);

fclose(fp);
return 0;
}

No.3 ベストアンサー 回答者： noname#144013 回答日時： 2010/07/16 06:18

こんにちは。

ご提示の「入力データファイル」のフォーマット（形式）と、プログラム上の
データ読込み処理の想定フォーマットに、くい違いがあるようです。

入力データファイルのフォーマットでは、
　１行につき１データとし、２行で１組分の以下のデータ形式
　　　１行目：　数値　─┬─　１組目（１レコード目）
　　　２行目：　名前　─┘
　　　３行目：　数値　─┬─　２組目（２レコード目）
　　　４行目：　名前　─┘
　　　　　：　　　　：
となっているのに対し、プログラム上のデータ読込み処理では、
　　　１行目：　数値 [スペース] 名前　　・・・　１組目（１レコード目）
　　　２行目：　数値 [スペース] 名前　　・・・　２組目（２レコード目）
　　　　　：　　　　：　　　　　　　　　：
という形式を想定したものになってしまっているようです。

fgets関数では、改行文字を区切りとしてファイル上のデータ（文字列）を
指定されたバッファに読み込みます。
※その際、読込みバッファの終端に改行文字も含まれます。

ご提示のプログラムでは、ファイルから１行読込む度に、スペース区切りで
【数値】と【名前】の２データ分を読込むようになっていますが、実際の読み
込みバッファには、１個のデータ（数値or名前）しか格納されていないので、
２回目のstrtok関数では読込むデータがないため、戻り値 tp には NULL値
が返されているはずです。

その状態（コピー元ポインタ=NULL）で、strcpy関数により文字列コピーを
実行しようとしているため、例外エラーが発生しているものと思われます。

上記の問題を回避するためには、
　【１】ファイルデータフォーマットを以下の形式に変更する。
　　　１行目：　数値 [スペース] 名前　　・・・　１組目（１レコード目）
　　　２行目：　数値 [スペース] 名前　　・・・　２組目（２レコード目）
　　　　　：　　　　：　　　　　　　　　：
または、
　【２】ファイル読み込み処理を、データファイルの形式（１行＝１データ）に
　　　合わせて変更する。
　※この場合、２行毎に、１行目で【数値】の読み込み、２行目で【名前】の
　　読み込みを行う必要があります。
の何れかの処置を行う必要があると思います。

また、プログラム上のデータ格納用の配列の確保要素数は、１０個が上限
となっていますので、仮にデータファイル上のデータ数が１０個を超えていた
場合は、読み込み途中で配列の確保分をオーバーしてしまう可能性があり
ます。
↑この対策（１０個分読込んだらループを抜けるなど）も採っておく必要がある
と思います。

あと、細かい事ですが、【名前】格納用の文字列バッファのサイズは、２０文字
（終端コード'\0'も含めて）となっていますが、仮にデータファイル上の【名前】が
２０文字以上だった場合は、文字列コピー時にバッファの確保分を超えてコピー
される可能性があります。
↑この対策（コピー文字数を上限２０文字とするなど）も採っておく必要があると
思います。

参考までに（なるかどうか判りませんが）、以上を踏まえて、データファイルの
フォーマットが、前述の【１】のパターンだったものとして、ご提示のプログラム
を改造したソースを下記に掲載しました。（※下記リンク先参照）
※宜しければ試してみて下さい。

■改造版ソース
http://ideone.com/3xAR9

■改造版プログラムでの実行例
＜入力データファイル＞
20 田中
10 鈴木
30 佐藤
40 鈴木
50 伊藤
60 小林
70 藤田
80 渡部
90 桜井
100 安藤
99 松本

＜実行結果＞
開けた
レコード件数=10
No= 1: 20　　　 田中
No= 2: 10　　　 鈴木
No= 3: 30　　　 佐藤
No= 4: 40　　　 鈴木
No= 5: 50　　　 伊藤
No= 6: 60　　　 小林
No= 7: 70　　　 藤田
No= 8: 80　　　 渡部
No= 9: 90　　　 桜井
No=10: 100　　 安藤

以上です。

この回答へのお礼

詳細まで丁寧に説明していただきありがとうございます！
学びだしたばかりで、すぐには理解できないので助かりました！＾＾

お礼日時：2010/07/21 19:00

No.2 回答者： yaemon_2006 回答日時： 2010/07/16 03:55

　

　何故、strtok？
　

No.1 回答者： Tacosan 回答日時： 2010/07/16 00:50

「strtok の返り値をちょっと調べてみよう」とは, これっぽっちも思わなかったと?