2次元配列の条件分岐がわからない。

Question

###実現したいこと
合格基準はすべての試験科目の点数が60点以上かつその人の平均点が70点以上。
合格者を表示したい。
###問題点
ご指摘いただいたところを治しましたところ満たしていないはずの2番さんが表示されてしまいました。

学籍番号1
70 60 80
学籍番号2
75 58 90
学籍番号3
75 80 90
学籍番号4
65 80 60
学籍番号5
60 100 60
合格者一覧
 1番 210点 平均70.00
 2番 223点 平均74.33
 3番 245点 平均81.67
 5番 220点 平均73.33

条件分岐が間違っているとおもうのですがどう変えればいいのかわかりません。ご指摘お願いします。
###コード

/*2次元配列を用いた項目別成績の集計プログラム*/
#include<stdio.h>
#define NUMBER 5 //学籍番号
#define SUBJECT 3 //教科数
int main(void)
{
    int i,j;
    int ssum=0; //合格者の人数
    int tensu[NUMBER][SUBJECT+1]={0}; //項目別の点数
    int succs[NUMBER]; //合格者のリスト
    double avg[5];
    for(i=0;i<NUMBER;i++){
        printf("学籍番号%d\n",i+1);
        for(j=1;j<=SUBJECT;j++){
            //printf("項目%d:",j);
            scanf("%d",&tensu[i][j]);
            tensu[i][0]+=tensu[i][j];
            avg[i]=(double)tensu[i][0]/SUBJECT;
        /*printf("平均点:%.2f\n",avg[i]); //デバック用
        printf("合計点:%2d\n",tensu[i][0]);*/
        if(tensu[i][j]>=60 && avg[i]>=70){ //すべてが60点以上かつ平均70点以上
            succs[ssum++]=i;
        }
        }
    }
    printf("合格者一覧\n");
    for(i=0;i<ssum;i++){
        printf("%2d番 %3d点 平均%.2f\n",succs[i]+1,tensu[succs[i]][0],(double)tensu[succs[i]][0]/SUBJECT);
    }
    return 0;
}

Comments

[thkana]

https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q13226565128
teratailでは「マルチポスト」は推奨されていません。https://teratail.com/help#posted-otherservice 「禁止」ではありませんが、それでもやるんだ、というなら書かれているように「やむを得ず複数のサイトに質問を投稿された場合は、質問内容にマルチポストをする理由を書き、他のサイトの投稿へのリンクを貼ってください。また、解決した際には必ずteratail及びすべての投稿に解決した旨と、どのように解決したかを記載してください。」

Answer 1

if(tensu[i][j]>=60 && avg[i]>=70){ //合計180点以上かつ平均70点以上

avg[i]が平均点で、かつ70点以上なのはいいですが
tensu[i][j]が合計点なのか？
なぜ60と比較することで「合計180点以上」になるのか？は考えないといけません。

また、「合計180点以上」では「平均60点以上」と何も変わらない事に注意が必要です。

Answer 2

        if(tensu[i][j]>=60 && avg[i]>=70){ //合計180点以上かつ平均70点以上
            succs[ssum++]=i;
        }

ここでのjは、上でループを抜けきった後なので、4になっています。結果、配列の範囲外を参照する形となっていて、まともな比較条件ではありません。

Answer 3

なぜにわざわざ難しく考えるんだろう。
なぜにすべてをmainに詰め込もうとするんだろう。
解決したい問題を分割し、それを関数で解決すればいいのに。

#include<stdio.h>

#define NUMBER 5 //学籍番号
#define SUBJECT 3 //教科数

// 総得点を返す
int sum(int score[]) {
    int result = 0;
    int i;
    for ( i = 0; i < SUBJECT; ++i ) {
      result += score[i];
    }
    return result;
}

// score[0]～score[SUBJECT-1]のすべてが60点以上 
// かつ sum(score) >= 70*SUBJECT なら 1、さもなくば 0 を返す
int pass(int score[]) {
    int i;
    for (i = 0; i < SUBJECT; ++i) {
        if ( score[i] < 60 ) return 0;
    }
    if ( sum(score) >= 70*SUBJECT ) return 1;
    else return 0;
}

int main(void) {
    int i, j;
    int tensu[NUMBER][SUBJECT];

    // 得点表を作る
    for (i = 0; i < NUMBER; i++) {
        printf("学籍番号%d\n", i + 1);
        for (j = 0; j < SUBJECT; j++) {
            //printf("項目%d:",j);
            scanf("%d", &tensu[i][j]);
        }
    }

    // 合格者のみプリント
    printf("合格者一覧\n");
    for (i = 0; i < NUMBER; i++) {
        if ( pass(tensu[i]) ) { // 合格してたら書く。
          printf("%2d番 %3d点 平均%.2f\n", 
                 i+1, sum(tensu[i]), (double)sum(tensu[i])/SUBJECT );
        }
    }
    return 0;
}

Comments

[langhtorn]

関数で考えた方がすっきりしていていいかもしれません。ありがとうございました。

[episteme]

いいに決まってんじゃない? 合格条件が変わっても関数pass()を修正するだけなんだし。