答えがおかしい

配列と関数を使って教科ごとの平均点を求める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　というような感じで出したいです。

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 );
　}

この回答へのお礼

回答ありがとうございます。
詳しい説明でとても分かりやすかったです。
結果が小数点以下が0だったのは気になっていました。sumを初期化
してなくてもエラーは出ませんでした。

お礼日時：2008/11/13 19:32

No.5 回答者： chie65536 回答日時： 2008/11/13 13:10

int i, sum, ave;

↓
int i, sum = 0;
double ave;

for (i=1;i<no; i++)
↓
for (i=0;i<no; i++)

ave=(double)sum/NUMBER;
↓
ave=(double)sum/(double)no;

No.3 回答者： okg00 回答日時： 2008/11/13 09:57

平均を求める際には合計を求めていますが、合計が初期化されていません。

足しこむ場合は0で初期化してください。

配列の添字は0で始まりますので、for文は0から始めてください。

No.2 回答者： postal0x02 回答日時： 2008/11/13 09:56

原因１

for (i=1;i<no; i++)
sum+=vb[i];

原因２
int i, sum, ave;

プログラムを殆ど見てないが、ここだと思う。

No.1 回答者： himajin100000 回答日時： 2008/11/13 09:54

コンパイルできる環境がなくて検証してないけど

/* これって本当に i = 1かな？i=0じゃなくて*/
for (i=1;i<no; i++)