答えがおかしい

解決済

質問者：noname#153262
質問日時：2008/11/13 09:48
回答数：5件

配列と関数を使って教科ごとの平均点を求めるC言語のプログラムを
作りました。しかし、結果がおかしくいろいろ考えてみたのですが、
よくわからないので質問に出すことにしました。
作ったものは以下です。どうすればよいでしょうか？
#include <stdio.h>
#define NUMBER3

/*--- 要素数noの配列の平均値を求める ---*/
double ave_of(int vb[], int no)
{
int i, sum, ave;
for (i=1;i<no; i++)
sum+=vb[i];
ave=(double)sum/NUMBER;
return(ave);
}

int main(void)
{
int i;
int a[NUMBER];
int b[NUMBER];
int c[NUMBER];
double ave_a, ave_b, ave_c;

printf("%d人の点数を入力してください。\n",NUMBER);
for(i=0;i<NUMBER; i++){
printf("[%d番]理科：",i+1);scanf("%d",&a[i]);
printf("　　数学：");scanf("%d",&b[i]);
printf("　　社会：");scanf("%d",&c[i]);
}

ave_a=ave_of(a, NUMBER);
ave_b=ave_of(b, NUMBER);
ave_c=ave_of(c, NUMBER);

printf("【理科】平均点：%.1f\n", ave_a);
printf("【数学】平均点：%.1f\n", ave_b);
printf("【社会】平均点：%.1f\n", ave_c);

return(0);
}

結果は例えば45.2　というような感じで出したいです。

通報する

この質問への回答は締め切られました。

質問の本文を隠す

回答 (5件)

ベストアンサー優先
最新から表示
回答順に表示

No.4ベストアンサー

回答者： yama5140
回答日時：2008/11/13 11:43

＞しかし、結果がおかしく、いろいろ考えてみたのですが、よくわからないので質問に出すことにしました。

　・sum を初期化していない「警告」は出ませんでしたか。
　・「結果」が「小さい」、と気づきませんでしたか。
　・「結果」が、小数点以下は０固定ではなかったですか。
　
　等々、なぞなぞを解く感覚で考えてみるのも楽しいかと・・。

☆今回に限らず、この様（プログラムは動くが、結果がおかしい）な場合、
　１人目は１桁点、２人目は２桁点、３人目は３桁点などのように、いろいろ試行すると、気づくのが早いかも・・。

（前の２つは、他の回答者さんが答えています）

残る１つは、

☆平均の計算（右辺◆）は double なのに、int 変数に代入すると・・・。

　小数点以下は、省かれます。
　
　４５ ← ４５．２　で、４５.０が「返り値」となります。
+++++++++++++++++++++++++++++++++++++++++++++
修正してみました。

　double　ave_of( int vb[], int no )
　{
　　　int　i, sum = 0;　// 初期化
　　　double　ave;　　　// 関数の型に！

　　　for( i = 0; i < no; i++ )　sum += vb[i];　// 全員集計

　　　ave = (double)sum / NUMBER;// ◆

　　　return( ave );
　}