変数の定義が影響する範囲

Question

配列の要素数の平均を求める関数を作っています。以下のようなコードで変数sを導入し、for文中で合計を求め、このsを使ってfor文を抜けたところで平均を求めようとしたのですが、s=0と認識されてしまい、実行の結果平均は0と表示されてしまいます。この場合、変数の設定場所が間違っているのでしょうか。それとも、合計を求める変数と平均を求める変数で分けなければいけないのでしょうか。
よろしくお願い致します。

#include <stdio.h>
#define ARRAY_SIZE 10

double ave(double dt[], int n);

double ave(double dt[], int n) {

	int s = 0;

	for (int i = 0; i < n; i++) {
	
		s += dt[i];
	}
	return s / n;

}

int main (void)
{
	double dt[ARRAY_SIZE] = { 1 };

	for (int i = 0; i < ARRAY_SIZE; i++) {
		dt[i] = i;
	}

	for (int i = 0; i < ARRAY_SIZE; i++) {
		printf("%f\n", dt[i]);
	}

	printf("%f\n", ave(dt, ARRAY_SIZE));

	return 0;
}

Answer 1

dt にはなにがはいってるんでしょうか

  s += dt[i];

単にこの結果が０になってるだけなんでは。
doubleの積算をintでやってるというのがちと意味不明ですが、これは意図してやってることでしょうか

そもそも、

    printf("%d\n", dt[i]);

これアカンやんｗ

Comments

[h_proc]

すみません。intではなく、doubleでした。
dtに入る数値をmain関数で代入しようとしたのですが、、、

[y_waiwai]

dtに何が入ってるのかをまず確認しましょう

> printf("%d\n", dt[i]);
これ、ダメですよ。

[y_waiwai]

んで、
> int ave(double dt[], int n) {
これは修正しましたか？

[h_proc]

int ave(double dt[], int n)はdouble ave(double dt[],int n)に書き換えました。
printf("%d\n",dt[i])はdouble型の配列を定義していたため、printf("%f\n",dt[i])に書き換えました。このあたりがよく分かっておらず、自信がないのですが、これでよいのでしょうか。

[y_waiwai]

そんでいいとおもいます
そんでもまだ結果がおかしいと言うなら、質門を編集して、現状のソースを提示しましょう

[h_proc]

解決しました。ありがとうございます。表記の細かいところに注意していきたいと思います。

Answer 2

コンパイラの警告をよく読みましょう。

prog.c: In function 'main':
prog.c:30:24: warning: passing argument 1 of 'ave' from incompatible pointer type [-Wincompatible-pointer-types]
     printf("%f\n", ave(dt, ARRAY_SIZE));
                        ^~
prog.c:6:16: note: expected 'double *' but argument is of type 'int *'
 int ave(double dt[], int n) {
         ~~~~~~~^~~~
prog.c:30:14: warning: format '%f' expects argument of type 'double', but argument 2 has type 'int' [-Wformat=]
     printf("%f\n", ave(dt, ARRAY_SIZE));
             ~^     ~~~~~~~~~~~~~~~~~~~
             %d

そもそもave関数の戻り値がなんでdoubleではなくintなのでしょうか？

Comments

[h_proc]

すみません、私のミスです。double でした。しかし、double でやってもうまくいきませんでした。

[episteme]

値を積み上げてる変数 s も double に改めました?

[h_proc]

はい。doubleにしたら、上手くいきました。ありがとうございます。