C言語の平均値を求める問題

Question

4つの整数型の引数a, b, c, dを入力とした際、1以上の数を持つのみ引数を選び、その平均をdouble型で計算して戻り値とする関数を定義したい。
C言語プログラムのソースコードを完成させよ。ただし、すべての引数が
0以下の場合は「0」を返すこととする。　　　

これでコンパイルして実行しても、0が出力されてしまいます。
原因を教えていただきたいです。

#include<stdio.h>
int avr(int a,int b,int c,int d){
  scanf("%d %d %d %d",&a,&b,&c,&d);
  int sum=0,count=0,row[4] = {a,b,c,d};
  double mean;
  for(int i=0;i<4;i++){
    if(row[i] >= 1){
      sum+=row[i];
      count++;
  }
  if(count == 0){
    return count;
  }else{
    mean = (double)(sum / count);
    return mean;
  }
  }
}

int main(void){
  int x,y,z,w;
  printf("%.3f",avr(x,y,z,w));
  return 0;
}

Comments

[episteme]

0より大きく1未満の引数に対してどう扱うか明記されていません。

[cateye]

「double型で計算して戻り値とする」のに、intが戻り値なのは何故でしょう？
(double)(sum / count);は、(double)sum / count;なのでは？
・・・先の例では“整数除算の結果”をdoubleにしているだけ。
int avr(int a,int b,int c,int d)←引数は何処で設定？
scanf("%d %d %d %d",&a,&b,&c,&d);・・・は、題意とちがう・・・引数a, b, c, dを入力・・・main()でするべき。
if(row[i] >= 1){の終端は？

[cateye]

epistemeさん＜“4つの整数型の引数”とあるので「0より大きく1未満の引数」はないかと・・・

[pepperleaf]

> 「0より大きく1未満の引数」
scanf()で、%d で読んでるので 1未満はないかと。... いや、そもそも整数なので、0より大きく、1未満はありません。main()で、x, y, z, w と引数を与えているにも関わらず、関数内で、scanf()してのはなぜ? と色々と疑問は付きませんが。

[episteme]

あー、そかそか

Answer 1

printf("%.3f",avr(x,y,z,w));

avr関数の戻り値はintです。
ところが、ここではfloat変数の出力指定、となってます。
これではまともな数字が出力されません

整数を出力するように、フォーマット文字列を設定しましょう

Comments

[cateye]

「double型で計算して戻り値とする」とあるので、avr() の方をdoubleにするのでは？

Answer 2

for(int i=0;i<4;i++){

このループは終わりはどこでしょうか?
if(count == 0){ まで含んでます。その結果、 i=0 しか、実行されてません。
そして、 avr()の戻り値は、intですが、 printf()での出力指定は、%.3fなので、floatして出力しようとしています。 その結果(多分)、0なのでしょう。

Answer 3

皆様ありがとうございました。
解答を拝見し、5分ほど格闘した結果できました。

関数avrがint型になっていたことと、meanのdouble型への変換がおかしかったのだと思います。

#include<stdio.h>
double avr(int a,int b,int c,int d){
  int sum=0,count=0,row[4] = {a,b,c,d};
  double mean;
  for(int i=0;i<4;i++){
    if(row[i] >= 1){
      sum+=row[i];
      count++;
    }
  }
  if(count == 0){
    return count;
  }else{
    mean = (double)(sum) / count;
    return mean;
  }
}

int main(void){
  int a,b,c,d;
  scanf("%d %d %d %d",&a,&b,&c,&d);
  printf("%.3f",avr(a,b,c,d));
  return 0;
}