構造体を利用したデータの読み込みと表示

Question

###前提・実現したいこと
Cで構造体を利用してbaseball_player.defというファイルを読み込み、各データを逆順に表示するプログラムなのですが、コアダンプになってしまいます。読み込んだデータを格納するポインタ配列data[MAX_LEN]を二次元配列data[player_num][MAX_LEN]にしてみたりもしましたがうまくいきませんでした。
どこが間違っているのでしょうか？できれば詳しい解説等もつけてくださればありがたいです。
変更するのはmain関数内のみで、関数等は変更しないようにお願いします。

###ソースコード

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

#define MAX_LEN 100

/*PLAYER構造体の定義*/
typedef struct string
{
	char name[10]; /*名前*/
	int number;  /*背番号*/
	int at_bat;  /*打数*/
	int hit;     /*安打数*/
	double ave;　/*打率*/
}PLAYER;



/*makePlayerData関数の定義*/
PLAYER makePlayerData(char *data)
{
	PLAYER a;

	/*文字列より各種データの読み込み*/
	sscanf(data, "%s %d %d %d", a.name, &a.number, &a.at_bat, &a.hit);
	/*打率の計算*/
	a.ave = (double)a.hit / (double)a.at_bat;

	return a;
}



/*printPlayerData関数の定義*/
void printPlayerData(PLAYER a){
	printf("%s     %d   %d   %d   %f\n", a.name, a.number, a.at_bat, a.hit, a.ave);
}



int main(int argc, char *argv[])
{
	char buff[MAX_LEN];
	char *filename;
	FILE *fp;
	int i;
	int player_num;
	char *data[MAX_LEN];
	PLAYER p[player_num];


	if(argv[1] != NULL) filename = argv[1];

	/*コマンドラインパラメータの数が間違っている場合の処理*/
	if(argc != 2){
		printf("Usage: ./print_player_data.c [filename]\n");
		exit(1);
	}

	/*ファイルを開く*/
	fp = fopen(filename, "r");

	/*ファイルが存在しない場合の処理*/
	if(fp == NULL) {
		printf("File %s doesn't exist．\n", filename);
		exit(1);
	}

	/*ファイルの読み込み*/
	i = 0;
	while(fgets(buff, sizeof(buff) , fp) != NULL){
		sscanf(buff, "%s", data[i]);
		p[i] = makePlayerData(data[i]);
		i++;
	}

	/*ファイルを閉じる*/
	fclose(fp);

	/*データの表示*/
	player_num = i;
	for(i = player_num - 1; i >= 0; i--){
		printPlayerData(p[i]);
	}

	return 0;
}

#baseball_player.defの内容

Tanaka 3 483 125
Yamada 9 509 143
Takahashi 24 503 152
Abe 24 498 138
Suzuki 31 518 149

Answer 1

まず、

PLAYER p[player_num];

player_numに値を設定してない状態で、これはマズイですね。
人数の最大値が決まっているのであれば、例えば

#define MAX_PLAYER 10

で

PLAYER p[MAX_PLAYER]; 

とでもやっておけば良いのですが。

あと、

sscanf(buff, "%s", data[i]);

これも、data[i]はポインタだけど値を設定してませんね。
さらに、空白の手前までしか入力できない。

Answer 2

    /*ファイルの読み込み*/
    i = 0;
    while(fgets(buff, sizeof(buff) , fp) != NULL){
        sscanf(buff, "%s", data[i]);
/* 
 * ここで一旦 data[i] をプリントしてみよう。
 * data[i] にはファイルから読んだ一行分("Tanaka 3 483 125"とか)
 * が納められてなきゃ makePlayerData() に与えられんよね。
 * ...ちょっとﾏﾃ。data[i]には一行分の文字列を納めるのに十分な領域が確保されているかい?
 */
        p[i] = makePlayerData(data[i]);
        i++;
    }

Answer 3

とりあえず目についたのは、これです。
コマンドライン引数がないときもargv[1]が評価されてしまいます。

if(argv[1] != NULL) filename = argv[1];

次のように処理の順序を変えると良いでしょう。

if(argc < 2){
    printf("Usage: ./print_player_data.c [filename]\n");
    exit(1);
}

filename = argv[1];

---

そもそも適切に実行しているかも不明です。
コマンドライン引数をどのように与えているかも書くべきでしょう。