C言語 GPAを計算し、表示するプログラム

Question

#include "wakabalite.h"
#include "gpa1.h"

// gpa算出関数(単位数, MARK構造体ポインタ, 科目数)
double get_gpa(int units, Mark *marks, int n){
    double gpSum = 0.0;
    int i;

    for(i=0;i<n;i++){
        gpSum += (double)(marks[i].score-54.5)*marks[i].unit/10;
    }
    return (double)gpSum/units;
}

int gpa1(pthread_t selfId, PGconn *con, int soc, char *recvBuf, char *sendBuf){
    PGresult *res, *resStd, *resSub, *resMrk, *resMrk2;  // SQL実行結果
    int resultRows, resultRows2, sendLen, i, n;
    int stNum, units=0;
    double gpa;
    char *subCode;
    char comm[BUFSIZE];
    char sql[BUFSIZE];
    char *sqlBegin="BEGIN", *sqlCommit="COMMIT", *sqlRollback="ROLLBACK";
    char userid[10];
    char year[10];
    char term[10]; 

    WakabaUser account;
    Student student;
    Mark marks[100];

    /* コマンド引数取得 */
    n = sscanf(recvBuf, "%s %s %s %s", comm, userid, year, term[0]);
    if(n!=4){
        sprintf("%s %d%s", ER_STAT, E_CODE_2, ENTER);
        return -1;
    }

    /*入力された学期を格納*/
    marks->term[0] = term[0];
    
    /* 学籍番号・科目コード・点数をとりだす*/
    sprintf(sql,"SELECT user_id,subject_code,score,year FROM marks WHERE user_id='%s' AND year='%s'", userid, year);
    printf("[%s]\n",sql);
    resStd = PQexec(con, sql);
    if(PQresultStatus(resStd) != PGRES_TUPLES_OK){
        printf("%s\n", PQresultErrorMessage(resStd));
        sprintf("%s %d%s", ER_STAT, E_CODE_4, ENTER);
        return -1;
    }
    resultRows = PQntuples(resStd);
    if(resultRows != 1){
        sprintf("%s %d%s", ER_STAT, E_CODE_1, ENTER);
        return -1;
    }

    // 学生・成績オブジェクトに各値を代入
    for(i=0; i<resultRows; i++){
    strcpy(student.userId,PQgetvalue(resStd, i, 0));
    strcpy(marks[i].subCode,PQgetvalue(resStd, i, 1));
    marks[i].score = atoi(PQgetvalue(res, i, 2));
    strcpy(marks[i].year,PQgetvalue(resStd, i, 3));
   }
    
    // 単位数を取得（科目コードから）
    sprintf(sql, "SELECT subject_code,unit_number,term FROM subject_t WHERE subject_code = 'marks->subCode' term = 'marks->term[0]'");
   
    printf("[%s]\n",sql);
    resMrk = PQexec(con, sql);
    if(PQresultStatus(resMrk) != PGRES_TUPLES_OK){
        printf("%s\n", PQresultErrorMessage(resMrk));
        sprintf("%s %d%s", ER_STAT, E_CODE_4, ENTER);
        return -1;
    }
    resultRows = PQntuples(resMrk);

    for(i=0;i<resultRows;i++){
       marks[i].unit = atoi(PQgetvalue(res, i, 1));
       
        // 単位数カウント
        units += marks[i].unit;
    }
    
    // GPA算出
    gpa = get_gpa(units,marks,resultRows);

    // レスポンス送信
    sendLen = sprintf(sendBuf,"units：%d GPA：%f%s", units, gpa, ENTER);
    send(soc, sendBuf, sendLen, 0);
    printf("[C_THREAD %ld] SEND=> %s", selfId, sendBuf);

    return 0;
}

ーー追記ーー
makeコマンドを使い、以下のファイルを一気にコンパイルしました。

$make
gcc wakabalite_server_main.o setup_listen.o wakabalite_controller.o common_func.o get_mark_clerk_func.h mark_divi.o gpa1.o -lpq -lpthread -o wakabalite_s

サーバーに接続してプロトコルコマンドを入力し、挙動を確認。

[C_THREAD 140385380718336] WAKABALITE CONTROLLER START (4)
[C_THREAD 140385380718336] RECV=> GPA 66660320 2020 前期  
[SELECT user_id,subject_code,score,year FROM marks WHERE user_id='66660320' AND year='2020']
Segmentation fault

コマンドプロンプトで実行したところ、セグメンテーションフォルトが出力されます。
どこがおかしいか教えていただけませんか？

Comments

[kay-ws]

・どのようにコンパイルしたか
・どのように実行したか
を追記してください。

[tiroha]

分かりました。

Answer 1

n = sscanf(recvBuf, "%s %s %s %s", comm, userid, year, term[0]);

term[0]になっている。
termもしくは&term[0]とする必要がある。

Comments

[tiroha]

回答ありがとうございます！termと＆term[0]でやってみたとことまだ、セグメンテーションフォルトが出てきてしまいます。

[ttyp03]

ここもおかしいですね。

marks->term[0] = term[0];

marksは配列なので、marks[0].term などと書くべきです。
また文字列のコピーをしたいなら、strcpyなどを使うべきです。

strcpy(marks[0].term, term);

しかしmarks配列のtermは使用していないので、これ自体が悪さすることはなさそうです（実装はおかしいので修正したほうがいいです）

他はPQgetvalue()の戻り値を直接strcpyしていますが、PQgetvalue()の戻り値はPGresult構造体のアドレスなのでそのままは使えないでしょう。
詳しくはないのでマニュアルを参照してください。
https://www.postgresql.jp/document/7.3/programmer/libpq-exec.html

あとはBUFSIZEの値がいくつかわかりませんが、足りているかなどご確認ください。

要所要所にデバッグ文を入れてどこまで進んだのかも確認してみてください。

Answer 2

printf(check1\n);

で地道にチェックしていったところ、セグメンテーションフォルトが無くなりました。

Answer 3

セグメンテーションフォルトが出力されます。

ってことはどこかでメモリ破壊起こしているということです。
VisualStudioやEclipseなど、ソースコードデバッグできる環境を整え、あなたのコードをデバッグしましょう。

＃んで、main関数がないようなので実行しようがないと思いますが、どうやって実行したんでしょうか

Comments

[tiroha]

いくつかのファイルを同時にコンパイルしています。このコードはmainの中のモジュールです。サーバーに接続してプロトコルコマンドで挙動をみています。