構造体やファイルの入力が絡んだ関数呼び出しが出来ません。

Question

前提・実現したいこと

プログラミングC言語初心者です。
構造体で表された個人データをファイルから読み込んで、データ処理として科目の平均やGPAなどを作成するプログラムを作っており、最初に必要な科目別の合計を求める関数がうまくmain関数に反映されません。
どうすればいいのでしょうか。ご協力お願いします。

該当のソースコード

/*
　　学生の成績管理システムを作成
*/
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define NUMBER 5

typedef struct {
    char str[128];                    /*在籍番号*/
    char name[64];                    /*氏名*/
    double Japanese;                  /*国語の点数*/
    double math;                      /*数学の点数*/
    double English;                   /*英語の点数*/
} Student;

int main(void){
    FILE   *fp;
    int i;
    Student std[NUMBER];
    if((fp = fopen("File1.dat", "r")) == NULL){
      printf("\aファイルをオープン出来ません。\n");
    }
    else{
        for(i = 0;i < NUMBER; i++){
            while (fscanf(fp, "%s %s %lf %lf %lf\n", std[i].str, std[i].name,
            &std[i].Japanese, &std[i].math,&std[i].English) == 5){
                printf("%-10s %-10s  %5.1f  %5.1f  %5.1f\n", std[i].str, std[i].name,
                std[i].Japanese, std[i].math,std[i].English);
                
            }
            
        }
        printf("------------------------------------------\n");
        sum(std, NUMBER);
        printf("科目別の合計点　　　　　 %5.1f %5.1f %5.1f\n",std[i].Japanese , std[i].math, std[i].English);
        
    }
    fclose(fp);
    return 0;
    }

    
double sum(Student a[], int n)
{
    int i;
    double jsum = 0.0;
    double msum = 0.0;
    double esum = 0.0;
    
    for(i = 0; i < n; i++){
        jsum += a[i].Japanese;
        msum += a[i].math;
        esum += a[i].English;
    }
    
}

補足情報（ファイルの入力）

T19H100a  DAICHI_ARAI    60     70  55
T19H134j  RIKU_TAKAMA    70     95  70
T18H236k  YUYA_ASAKA     55     75  69
E19C554m  DAIKI_TOTSUKA  100    50  95
L15U934o  KAITO_ARAI     40     30  58

前提・実現したいこと

科目別の合計点を求める関数をmain関数に反映させたいです。

エラーコード

Main.c:35:9: warning: implicit declaration of function 'sum' is invalid in C99 [-Wimplicit-function-declaration]
        sum(std, NUMBER);
        ^
Main.c:43:8: error: conflicting types for 'sum'
double sum(Student a[], int n)
       ^
Main.c:35:9: note: previous implicit declaration is here
        sum(std, NUMBER);
        ^
1 warning and 1 error generated.

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

paiza.ioで製作しています。

Comments

[cateye]

error: conflicting types for 'sum'・・・ sum()が2重定義
・・・・標準ライブラリある関数名は使わない方が・・・どっちが、使われるか分からない。

[Masa_55]

科目別の合計点を求める関数名をtotalにしてやってみます！
ご指摘ありがとうございます！

Answer 1

sum関数のプロトタイプ宣言を置く必要があります

double sum(Student a[], int n);

int main(void){
    FILE   *fp;
    ...

結果を返す関数の一例ですが

void gokei(Student a[], int n,Student* result)
{
    int i;
    result->Japanese=0;
    result->math=0;
    result->English=0;

    for(i = 0; i < n; i++){
        result->Japanese += a[i].Japanese;
        result->math += a[i].math;
        result->English += a[i].English;
    }
}

Comments

[Masa_55]

回答ありがとうございます。
出力結果が以下のようになってしまったのですが、どうすればいいですか？

T19H100a DAICHI_ARAI 60.0 70.0 55.0
T19H134j RIKU_TAKAMA 70.0 95.0 70.0
T18H236k YUYA_ASAKA 55.0 75.0 69.0
E19C554m DAIKI_TOTSUKA 100.0 50.0 95.0
L15U934o KAITO_ARAI 40.0 30.0 58.0
------------------------------------------
科目別の合計点　　　　　 0.0 0.0 0.0

[y_waiwai]

それをどうしたいというはなしでしょうか。

質問が変わってしまっていますが。。

[Masa_55]

申し訳ありません。
合計点が何で0.0と表示されてしまうのか教えていただきたいです。

[y_waiwai]

sum関数というのは計算の結果をどこにも返していません。
ただ計算して捨てているだけです

んで、もひとつ、
提示のコードでは、std配列の範囲を超えてアクセスしています。
不正アクセスとなってますね。

[Masa_55]

関数使って求めるのできなさそうなので、普通にmain関数で計算するプログラム作ろうと思います。
ありがとうございました。

[y_waiwai]

やり方がまずいだけで、関数使ってでもできますよ。
まあ、とりあえず関数使わないで思うようになるようにコードを組んでみてください。

んで、C言語でコードを組むなら、デバッグできる環境を揃えましょう
Eclipseや、WindowsならVisualStudioなど。
コードの任意の行で実行を止め、変数の中身を参照できます。
また、そこから１行づつ実行させてコードの流れや変数の変化を見ることができます

[Masa_55]

一応関数使わないで出来たのですが、この先標準偏差なども求めようと思っているのでやっぱり関数は必要です。
関数使った場合の科目別の合計点を求めるプログラムを見せていただけませんか？
参考にさせていただきたいです。