ファイルをfscanfで読み込んで構造体に入れて最後のデータを表示したい。

Question

前提・実現したいこと

キーボードから指定されたファイル名のデータを読み込ませよ．ファイル名は99文字までとする．またデータは，番号，氏名，数学の点数，英語の点数が空白文字で区切られたデータである．読み込んだ最後のデータの氏名(最大99文字)を表示するプログラムを作成せよ．もしもファイルが存在しない場合にはNGと表示せよ．
という問題あり、自分なりに書いてみたのですが理解が及んでいないところがありどうにも解決できません

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

出力されない、for分の条件の付け方がわからない
subscripted value is not an array, pointer, or vector
for(i=0;d[i]!=EOF;i++)
と表示される

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
typedef struct data{
  int no;
  char name[99];
  int e;
  int m;
} DATA; 

int main()
{
int i=0;
 FILE *fp;
  char filename[99];
  DATA d;
 scanf("%s",filename);
 fp=fopen("filename","r");
 if(fp==NULL)
{
printf("NG");
return -1;
}
for(i=0;d[i]!=EOF;i++)
{
fscanf(fp, "%d%s%d%d", &d.no, d.name, &d.e, &d.m );
}
printf("%c",d.name[i]);
}

試したこと

whlie分などやってみましたがそれ以前に理解が足りていないようでできませんでした

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

for分のd［i］の部分には読み出したファイルの最後まで繰り返すようにしたいのですがやり方がわかりません

DATA dはstructで作った構造体の変数だと思っているのですがちがうのかもしれません

初めてC言語でファイルを触るのでほとんど手探りの状態です

Comments

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか。
詳しく説明してください

Answer 1

回答用のコードは用意しましたが、その前に説明をご覧ください。

質問にあったエラーを要約しますと、(DATA型の)変数dが、ポインタ、配列、ベクトルのいずれでもないため、d[i]という表記はおかしい、ということです。例えば100人分のデータとしてDATA d[100];のように宣言すればi番目の要素、というものが考えられますが、今回は1人分しか宣言していません。

では、100人分宣言すればよいのか? 答えは違います。必要なのは最後の1人分だけですので、実は1人分で足ります。100人分宣言してもよいのですが、その場合は別の箇所を修正する必要が出てきます。

他の問題点を以下に列挙します。

1. 点数は、「数学、英語」の順のはずだが、質問者さんの回答では逆になっている。
2. 「ファイル名」は99文字以下だが、「絶対パス」が99文字以下とは書いていない。
3. ファイル名に空白が含まれないとは書いていない。
4. ファイル名、人名ともに99文字入力した場合にバッファオーバーフローを引き起こす。
5. for文での読み取り終了条件がおかしい。
6. 開いたファイルをfcloseを用いて閉じていない。

1-6をすべて一度に理解しなさい、というつもりはありませんが、英語のエラーメッセージを日本語に置き換える練習はしてください。ちなみに、Cコンパイラであれば通常、C2440のようにエラー番号を教えてくれるので、それをキーワードにして検索すれば、日本語で解説しているページが見つかります。

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

typedef struct data{
  int no;
  char name[100];
  int math;
  int eng;
} DATA; 

int main()
{
  FILE *fp;
  char path[200];
  DATA d;

  if(scanf("%199[^\n]%*[^\n]", path) < 1) path[0] = '\0';
  getchar();

  fp = fopen(path, "r");
  if(fp == NULL){
    printf("NG");
    return -1;
  }
  for(; fscanf(fp, "%d %s %d %d\n", &d.no, d.name, &d.math, &d.eng) == 4;);
  printf("%s", d.name);
  fclose(fp);
  return 0;
}

Comments

[Aru-0]

DATA dで宣言した構造体のi列目だと思っているのですが如何せん構造体もファイルも扱ったのが最近なので勘違いしてるかもしれないです

[yumetodo]

> DATA dで宣言した構造体のi列目だと思っているのですが

これは間違いですよね。まずはそこを自分で調べてみてください。私達が回答したのではおそらく身につきません。

[Aru-0]

散々調べているのですが見つけられないのか理解していないから正解が分からないのかで、どうしようも無くなったのでここに投稿しました
せめてこれがなんなのか教えて頂けるとまた調べるためのボキャブラリーが増え、自己理解のために繋がるかと思います なのでヒントだけでもいいので助けていただけませんか？

[yumetodo]

うーん、そもそもC言語の入門を間違えている気がしますね。
前橋和弥さんのC言語ポインタをおすすめしたいところです。

ヒントですが
int i;
iの型はintですよね？
int j[3];
jの型はint[3]型ですよね？
typedef struct { int foo } foo;
foo k;
kの型はfooですよね？

[Aru-0]

typedef struct {略}DATAで構造体の中身を作ってDATA　dでこれについての変数を宣言したのではないあのですか？

[yumetodo]

そこまではあってます。そこまでは。

[Aru-0]

for分の所の条件に入れるものが違うのが原因だと思っているのですがここには*fp！=EOD なのでしょうか
読み出したファイルを最後までループするようにしたいのでEODを使えばいいのかなとは思うんですがポインターなどに不慣れで特性がうろ覚えです

[majiponi]

仕事が終わって帰宅したので正式な回答を書きます。30分お待ちください。

[majiponi]

ごめんなさい、最後のループ条件、サンプルに誤りがありました。

[Aru-0]

すいませんまだ纏まっていないので後に質問してもよろしいですか？

[Aru-0]

まずif(scanf("%199[^\n]%*[^\n]", path) < 1) path[0] = '\0';についてお伺いしたいのですが"%199[^\n]%*[^\n]"の部分は書式指定子でファイル名の空白対策なのかなとは思うのですがなぜこのような書き方になるのかがわからないです。次にpath[0] = '\0';でpath0になぜ\0をいれるのか入れてなぜ問題ないのかを知りたいです。それとこの下にgetchar();が入る意味はなんでしょうか、次に最後のループ条件は==4で四つのデータが埋まるまで繰り返すものだと思っていたので、初めの一人分のデータを取得して終わってしまうのではないかと思いましたが、これでファイルの最後のデータまで取得されます。
ですがここで==4以外の数（==3など）を打ち込むと先頭の名前が表示されます、これはなぜなのでしょうか。

[Aru-0]

お時間ありましたら申し訳ありませんがお答えいただければとてもありがたいです。

[majiponi]

scanfの書式設定と、getcharについてはこちら
https://marycore.jp/prog/c-lang/scanf-string-safely/
を、forの条件指定は、「一人ぶんのデータをすべて読み取れている間はループして、dを上書きせよ」というものです。最後の一人を読んだ後、ファイルの終わりに到達すると、読み取れるデータ数が0になるのでそこでループが終わります。そして読み取りに失敗した場合は上書きされないので、最後の一人のデータがdに残っていますので、それを表示します。

[Aru-0]

本当に助かりました。ありがとうございます！！！