while文を用いて指定された文字数が入力されるまで読み込みを続ける

Question

実現したいこと

while文を用いて指定された文字数（12文字）が入力されるまで読み込みを続けること。

前提

C言語で以下のような課題のプログラムを作っています。

バイナリファイルには，先頭から，生徒0のデータ→生徒」のデータ → 生徒2のデータ→・・・のように複数の生徒（30人以内）のデータが格納されており，各生徒のデータは次のような構成になっている。
姓（char型12個分）
科目1の点数（int型1個分）
科目2の点数（int型1個分）科目3の点数（int型1個分）

文字型2次元配列name（要素数30x12），整数型2次元配列sub（要素数30
x3)，double型配列 ave（要素数3）を宣言する、そして，このファイルから，i番目の生徒の姓，科目1の点数，科目2の点数，科目3の点数をname［i］,sub［i］［0], sub［i］［1］, sub［i]［2]に格納する。その後，生徒の絵数，科目1,科目2,科目3の平均点を求めて ave[0］, ave[1], ave[2]に格納し，画面出力（表示）するプログラムを作成せよ。
※データ個数が与えられていないのでfor文ではなくwhile文を用いる..
※ fopen ではファイル名は“./../bin-score.dat"と設定する。

そして、自分なりにプログラムを作りました。

while文のところでおそらく間違っているのですが、「nameに対して文字が12個分読めている間繰り返す」という指示と、「ファイルの内容の読み出しデータの個数の情報は与えられていないので、ファイルの末尾まで読み出しを繰り返す」という指示があります。

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

Kawai 84 62 74
Abe 62 58 68
Kubota 88 95 54
Yamada 77 39 100
Suzuki 82 55 66
Ogawa 48 100 61
Asai 88 92 96
Yoshida 94 92 96
Number of students : 8
Average : 77.875000 74.125000 76.875000

C:\Lecture\Prog2\Prog2_3_hw5\x64\Debug\Prog2_3_hw5.exe (プロセス 2364) は、コード 0 で終了しました。
このウィンドウを閉じるには、任意のキーを押してください...

という風に正しいように表示されるのですが、正答例を見てみると"Kawai"の上に"Kimura"というデータがあるそうなんです。

該当のソースコード

#include <stdio.h>

int main(void)
{
	FILE* fp;
	char name[30][12];
	int sub[30][3];
	double ave[3] = { 0.0,0.0,0.0 };
	int i;
	fp = fopen("../../bin-score.dat", "rb");
	if (fp == NULL)
	{
		printf("File open error.\n");
		return 1;
	}
	i = 0;
	while (fread(&name[i], sizeof(char), 12, fp) == 12)
	{
		fread(&sub[i], sizeof(int), 3, fp);
		fread(name[i], sizeof(char), 12, fp);
		fread(&sub[i], sizeof(int), 3, fp);
		ave[0] += sub[i][0];
		ave[1] += sub[i][1];
		ave[2] += sub[i][2];
		printf("%s %d %d %d\n", name[i], sub[i][0], sub[i][1], sub[i][2]);
		i++;
	}
	ave[0] /= (float)i;
	ave[1] /= (float)i;
	ave[2] /= (float)i;
	printf("Number of students : %d\n", i);
	printf("Average : %lf %lf %lf\n", ave[0], ave[1], ave[2]);
	fclose(fp);
	return 0;
}

試したこと

while文の中にfgetsを入れてみたりsizeof(Int)にしたりしましたが、ファイルがうまく出力されませんでした。
また、teratailでの皆様の回答を参考にさせていただき、whileの｛｝中のfreadの文を増やしてname→subのデータでループできるようにしたところ8個の正確なデータと平均点もデータが出たのですが、正答例を見ると"Kawai"のデータの上に"Kimura"というデータがあるらしく、やはりwhile文の中が間違えていると思うのです。

Comments

[y_waiwai]

質問は編集できます
無駄に質問建てないで追記修正しましょう

[episteme]

> データ個数が与えられていないのでfor文ではなくwhile文を用いる..

これには異議アリ。
いくらたくさんのデータがファイル内にあっても30個で頭打ちなんだから for文(による30回の固定loop) でOK。

[jimbe]

タグの Visual studio は関係ありますか? Visual studio で無いと発生しないのでしょうか。
バイナリも、バイナリファイルとは違う意味と思いますが。

Answer 1

	while (fread(&name[i], sizeof(char), 12, fp) == 12)
	{
		fread(name[i], sizeof(char), 12, fp);
		fread(&sub[i], sizeof(int), 3, fp);
		fread(&ave, sizeof(double), 3, fp);

少なくとも、 while の () 中に書いた fread によって name は読み取られてしまいますので、後の {} 内の fread の段階では読み取り位置がずれていて、想定通りの読み込みになっていません。
また、バイナリファイル内に ave のデータがあるとは書かれていないのに何を読もうというのでしょうか。

Answer 2

バイナリファイルを読みだすとこだけ:

  FILE*  fp;
  char   name[30][12];
  int    sub[30][3];
  double ave[3] = { 0.0,0.0,0.0 };  
  int    count = 0;

  const char* path = "bin-score.dat";

  fp = fopen(path, "rb");
  if ( !fp ) return -1;

  for ( int i = 0; i < 30; ++i ) {
    if ( fread(&name[i][0], sizeof(char), 12, fp) != 12 ) break;
    if ( fread(&sub[i][0],  sizeof(int),   3, fp) !=  3 ) break;
    printf("%s %d %d %d\n", name[i], sub[i][0], sub[i][1], sub[i][2]);
    ++count;
  }
  fclose(fp);