C言語のポインタ配列について

質問

ポインタ配列を使って、各組(横一列が1組)の最高点と平均点、学年平均を表示するプログラムを考え中です。
各組の最高点を求める部分をどのように書けばいいのか迷ってます。

下記のプログラムでマズイ部分などありましたら教えていだたきたいです。

#include <stdio.h>

int main(void)
{
　　int ten1[] = { 52, 38, 79, 89, 57, 71, 90, -999};
　　int ten2[] = { 67, 89, 56, 45, 91, -999};
　　int ten3[] = { 89, 34, 67, 65, 67, 55, -999};
　　int* p[3];
　　int k;
　　int i;
　　int sum;
　　int total;
　　float ave;
　 int nin = 18;

　　p[0] = ten1;
　　p[1] = ten2;
　　p[2] = ten3;

　　total = 0;
　　for (k = 0; k < 3; k++)
　　{
　　　　i = 0;
　　　　sum = 0;
　　　　while (p[k][i] != -1)
　　　　{
　　　　　　sum += p[k][i];
　　　　　　i++;
　　　　}
　　　　ave = ave = (float)total / 3;
　　　　total += sum;
　　　　printf("平均点%3.1f\n", ave);
　　}
　　ave = (float)total / nin;
　　printf("学年平均点：%3.1f\n",ave);

　　return 0;
}

回答No.5

asuncion

おっと失礼。最高点を求めるロジックが抜けていました。

#include <stdio.h>

int main(void)
{
int ten1[] = { 52, 38, 79, 89, 57, 71, 90, -999};
int ten2[] = { 67, 89, 56, 45, 91, -999};
int ten3[] = { 89, 34, 67, 65, 67, 55, -999};
int *p[3], k, i, sum, total, nin, max;

p[0] = ten1, p[1] = ten2, p[2] = ten3;
for (total = nin = k = 0; k < 3; k++) {
for (sum = i = 0, max = p[k][0]; p[k][i] != -999; i++, nin++) {
if (p[k][i] > max) {
max = p[k][i];
}
sum += p[k][i];
total += p[k][i];
}
printf("最高点：%d, 平均点：%3.1f\n", max, (double) sum / i);
}
printf("学年平均点：%3.1f\n", (double) total / nin);
return 0;
}

回答No.4

asuncion

こういうことでしょうか。

#include <stdio.h>

int main(void)
{
int ten1[] = { 52, 38, 79, 89, 57, 71, 90, -999};
int ten2[] = { 67, 89, 56, 45, 91, -999};
int ten3[] = { 89, 34, 67, 65, 67, 55, -999};
int *p[3], k, i, sum, total, nin;

p[0] = ten1, p[1] = ten2, p[2] = ten3;
for (total = nin = k = 0; k < 3; k++) {
for (sum = i = 0; p[k][i] != -999; i++, nin++) {
sum += p[k][i];
total += p[k][i];
}
printf("平均点%3.1f\n", (double) sum / i);
}
printf("学年平均点：%3.1f\n", (double) total / nin);
return 0;
}

回答No.3

asuncion

変数totalを初期化するタイミングと加算するタイミングに
問題がありそうです。

回答No.2

Wr5

> 下記のプログラムでマズイ部分などありましたら教えていだたきたいです。

バッファオーバーランします。
>　　int ten1[] = { 52, 38, 79, 89, 57, 71, 90, -999};
>　　int ten2[] = { 67, 89, 56, 45, 91, -999};
>　　int ten3[] = { 89, 34, 67, 65, 67, 55, -999};
各配列の終端は「-999」です。
>　　　　while (p[k][i] != -1)
-1が見つかるまでループします。
ポインタが進んでいって、たまたま-1になった箇所を指すまで進みます。

>　　　　ave = ave = (float)total / 3;
この「3」はどこから？
# 「ave = ave = 」になっているのはただのコピペミスでしょう。

>　 int nin = 18;
>　　ave = (float)total / nin;
この18というのはどこから？
# ten1[]、ten2[]、ten3[]の有効データの数。
# と思われますが有効データ数が変わったら修正が必要になります。
# 合計点数の算出のループで件数が出てきますからそこから加算する方がよいでしょう。

回答No.1

koko_u_

>while (p[k][i] != -1)
いつまで while ループをまわすつもりですか？