質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • C

    3692questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

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

受付中

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 254

matti0518

score 1

前提・実現したいこと

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
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+1

まず、

PLAYER p[player_num];


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

#define MAX_PLAYER 10


PLAYER p[MAX_PLAYER]; 


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

あと、

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

とりあえず目についたのは、これです。
コマンドライン引数がないときも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];

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

    /*ファイルの読み込み*/
    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++;
    }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C

    3692questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。