ｃ言語について教えてください

二次元配列で、各配列の平均と最大の値を求めたいのですが上手くいきません。
どこをどのように直せば良いのでしょうか？

#include <stdio.h>
int main(void){
int i;
float sum1,sum2,sum3,sum4;
float max1,max2,max3,max4;
float a[4][3]={{76.51,78.25,76.99},{78.71,78.16,77.12},{79.14,80.59,78.88},{73.31,73.35,74.95}};

for(i=0;i<3;i++){
sum1=a[0][i]+sum1;
sum2=a[1][i]+sum2;
sum3=a[2][i]+sum3;
sum4=a[3][i]+sum4;
}

max1=a[0][1];
if(max1<a[0][2]){
max1=a[0][2];
}
if(max1<a[0][3]){
max1=a[0][3];
}

max2=a[1][1];
if(max2<a[1][2]){
max2=a[1][2];
}
if(max2<a[1][3]){
max2=a[1][3];
}

max3=a[2][1];
if(max3<a[2][2]){
max3=a[2][2];
}
if(max3<a[3][3]){
max3=a[3][3];
}

max4=a[4][1];
if(max4<a[4][2]){
max4=a[4][2];
}
if(max4<a[4][3]){
max4=a[4][3];
}

printf("<選手番号1>平均距離:%.2f,最大距離:%.2f\n",sum1/3,max1);
printf("<選手番号2>平均距離:%.2f,最大距離:%.2f\n",sum2/3,max2);
printf("<選手番号3>平均距離:%.2f,最大距離:%.2f\n",sum3/3,max3);
printf("<選手番号4>平均距離:%.2f,最大距離:%.2f\n",sum4/3,max4);

return 0;
}

No.3 ベストアンサー 回答者： cametan_42 回答日時： 2021/03/31 17:02

> ありがとうございます。

参考にさせていただきます。

ほい、頑張って。

どこまで勉強進んでるか分からんけど、大きく言うと、

☆ 何でもかんでもmain関数内で処理しない

多分配列やってる辺りだともう関数を習ってる筈なんですが。
この問題のように、

> 各配列の平均と最大の値を求めたい

と言う場合、すぐ「平均と最大の値を同時に求めよう」としない。
そうじゃなくって、平均なら平均、最大なら最大を求める、と。
プログラムをする際に機能を最小限に抑える。そして「それらを組み合わせる」。
そのために「関数がある」のです。一つの機能を実装する為に。

大きく「全部いきなりやろう」としない事。全部いきなりやろうとすると、貴方のように混乱するんです。
小さいものを書く。そして小さいもの同士を「組み合わせていく」。結局それで目的のモノを作り上げるのです。

No.2 回答者： cametan_42 回答日時： 2021/03/31 16:42

#include <stdio.h>

#include <float.h>

float average(int n, float a[]){
　int acc = 0;
　for (int i = 0; i < n; i++) {
　　acc += a[i];
　}
　return acc/n;
}

float max(int n, float a[]) {
　float min = -FLT_MAX;
　for (int i = 0; i < n; i++) {
　　if (a[i] > min) {
　　　min = a[i];
　　}
　}
　return min;
}

int main(void) {
　float a[4][3] = {{76.51, 78.25, 76.99},
　　　　　　　　{78.71, 78.16, 77.12},
　　　　　　　　{79.14, 80.59, 78.88},
　　　　　　　　{73.31, 73.35, 74.95}};
for (int i = 0; i < 4; i++) {
　printf("<選手番号%d>平均距離:%.2f, 最大距離:%.2f\n", i + 1, average(3, a[i]), max(3, a[i]));
　}
　return 0;
}

この回答へのお礼

ありがとうございます。参考にさせていただきます。

お礼日時：2021/03/31 16:54

No.1 回答者： neKo_deux 回答日時： 2021/03/31 16:27

> for(i=0;i<3;i++){

> sum1=a[0][i]+sum1;

いきなり足し合わせる前に、sum1～sum4を0.0に初期化した方が良いのでは。
変数宣言すると0に初期化してくれるコンパイラとか処理系もあるけど、Cの仕様では中身の保証は無かったハズ。


改行無くすと、
max1=a[0][1];if(max1<a[0][2]){max1=a[0][2];}if(max1<a[0][3]){max1=a[0][3];}
max2=a[1][1];if(max2<a[1][2]){max2=a[1][2];}if(max2<a[1][3]){max2=a[1][3];}
max3=a[2][1];if(max3<a[2][2]){max3=a[2][2];}if(max3<a[3][3]){max3=a[3][3];}
max4=a[4][1];if(max4<a[4][2]){max4=a[4][2];}if(max4<a[4][3]){max4=a[4][3];}

配列の添え字が変じゃね？0,1,2,4とか。0,1,3,4とか。


課題の意図としては、最初に数値設定して以降の4回とか3回似たような事書いてる所は、繰り返し処理にまとめて欲しかったんだと思うけど。

この回答へのお礼

ありがとうございます！もう一度考え直します。

お礼日時：2021/03/31 16:43