質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.81%

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

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 378

Masa_55

score 0

前提・実現したいこと

プログラミング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で製作しています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • cateye

    2020/07/05 16:42 編集

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

    キャンセル

  • Masa_55

    2020/07/05 16:51

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

    キャンセル

回答 1

0

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;
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/05 17:35

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

    キャンセル

  • 2020/07/05 17:40

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

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

    キャンセル

  • 2020/07/05 17:57

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.81%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る