C言語の構造体についての質問です。

質問

C言語の構造体についての質問です。
学校でC言語の授業を受けていて、課題が出たのですが、わかりませんでした。回答よろしくお願いします。
課題内容は、
配列を用いて以下の処理をおこなうプログラムを作成しなさい。
テキストファイルから番号、名前、英語、国語、数学３科目の得点を読み込む。
各自の３科目平均点を計算する。
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MSTDNT 10
#define MNAMEL 10
#define MBFFRL 32

typedef struct student{
int id;
char name[MNAMEL];
int engl;
int jpan;
int math;
float mean;
} STDNT;

void readin(STDNT *p);
void disply(STDNT *p);
void dscend(STDNT *p);

int main(void){
STDNT students[MSTDNT];

readin(students);
disply(students);
printf("reorder by descending mean\n");
dscend(students);
disply(students);

return 0;
}

//_(1)_番号、名前、英語、国語、数学の得点を読込む
//_(2)_各自の３科目平均点を計算する
void readin(STDNT *p){
int i;
FILE *file;
char *fnm="h109113.txt";
char bffr[MBFFRL];

if((file=fopen(fnm,"r"))==NULL){
printf("ファイル%sはありません\n",fnm);
exit(1);
}
for(i=0;i<MSTDNT;i++){
if(fgets(bffr,MBFFRL,file)!=NULL){
printf("ファイル%sは不完全です\n",fnm);
printf("No.%dレコード？\n",i);
exit(2);
}
p->id=atoi(strtok(bffr,",\n"));
strcpy(p->name,strtok(NULL,",\n"));
p->engl=atoi(strtok(NULL,",\n"));
p->jpan=atoi(strtok(NULL,",\n"));
p->math=atoi(strtok(NULL,",\n"));
p->mean=(float)(p->engl+p->jpan+p->math)/3;
}
fclose(file);
return;
}

//成績表の格納状況を表示する
void disply(STDNT *p){
int i;

printf("番号 名前 英語 国語 数学 平均\n");
for(i=0;i<MSTDNT;i++){
printf("%4d %4s %4d %4d %4d %4.1f\n",
p->id,p->name,p->engl,p->jpan,p->math,p->mean);
}
return;
}

//(3)３科目の平均点の高い順に並び変える
void dscend(STDNT *p){
int i,j;
STDNT tmp;

for(i=0;i<MSTDNT-1;i++){
for(j=i+1;j<MSTDNT;j++){
if(p[i].mean>p[j].mean){
tmp=p[i];p[i]=p[j];p[j]=tmp;
}
}
}
return;
}
出力がちゃんとできません。よろしくお願いします。

回答No.4

kmee · ベストアンサー

> ファイルh109113.txtは不完全です
> No.0レコード？

→ この表示はどこか?

>if(fgets(bffr,MBFFRL,file)!=NULL){
>printf("ファイル%sは不完全です\n",fnm);
>printf("No.%dレコード？\n",i);
>exit(2);
>}

でifの条件が成立ったとき。

→内容からして、エラー時の対策のようだ
→→入力ファイルは正しいか? → 正しそうだ
→→本当に読めて無いのだろうか?→bffrの内容を確認(デバッガの使用、printfでの表示等)→どうやら入力できているようだ
→→そもそも、この判定は正しいか→fgetsのマニュアルを確認→戻り値は「EOF等で正しく読めなかった場合はNULL、それ以外は読み込んだ第1引数のポインタ(!=NULL)」→正常時にエラー処理をしている!

以上、期待する動作→実際の動作の違いから、プログラムの問題点を見つけるまでの一例です。
何がどう「出力がちゃんとでき」ないか、をはっきりさせることが、解決に繋がるのがわかるのではないでしょうか?

プログラムの勉強で、文法やコマンドを覚えるのは確かに大事ですが、そんなのは、忘れた時はマニュアル等を読み直せばいいだけのことです。
それよりも、何か不具合があったとき、現象から原因をつきとめらるようにすることが、とても重要です。
これには、ある程度の訓練と経験が必要です。「ちゃんとできません」だけでは訓練になりません。

回答No.3

Wr5

>上記のように出力したいのですが、
>ファイルh109113.txtは不完全です
>No.0レコード？
>と出力されます。

……回答の内容見ています？
#1でkmeeさんが
>fgetsの戻り値の仕様をよく確認してください。下の内容から、エラー時の処理に見えますが、fgetsの戻り値が !=NULL ならエラーではありません。
と回答されていますよね？
# 私はそこまで見ていませんでしたが…

!=NULLで正常動作なのにエラー処理として中断しているのですから、その後の動作が続くはずがありませんよね？

回答No.2

Wr5

>出力がちゃんとできません。

とは、「どういう出力」を期待していて、「どういう結果」になったのですか？

とりあえず、readin()とdisply()の操作対象はmain()のSTDNT students[0]だけ。
のようですが、期待する動作ですか？
# STDNT *pの値が、関数コールされてから変化していないようですが…。

ちなみに、readin()は期待する形式以外のファイルが指定された場合に吹っ飛びます。
1行が
10,聖徳太子,30,100,70
の場合は問題ないでしょうが、
10,聖徳太子,30,,70
や
10,聖徳太子,30,100
の時は……。

補足

番号_名前_英語_国語_数学__平均
___1_安藤___30___50___90__56.7
___2_井上___70___80___60__70.0
___3_上田___80___70___80__76.7
___4_遠藤___90___80___80__83.3
___5_岡田___90___70___80__80.0
___6_加藤___80___80___70__76.7
___7_木村___70___70___60__66.7
___8_工藤___80___80___80__80.0
___9_小林___80___80___80__80.0
__10_佐藤___70___80___80__76.7
reorder_by_descending_mean
番号_名前_英語_国語_数学__平均
___4_遠藤___90___80___80__83.3
___5_岡田___90___70___80__80.0
___8_工藤___80___80___80__80.0
___9_小林___80___80___80__80.0
___3_上田___80___70___80__76.7
___6_加藤___80___80___70__76.7
__10_佐藤___70___80___80__76.7
___2_井上___70___80___60__70.0
___7_木村___70___70___60__66.7
___1_安藤___30___50___90__56.7
上記のように出力したいのですが、
ファイルh109113.txtは不完全です
No.0レコード？
と出力されます。
テキストファイルの中身は
1, 安藤, 30, 50, 90
2, 井上, 70, 80, 60
3, 上田, 80, 70, 80
4, 遠藤, 90, 80, 80
5, 岡田, 90, 70, 80
6, 加藤, 80, 80, 70
7, 木村, 70, 70, 60
8, 工藤, 80, 80, 80
9, 小林, 80, 80, 80
10, 佐藤, 70, 80, 80
です。

回答No.1

kmee

> 出力がちゃんとできません。よろしくお願いします。

どのように「ちゃんとでき」ないのか、自分の期待する結果とどう違うかを、明記するようにしましょう。
自分で解決するにしても、人に頼むにしても、「期待する結果とどう違うか」が解決への第一歩です。

> if(fgets(bffr,MBFFRL,file)!=NULL){
> printf("ファイル%sは不完全です\n",fnm);
> printf("No.%dレコード？\n",i);
> exit(2);
> }

fgetsの戻り値の仕様をよく確認してください。下の内容から、エラー時の処理に見えますが、fgetsの戻り値が !=NULL ならエラーではありません。

> for(i=0;i<MSTDNT;i++){
(略)
> p->id=atoi(strtok(bffr,",\n"));
(略)
>p->mean=(float)(p->engl+p->jpan+p->math)/3;
>}

このループで、ポインタpの値は変化せず、構造体の配列の先頭要素を指したままです。
つまり、main関数の students[0];に全部のデータが上書きされます。
p[i].id 等とするか、(p自体はポインタを値渡ししたもので、p自体を変化させても呼び出し側に影響が無いので) ループ終了前に ++ p ; とするか、です。

> //成績表の格納状況を表示する
> void disply(STDNT *p)
も同様。

> //(3)３科目の平均点の高い順に並び変える
> void dscend(STDNT *p){
このソートのアルゴリズムをよく見てください。
i=0の処理が終った段階で、p[0]に入っていなければならないのは、どんな構造体でしょうか?
実際にはどの構造体が入っているでしょうか?

補足

テキストファイルの中身は
1, 安藤, 30, 50, 90
2, 井上, 70, 80, 60
3, 上田, 80, 70, 80
4, 遠藤, 90, 80, 80
5, 岡田, 90, 70, 80
6, 加藤, 80, 80, 70
7, 木村, 70, 70, 60
8, 工藤, 80, 80, 80
9, 小林, 80, 80, 80
10, 佐藤, 70, 80, 80
です。