C言語の条件文について

Question

3つのテストの点数を5人分入力して、すべての科目の点数が60点以上かつ平均点が70点以上の人の番号と平均点を出力するというプログラムなのですが、すべての科目の点数がx点以上かつ平均点がy点以上という条件をif文で書こうとしても配列を用意して（例えばa[]は点数の配列、配列の要素数はテストの数、yは合計点とする）a[0]>=x && a[1]>=x &&..a[n]>=x && scr>=yという冗長な条件文しか思いつきません。3つのテストでなく20つぐらいのテストでも簡潔に書くことのできる方法が知りたいです。下のコードでは上の条件のような限られた条件でしかうまく動作しません。(理由がすべての科目の点数に対しての条件文を書いていないことは分かっています)

＃include<stdio.h>
＃define A 5
＃define B 3

int main(void) {
int scr[A][B], cl[A],t = 0;
double fig[A];
for (int i = 0; i < A; i++) {
int tmp = 0;
printf("%d番 :",i+1);
for (int k = 0; k < B; k++) {
scanf_s(" %d", &scr[i][k]);
if(scr[i][k]>=60) tmp = tmp + scr[i][k];
}
if (tmp >= 210) {
cl[t] = i+1;
fig[t] = tmp / 3.0;
t++;
}
}
printf("合格者一覧\n");
for (int i = 0; i < t; i++) {
printf("%d番 : %.2f\n", cl[i], fig[i]);
}
return 0;

Answer 1

簡潔に、かつ柔軟にですか。

#include <stdio.h>

#define N_STUDENT   5   // 生徒の人数
#define N_SUBJECT   3   // 科目数
#define MIN_SCORE   60  // 各科目の最低点
#define MIN_AVERAGE 70  // 平均点の最低点

int main(void)
{
	int n = 0;              // 合格者数
	int id[N_STUDENT];      // 合格者番号
	double ave[N_STUDENT];  // 合格者平均点
	for (int i = 1; i <= N_STUDENT; i++) {
		int sum = 0, pass = 1;  // 合計点、合格フラグ
		printf("%d番 :", i);
		for (int k = 0; k < N_SUBJECT; k++) {
			int score;
			scanf("%d", &score);
			if (score < MIN_SCORE) pass = 0;  // 合格取り消し
			sum += score;
		}
		if (sum < MIN_AVERAGE * N_SUBJECT) pass = 0;  // 合格取り消し
		if (pass) id[n] = i, ave[n++] = (double)sum / N_SUBJECT;
	}
	puts("合格者一覧");
	for (int i = 0; i < n; i++)
		printf("%d番 : %.2f\n", id[i], ave[i]);
}

Answer 2

下記でいかがでしょうか？

#include<stdio.h>
#define A 5
#define B 3 
#define TH1 60
#define TH2 70.0f

int main(void) {
    int scr[A][B], cl[A],t = 0;
    double fig[A];
    for (int i = 0; i < A; i++) {
        int tmp = 0;
        printf("%d番 :",i+1);
        for (int k = 0; k < B; k++) {
            scanf_s(" %d", &scr[i][k]);
        }

        if( (getMinArr(&scr[i], B) >= TH1)  // 最小値が60以上
         && (getAveArr(&scr[i], B) >= TH2)) // 平均値が70以上
        {
            cl[t] = i+1;
            fig[t] = getAveArr(&scr[i], B);
            t++;
        }

    }
    printf("合格者一覧\n");
    for (int i = 0; i < t; i++) {
        printf("%d番 : %.2f\n", cl[i], fig[i]);
    }
    return 0;
}


// 配列の最小値を返す(ただし、Sizeが0の時は-1を返す)
int getMinArr(int arr[], int size)
{
    int i;
    int Min = arr[0];

    if(size == 0){
        return -1;
    }

    for(i=0;i<size;i++){
        if(Min > arr[i]){
            Min = arr[i];
        }
    }

    return Min;
}

// 配列の平均値を返す(ただし、Sizeが0の時は-1を返す)
double getAveArr(int arr[], int size)
{
    int i;
    double Ave = 0;

    if(size == 0){
        return -1;
    }

    for(i=0;i<size;i++){
        Ave += arr[i];
    }

    return Ave/size;
}

Comments

[退会済みユーザー]

回答ありがとうございます。初心者にも分かりやすいコードで助かりました。ただ一つメソッドの呼び出し側で&scr[i]というように&が付いている理由がよく分かりませんでした。

[takayukid]

&scr[i]は、関数の引数に2次元配列の1次元目の要素iの先頭アドレスを渡すためです。
getMinArr(), getAveArr()の引数に配列の先頭アドレスarr[]を受け取り、
arr[]からサイズ(size)分の要素の中から、最小値、平均値を算出するようにしています

Answer 3

// score[0]～score[n-1] の合計を返す
int sum(int score[], int n) {
    int result = 0;
    int i;
    for ( i = 0; i < n; ++i ) {
      result += score[i];
    }
    return result;
}

// score[0]～score[n-1]のすべてが60点以上 
// かつ sum(score) >= 70*n なら 1、さもなくば 0 を返す
int pass(int score[], int n) {
    int i;
    for (i = 0; i < n; ++i) {
        if ( score[i] < 60 ) return 0; // 60点未満だったら即不合格
    }
    // ここに達したら合格かもしれない。が...
    if ( sum(score) >= 70*n ) return 1; // うん、合格!!
    else return 0; // 平均70点未満だと不合格
}

Comments

[episteme]

類似(てかむしろ同一の)質問がココ↓に。

https://teratail.com/questions/269030

[退会済みユーザー]

回答ありがとうございます。ぱっと見理解できないところがあるので自分で調べて紐解いてみます。

[退会済みユーザー]

理解できました。ありがとうございました。

Answer 4

「すべての科目の点数がx点以上」をチェックする必要があるので、
案1) これを展開して全て書く(またはマクロで展開する)
案2) for文等のloop処理内で判定する
のどちらかしかありえません。
それから、「60」、「210」、「3.0」とかのマジックナンバーはやめてdefine文で書いておかないと、条件が変わって修正が必要になった時に該当個所を探すのがたいへんです。

Answer 5

すべての科目の点数が60点以上かつ平均点が70点以上、という判定を行う関数を作りましょう
そうすれば単純なループとif文で事が済みます

struct _tensu{
  int rika;
  int sansu;
  int syakai;
}

...
int hantei(struct _tensu* t)
{
  if(i->tika<60 || t->sansu<60 || t->syakai<60) return 0;
  if((i->tika+t->sansu+t->syakai)<(70*3)) return 0;
  return 1;
}

struct _tensu tensu[5];  // ５人分

...

for(int i=0;i<5;i++){
  if(hantei(tensu+i)){ 平均点出す }
}