C言語のファイル操作がよくわからないです

Question

前提・実現したいこと

授業のプログラミングの課題でファイル操作をするところがあるのですが、なぜ正常に動かないのかがよくわかりません。
課題の内容は、バブルソートを用いて成績管理システムを作るといったものです。バブルソートに関してはある程度理解できたかと思うのですが、ファイル操作に関しては授業のメインでないこともありあまり説明がなく、調べてみてもよくわからなかったので、どう使えばいいのか十分に理解できていない現状です。
ファイルを読み込んで、その内容を構造体に格納したいのですが、実行すると名前がNULLで、それぞれの点数も0点と表示されてしまっているのですが、原因がわからず困っています。ご教授頂けますと幸いです。

該当のソースコード

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

#define seisekiMAX 256

typedef struct{
    char name[seisekiMAX];  // 氏名
    int kokugo;             // 国語の評点
    int sansu;              // 算数の評点
    int rika;               // 理科の評点
}Seiseki;


int seisekiCount = 0;   // ファイルの行数(＝データの数)


void putSeisekiArray(Seiseki*);     // ファイルからデータを読み込み配列へ格納する
void view(Seiseki*);    // 構造体配列に格納されている各データを表示する
void sort(Seiseki*);    // 国語を基準に降順でバブルソート


int main(){

    // 構造体配列の宣言
    // seisekiMAXはメイン関数の外で#define で256に設定
    // Seiseki 型もメイン関数の外で定義しているものとする
    Seiseki seiseki[seisekiMAX];

    // putSeisekiArray の中でファイルからデータを読み込む
    putSeisekiArray(seiseki);

    // ソート前のデータを表示
    printf("**** origin ****\n");
    view(seiseki);

    printf("\n");

    // バブルソートを行う
    sort(seiseki);

    // ソート後のデータを表示
    printf("**** sorted ****\n");
    view(seiseki);

    return 0;
}

// ファイルからデータを読み込み配列へ格納する
void putSeisekiArray(Seiseki* seiseki){
    FILE *fp;   // FILE型構造体
    char buf[256];  // ファイルの一行分を格納


    // ファイルを開く。失敗するとNULLを返す
    fp = fopen("data.txt", "r");
    if(fp == NULL){
        // ファイルオープンに失敗した場合の処理(異常終了)
        printf("ファイルのオープンに失敗しました\n");
        exit(1);
    }

    // ファイルの内容を全行読み込む
    while (fgets(buf, sizeof(buf), fp) != NULL){
        // 読み込んだデータを格納
        sscanf( buf, "%s %d %d %d", (seiseki+seisekiCount)->name, &(seiseki+seisekiCount)->kokugo, &(seiseki+seisekiCount)->sansu, &(seiseki+seisekiCount)->rika );
        seisekiCount++;
    }

    // ファイルをクローズする
    fclose(fp);
}

// 構造体配列に格納されている各データを表示する
void view(Seiseki *seiseki){
    for(int i = 0; i < seisekiCount; i++){
        printf( "%s\n　国語：%d点　算数：%d点　理科：%d点\n", (seiseki+seisekiCount)->name,(seiseki+seisekiCount)->kokugo, (seiseki+seisekiCount)->sansu, (seiseki+seisekiCount)->rika );
    }
}

// 国語を基準に降順でバブルソート
void sort(Seiseki *seiseki){
    Seiseki tmp;

    for(int i = 0; i < seisekiCount-2; i++){
        for(int j = 0; j < seisekiCount-2-i; j++){
            if((seiseki+j)->kokugo > (seiseki+j+1)->kokugo){
                tmp = *(seiseki + j);
                *(seiseki + j) = *(seiseki + j + 1);
                *(seiseki + j + 1) = tmp;
            }
        }
    }
}   

data.txt（読み込みたいファイル）

Ichiro 92 95 98
Jiro 70 75 80
Saburo 75 75 75
Shiro 95 90 85
Goro 83 80 77
Rokuro 85 75 65
Shichiro 50 70 90
Hachiro 60 40 20

Comments

[jimbe]

※ご質問本題とは関係ありません
> (seiseki+seisekiCount)->name, ～
ポインタは配列として使えますので seiseki[seisekiCount].name と書いたほうが分かり易いように思います。(人それぞれでしょうけれど。)

Answer 1

ファイルの読み込みでは無く、表示のほうが間違っています。
表示のループの変数は i です。

「原因がわからず困って」いるだけでは何も変わりません。
誰かに聞くより先にまず何とか原因を探らなくてはなりません。この作業もプログラミングの一部なのですから。
簡単な所では、あちこちに printf を入れて変数の値を確認することです。
そもそもその表示がされなければそこを実行していないことになりますし、変数の値が想像と違っていれば、そこまでの間の何かで想像通りの動作をしていないことになります。
そのように情報を集めていって原因となる個所を見つけ、修正する・・・これを繰り返すことでプログラムが思ったように動作するようになります。
それでもいよいよ原因が絞れないとなったら個人の限界かもしれませんので、** コードに、それまで試したことを添えて** どこかにご質問されると良いでしょう。そこまでの調査で自身のプログラムがどこまでどう動いているかははっきり理解しているはずですので、提供する情報は具体的となり回答も付き易く、回答が付けばその理解も早くなると思います。

Comments

[dodox86]

> println を入れて

printfの誤記ではないでしょうか。(Javaとの混同かと思われますが、初心の方だと悩んでしまうかもしれませんので、小さいことながら指摘させていただきました)

[nachu_]

ファイル操作に自信がなかったので、そこが問題だと思い込んでいました。どこが間違っているのか見当もつかなかったため質問しましたが、不足があったようで申し訳ないです。アドバイスありがとうございます。

[jimbe]

>printfの誤記

あわわ、やってしまいました。ありがとうございます。
小さいことどころか一番大切な箇所でしたのに間違えるとは…。

>どこが間違っているのか見当もつかなかった

作り方を教えるところは多数あっても、動作がおかしい時の調べ方・直し方はなかなかきっちりはやらないでしょうね。組み合わせが無数になりますから。
その為動作すれば先に進めますが、動かなければ止まってしまうのは理解できます。
ただ、逆に考えれば、ここでこのようなバグを経験したことで、今後作られるであろうプログラムがおかしな動作が見られた時に、ファイルの読み込み(等の処理部分)だけで無く表示のほうも疑わなければならないことを知ったのは収穫ではないでしょうか。
自信がないと言われるファイル操作は問題ありませんでしたから、自信の"足し"にしてもよいでしょうし。(問題が無かったので余計に見当がつかなくなったとも言えますが(^_^;))