C言語、配列に関して

Question

C言語についての質問です。
当方初心者で、ここに質問するのも初めてなので
コード

以下コード＆問題↓

/* 実行結果例に習い、以下のプログラムを作りなさい。
（1）長さ5のint型の配列を作成する。
（2）（1）の各要素に、1から10までの数をキーボードから入力する。
（3）配列の内容をすべて表示する。
（4）配列の値の合計値と、平均値を表示する。
（5）平均値よりも大きい数を表示する。
（6）平均値よりも小さい数を表示する。 */

#include<stdio.h>
int main()
{
	int i, j, a[5], b[5], c[5], big = 0, small = 0, sum = 0;
	double ave;
	printf("5つの数を入力してください。\n");
	
	//数値入力
	for(i=0; i<5; i++){
		scanf("%d", &a[i]);
	}
	
	//配列表示
	for(i=0; i<5; i++){
		printf("%d ", a[i]);
	}
	printf("\n");
	
	//合計値
	for(i=0; i<5; i++){
		sum = sum + a[i];
	}
	printf("合計値＝%d\n", sum);
	
	//平均値
	ave = sum / 5;
	printf("平均値＝%.f\n", ave);
	
	//平均値と比較(大)
	for(i=0; i<5; i++){
		if(a[i]>ave){
			b[i] = a[i];
			big = big + 1;
		}
	}
	
	//平均値と比較(小)
	for(j=0; j<5; j++){
		if(a[j]<ave){
			c[j] = a[j];
			small = small + 1;
		}
	}
	
	//結果表示
	printf("平均値より大きい数＝");
	for(i=0; i<big; i++){
		printf("%d ", a[i]);
	}
	printf("\n");
	
	printf("平均値より小さい数＝");
	for(j=0; j<small; j++){
		printf("%d ", c[j]);
	}
	printf("\n");
	return 0;
}

ここまで


これで配列の数字を1,2,3,4,5にして実行してみると次のようになります。

＞mondai1
5つの数を入力してください。
1
2
3
4
5
1 2 3 4 5
合計値＝15
平均値＝3
平均値より大きい数＝1 2
平均値より小さい数＝1 2


平均値より小さい数は、ちゃんと「1と2」になるんですが、
平均値より大きい数も「1と2」になってしまいます。


どうしたらいいのでしょうか。

Answer 1

平均より大きい数が格納されているのはbなのに、aの中身を出力しているからかと。
また、代入部分は次のように書き換えた方が良いでしょうね。

//平均値と比較(大)
for(i=0; i<5; i++){
    if(a[i]>ave){
        b[big] = a[i];
        big = big + 1;
    }
}

小さいほうに関しても同様です。

コードの書き方について

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

Comments

[退会済みユーザー]

本当ですね...
確認不足でした、回答ありがとうございました！

アドバイスまでありがとうございます！
無事に動きました。

Answer 2

こんにちは、きちんと思った通りの動きができたらならそれでいいのですが、
気になった点があるのでそれを述べたいと思います。

1.5という配列の要素数は記号定数にすべきです。

#define MAX_NUMBER 5

2.入力はscanfよりfgetsをおすすめめします...

3.平均を求めるとき 合計値を5で割っています。小数点以下切り捨てならかまいませんが、
小数点以下も表示したいなら(おそらく変数aveがdoubleなので)

ave = sum / 5.0;

という形で書きましょう。(今回私は記号定数にしたMAX_NUMBERをdoubleにキャストしています。)

最後に、

（5）平均値よりも大きい数を表示する。

（6）平均値よりも小さい数を表示する。

とありますが,表示だけなら新たな配列に格納する必要がありません.
(あくまでも、表示だけならの話です。そのあとに他の処理が必要なら別の話となります。)

以下、私が書いたソースコードです。

#include <stdio.h>
#include <stdlib.h>

#define BUFSIZE 256
#define MAX_NUMBER 5

int main(void){
	
	char buf[BUFSIZE];
	int num[MAX_NUMBER];
	int sum = 0;
	double average;
	int i;
	printf("5つの数を入力してください。\n"); 
	
	for(i = 0; i < MAX_NUMBER; i++){
		fgets(buf,sizeof(buf),stdin);
		num[i] = atoi(buf);
	}
	
	for(i = 0; i < MAX_NUMBER; i++){
		sum += num[i];
	}
	printf("合計値 = %d\n", sum);
	
	average = sum / (double)MAX_NUMBER;
	printf("平均値 = %.1f\n",average);
	
	printf("平均値より大きい数 = ");
	for(i = 0; i < MAX_NUMBER; i++){
		if(num[i] > average){
			printf("%d ",num[i]);
		}
	}
	printf("平均値より小さい数 = ");
	for(i = 0; i < MAX_NUMBER; i++){
		if(num[i] < average){
			printf("%d ",num[i]);
		}
	}
	
	return 0;
}