C言語 ファイルデータを構造体に入れる

Question

前提・実現したいこと

ファイルデータを構造体に入れたいです。
ファイル内容は以下
Yamada Hanako 50
Suzuki Takuya 40
：
：
計50人分の点数データがあります。

発生している問題・エラーメッセージ

実行するとなぜか点数が6418156や6418240など、どこから来たのか不明な数字が出てきます。名前は普通に表示されます。

該当のソースコード

#include <stdlib.h>

 typedef struct
  {
  char FamilyName[32];
  char FirstName[32];
  int   Score[5];
  } Student;

int main(void)
{
    int i, m;
    FILE*fp;
    Student data[50];  // 構造体配列の宣言

    fp=fopen("tensu.txt","r");
  if(fp == NULL){
    perror("ファイルの読み込みに失敗/n");
  }
    for( i = 0; i <= 50; i++ ) {
        if( fscanf( fp, "%s%s%d\n" , data[i].FamilyName, data[i].FirstName, data[i].Score ) != 3 ) break;
    }
    for(m=0;m<50;m++){
    printf("%s %s %2d\n", data[m].FamilyName, data[m].FirstName, data[m].Score );
    }
fclose( fp );
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

int Score[5];

と何故か配列にしているのが原因です。

printf("%s %s %2d\n", data[m].FamilyName, data[m].FirstName, data[m].Score );

のdata[m].Scoreは整数値じゃなくて変数のアドレスです。
コンパイル時に警告は出ませんでしたか？

Comments

[ishiko]

警告出ませんでした、、、
アドレスではなく点数を表示させたいときはint Score[5]を消さなくてはいけませんか？
typedef struct
{
char FamilyName[32];
char FirstName[32];
int Score[5];
} Student;
ここは固定なので、ここ以外を直したいです。

[otn]

何故配列なのでしょうか？
点数は1つじゃなくて最大5つある？

[ishiko]

点数は１つだと思います。

[otn]

> 点数は１つだと思います。
ならば、点数は、Score[0]、Score[1]、Score[2]、Score[3]、Score[4]のうち、どこに入れるのでしょう？全部？？

[ishiko]

５種類の点数を扱うとき、int Score[5]としますか？
質問を質問で返してしまってすみません。

[otn]

点数が1つなら、普通は int Score; （int Score[1]; でも間違いではないが）、
点数が5つなら、int Score[5]; ですね。

[ishiko]

では、点数が５つあるということにしたらどうなりますでしょうか？
試しにint Score[1]と直して実行してみたのですが、直りませんでした。

[otn]

fscanf( fp, "%s%s%d %d %d %d %d\n" , data[i].FamilyName, data[i].FirstName, &data[i].Score[0], &data[i].Score[1], &data[i].Score[2], &data[i].Score[3], &data[i].Score[4] )
と、
printf("%s %s %2d %2d %2d %2d %2d\n", data[m].FamilyName, data[m].FirstName, data[m].Score[0], data[m].Score[1], data[m].Score[2], data[m].Score[3], data[m].Score[4] );

[ishiko]

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

typedef struct
{
char FamilyName[32];
char FirstName[32];
int Score[1];
} Student;

int main(void)
{
int i, m;
FILE*fp;
Student data[50]; // 構造体配列の宣言

fp=fopen("tensu.txt","r");
if(fp == NULL){
perror("ファイルの読み込みに失敗/n");
}
for( i = 0; i <= 50; i++ ) {
if( fscanf( fp, "%s%s%d\n" , data[i].FamilyName, data[i].FirstName, &data[i].Score[0] ) != 3 ) break;
}
for(m=0;m<50;m++){
printf("%s %s %d \n", data[m].FamilyName, data[m].FirstName, &data[m].Score[0] );
}
fclose( fp );
}
というふうに直してみたのですが、変わりません、、

[otn]

直し方が間違っているからですね。コピペが正しく出来ないレベルだと、厳しい。

あと、i 件しか読んでいないときにも50件出力しているので、i+1 件以降はゴミが表示されますね。

[ishiko]

直し方が違うとはどこの部分が違いましたか？

50人分だからi<=50だと思ったのですが、正しくないのですか？

[ishiko]

コピペして点数５つバージョンで実行してみました。
今度は先頭のYamada Hanako 50 30 40 20 40　のみ正しく表示され、あとはなぞの文字やスペース数字などが乱立しました。i+1件以降はごみとはこれのことですかね？
ちゃんと全部きれいに表示されるにはどうすればよいですか？

[javahack]

答えを求めてばかりではなく、一旦落ち着いて最初の設計段階から見直してみてはいかがでしょうか。

[otn]

> 直し方が違うとはどこの部分が違いましたか？

&の有無に注意が向かないと言うことは、注意力が弱いです。

> ちゃんと全部きれいに表示されるにはどうすればよいですか？

50件でなく、i 件だけ表示すれば良いです。<50 でなく <i 。

もしくは、「ちょうど50件」のデータを用意するか。49件以下だとお書きの状況になり、51件以上あるとメモリ上データを破壊するので駄目ですね。

[ishiko]

!=8に直してみたらうまくいきました！
たくさん付き合っていただきありがとうございました。
もう少し自分なりに調べてみますね。