構造体とポインタについて

Question

構造体とポインタ

プログラミング演習の質問です
「stdata.hを読み込んで英語，数学，物理の平均値と分散を求めるプログラムを作成せよ。コンパイルに必要なヘッダーファイル（.hファイル）と
プログラムファイル(.cファイル)を全て提出してください。
stdata.hはプログラム中で以下のようにインクルードすることで
データを読み込んでください。
include "stdata.h"」
おそらくポインタが間違っていると思いますが、どこがだめで、どのように直せばいいのかわかりません。

stdata.hは

#define NUM 100
student st[NUM] = {
 {"0001", "ayase kurumi", 85, 84, 71}
,{"0002", "iijima mitsuru", 90, 70, 83}
,{"0003", "ishizaka yutaka", 55, 81, 88}
.....
,{"0100", "yoshinaga yuu", 78, 77, 65}
};

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

horyu1.c:23:17: error: expected identifier or ‘(’ before ‘[’ token
struct test_data[]=(student *p,char *id, char *name,
^
horyu1.c: In function ‘main’:
horyu1.c:46:11: error: invalid operands to binary + (have ‘int’ and ‘student’ {aka ‘struct <anonymous>’})
total += st[i];
^~ ~~~~~
horyu1.c:51:20: error: invalid operands to binary - (have ‘student’ {aka ‘struct <anonymous>’} and ‘float’)
var += ((st[i] - avg) * (st[i] - avg)) / (float)NUM;
~~~~~ ^
horyu1.c:51:36: error: invalid operands to binary - (have ‘student’ {aka ‘struct <anonymous>’} and ‘float’)
var += ((st[i] - avg) * (st[i] - avg)) / (float)NUM;

該当のソースコード

#include <stdio.h>
#define N 100

 typedef struct{
  char id [8];
  char name [64];
  int eng;
  int math;
  int phys;
  double avg;
  double var;
} student;

#include "stdata.h"


struct test_data[]=(student *p,char *id, char *name,
			      int eng, int math, int phys)
  
  {
  strcpy(p->id,id);
  strcpy(p->name,name);
  p->eng = eng;
  p->math = math;
  p->phys = phys;
}
  

  int main()
  {
   ; 
  int i; 
  int total;
  float avg;
  float var;
  total = 0;
  avg = 0.0;
  var = 0.0;
  for(i = 0; i < NUM; i++){
    total += st[i];
    avg = total/(float)NUM;
    printf("Average: %.2f\n",avg);
  }
  for(i = 0; i < NUM; i++){
    var += ((st[i] - avg) * (st[i] - avg)) / (float)NUM;
    printf("Variance: %.2f\n",var);
  }

  printf("----------------------------\n");
  
    printf(" English Average:%.2f\n",st[i].eng);
    printf(" Math Average:%.2f\n",st[i].math);
    printf(" Physics Average:%.2f\n",st[i].phys);
    printf(" English Variance:%.2f\n",st[i].eng);
    printf(" Math Variance:%.2f\n",st[i].math);
    printf(" Physics Variance:%.2f\n",st[i].phys);

    return 0;
}

試したこと

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

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

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

Comments

[bboydaisuke]

> 全て提出してください。

大学の宿題ですよね。他所でもまったく同じ質問がありますね。

[ishina_yum]

プログラム以前に「何すれば良いのか」が分かっていないように見えます。

まず『どういう手順で計算するのか』をプログラムではなく言葉で書いてみるところから始めるのが良いような気がします。

[jimbe]

> おそらくポインタが間違っていると思います
残念ながら、そのような段階までも進んでいません。
ソースの一部が C の構文としてメチャクチャなのでコンパイラがエラーと報告しているだけです。課題でしたら、過去に習ったことを見直すことをお勧めします。

[BeatStar]

基礎からやりましょう

Answer 1

どこがだめで、どのように直せばいいのかわかりません。

この↓部分がダメで、何をしたいのかわからんので直せません。

struct test_data[]=(student *p,char *id, char *name,
                  int eng, int math, int phys)

  {
  strcpy(p->id,id);
  strcpy(p->name,name);
  p->eng = eng;
  p->math = math;
  p->phys = phys;
}

Answer 2

見にくいコードになってしまい申し訳ありません。

stdata.hのなかでは#define NUM 4としています。
studentのavgとvarを削除しました。
test_data関数（？）も削除しました。

このコードの出力は,

 English Average:77.00
 Math Average:78.00
 Physics Average:76.75
 English Variance:0.25
 Math Variance:0.25
 Physics Variance:34.50

コード

#include <stdio.h>

typedef struct{
    char id [8];
    char name [64];
    int eng;
    int math;
    int phys;
} student;


#include "stdata.h"

extern student st[NUM];

int main(void){
    int english_total=0;
    int math_total=0;
    int physics_total=0;
    int english_num=0;
    int math_num=0;
    int physics_num=0;
    for(int i=0;i<NUM;i++){
        english_total+=st[i].eng;
        math_total+=st[i].math;
        physics_total+=st[i].phys;
    }
    for(int i=0;i<NUM;i++){
        english_num=(st[i].eng-english_total/(double)NUM)*(st[i].eng-english_total/(double)NUM);
        math_num=(st[i].math-math_total/(double)NUM)*(st[i].math-math_total/(double)NUM);
        physics_num=(st[i].phys-physics_total/(double)NUM)*(st[i].phys-physics_total/(double)NUM);
    }
    printf(" English Average:%.2f\n Math Average:%.2f\n Physics Average:%.2f\n English Variance:%.2f\n Math Variance:%.2f\n Physics Variance:%.2f\n",english_total/(double)NUM,math_total/(double)NUM,physics_total/(double)NUM,english_num/(double)NUM,math_num/(double)NUM,physics_num/(double)NUM);
    return 0;
}