結果の表示について

Question

課題で、
キーボードから入力された数値の平均を計算して表示し、平均以上の数値、平均より小さい数値を表示するプログラムを考えているのですが、
実行例

./a.out
数値(-1で入力終了)? 1
数値(-1で入力終了)? 2
数値(-1で入力終了)? 3
数値(-1で入力終了)? 4
数値(-1で入力終了)? 5
数値(-1で入力終了)? 6
数値(-1で入力終了)? 7
数値(-1で入力終了)? 8

数値(-1で入力終了)? 9
数値(-1で入力終了)? 10

数値(-1で入力終了)? 11
平均は 6.0 です。
平均以上の数値：6 7 8 9 10 11
平均より小さい数値：1 2 3 4 5

下の2行

平均以上の数値：6 7 8 9 10 11
平均より小さい数値：1 2 3 4 5

がどう書けばいいのか分かりません。
(※途中まで解決したので更新しました。ここまでアドバイスくださった
皆さんにお礼申し上げます。）
私のプログラムで実行すると、次のようになってしまい、すべて6.0となってしまいます。

./a.out
数値(-1で入力終了)?1
数値(-1で入力終了)?2
数値(-1で入力終了)?3
数値(-1で入力終了)?4
数値(-1で入力終了)?5
数値(-1で入力終了)?6
数値(-1で入力終了)?7
数値(-1で入力終了)?8
数値(-1で入力終了)?9
数値(-1で入力終了)?10
数値(-1で入力終了)?11
数値(-1で入力終了)?-1

平均は、6.0です。
平均以上の数値：6.0 6.0 6.0 6.0 6.0 6.0
平均以下の数値：6.0 6.0 6.0 6.0 6.0

私のプログラムです

#include <stdio.h>

int main(void)
{
  int val[100] = {0};
  int *p = val;
  int sum=0;
  int count = -1;
  int i;
  int j;
  do{
    printf("数値(-1で入力終了)?");
    scanf("%d",p);
    count++;
  }while(*p++ != -1);

  for(p=val;*p != -1 ;p++){
    sum += *p;
  }
  double ave= (double)sum/count;
  printf("\n");
  printf("平均は、%.1lfです。",ave);
  printf("\n");

  printf("平均以上の数値：");
  for(i = 0; i < count; i++ ){
    if( val[i] >= ave ){
      printf("%.1lf ", val[i]);
    }
  }
  printf("\n");
  printf("平均以下の数値：");
 
  for( i = 0; i < count; i++ ){
    if( val[i] < ave ){
      printf("%.1lf ", val[i]);
    }
  }
  printf("\n");
  

  return 0;
}

アドバイスよろしくお願いします。

Comments

[A.Ichi]

printf("%d ", val[i]); ではないでしょうか？

[cateye]

その通りです＾＾；ご指摘ありがとうございます。

Answer 1

#include <stdio.h>

int read_data(int*, int);
double calc_average(int*, int);
void show_result(int*, int, double);

#define MAX_SIZE (100)

int main(void)
{
  int vals[MAX_SIZE];
  int num = 0;

  num = read_data(vals, MAX_SIZE);
  show_result(vals, num, calc_average(vals, num));
}

int read_data(int vals[], int max_num) {
  int count;
  for (count = 0; count < max_num; count++) {
    int v;
    printf("数値(-1で入力終了)?");
    scanf("%d", &v);
    if (v == -1) {
      return count;
    }
    vals[count] = v;
  }
  return count;
}

double calc_average(int vals[], int num) {
  double sum = 0.0;
  for (int i = 0; i < num; i++) {
    sum += vals[i];
  }
  return sum /num;
}

void show_result(int vals[], int num, double average) {
  if (num == 0) {
    printf("データが 0 個です。\n");
    return;
  }
  printf("\n");
  printf("平均は、%.1lfです。\n", average);

  printf("平均以上の数値： ");
  for (int i = 0; i < num; i++) {
    int v = vals[i];
    if (v >= average) {
      printf("%d ", v);
    }
  }
  printf("\n");

  printf("平均未満の数値： ");
  for (int i = 0; i < num; i++) {
    int v = vals[i];
    if (v < average) {
      printf("%d ", v);
    }
  }
  printf("\n");
}

Answer 2

平均以下の数値

この箇所を未満に修正して簡単なプログラムを書いてみました。
よろしかったらご参考にして下さい。

#include <stdio.h>

int main(void)
{
	int val[100] = {0};
	int *p;
	int sum = 0;
	int count = 0;
	int i;
	int j;
	double ave = 0.0;
	p = val;

	printf("数値(-1で入力終了)?\n");
	scanf("%d", p);
	while(*p != -1){
		sum += *p;
		p++;
		count++;
		scanf("%d", p);
	}
	
	if (count == 0) {			//許可数値が1つも無い場合ここで終了する
		return 0;
	}
	
	ave = (double)sum / count;
	printf("\n");
	printf("平均は、%.1lfです。\n", ave);

	printf("平均以上の数値： ");
	for (i = 0; i < count; i++) {
		if (val[i] >= ave) {
			printf("%d ", val[i]);
		}
	}
	printf("\n");
	printf("平均未満の数値： ");
	for (i = 0; i < count; i++) {
		if (val[i] < ave) {
			printf("%d ", val[i]);
		}
	}
	printf("\n");
	
	return 0;
}

Comments

[bjbjeknr]

助かりました。ありがとうございます。おかげさまで解決できました.
感謝します。

Answer 3

ざっとこんな感じかなぁ？・・・検証はしていません。
※平均と同じ数値の場合はどうするんでしょう？

double mean= (double)sum/count;
printf("平均以上の数値：");
for( int i= 0; i < count; i++ ){
    if( val[i] >= mean ){
        printf("%d ", val[i]);
    }
}
fputc('\n',stdout);
printf("平均以下の数値：");
for( int i= 0; i < count; i++ ){
    if( val[i] < mean ){
        printf("%d ", val[i]);
    }
}
fputc('\n',stdout);

Comments

[bjbjeknr]

平均と同じ数値の場合は平均以上に含みます。すみません。。

[bjbjeknr]

助かりました。ありがとうございます。おかげさまで途中まで解決できましたので、質問を更新しました。

[cateye]

A.Ichiさんのご指摘により一部修正しました。

[bjbjeknr]

助かりました。ありがとうございます。おかげさまで解決できました.
感謝します。

Answer 4

配列にでも入れてifで判定すればいいかと

追記
ひとまず質問のコードを以下のように修正することで結果は表示できました。修正箇所はコメントで
１：valをdoubleにして平均値が例えば、4.5のようになっても判定できるように
２：出力は整数だから

  printf("平均以上の数値：");
  for(i = 0; i < count; i++ ){
    if( (double)val[i] >= ave ){   //１
      printf("%d ", val[i]);       //２
    }
  }
  printf("\n");
  printf("平均以下の数値：");

  for( i = 0; i < count; i++ ){
    if( (double)val[i] < ave ){    //1
      printf("%d ", val[i]);       //2
    }
  }

Comments

[bjbjeknr]

助かりました。アドバイスありがとうございます。おかげさまで途中まで解決できましたので、質問を更新しました。

[bjbjeknr]

助かりました。ありがとうございます。おかげさまで解決できました.
感謝します。