javaで1～10の乱数を5つ発生させ、合計と平均を出した後に平均よりも大きい値と小さい値を表示したい

Question

はじめまして。
javaで1～10の乱数を5つ発生させ、合計と平均を出した後に平均よりも大きい値と小さい値を表示したいのですが、
二つ以上表示できません。どうかご教示お願いします。

public static void main(String[] args) {

	int data[] = new int[5];
	int sum = 0;
	int avg = 0;
	int big = 0;
	int small = 0;
	
	for (int i = 0; i < data.length; i++) {
		data[i] = (int)(Math.random()*10)+1;
		System.out.print(data[i] + " ");
		
		sum += data[i];
		avg = sum / data.length;
		
		if(data[i] > avg) {
			big = data[i] ;			
			
		}else if(data[i] < avg) {
			small = data[i];
		}			
		
	}
	
	System.out.println();
	System.out.println();
	System.out.println("合計値； " + sum);
	System.out.println("平均値； " + avg);
	System.out.println();
	System.out.println("平均値より大きい数； " + big);
	System.out.println("平均値より小さい数； " + small);
	
}

}

表示される結果は、例えば
5 8 6 10 1

合計値； 30
平均値； 6

平均値より大きい数； 10
平均値より小さい数； 1

のようになり、平均値より大きいはずの8や小さいはずの5が出せません。
二つ以上の変数に代入するようなやり方が良いのでしょうか？わかりません。
よろしくお願いいたします。

Answer 1

良いも悪いも、一つの変数には一つの値しか入りません。
なぜ「乱数を5つ発生させ」たものを(sum や avg のような)"変数"では無く配列に入れているのかをお考えください。

別の考え方としまして、平均が求まった後にまた for 文で「平均より大きい/小さい値を探しながら表示する」というのもあるかと思います。

Comments

[blotors]

ご指摘の通り、平均が求まった後にまた for 文で平均より大きい/小さい値を探しながら表示するやり方で上手くいきました。ありがとうございます。big、smallという変数は必要ありませんでした。

Answer 2

２つ以上表示できないについては、すでに回答にありますが
変数１つには１つしか保存できないので、配列やList、文字列として結合するなどの仕組みが必要です。

その他指摘としては、現状このプログラムでは以下の問題があります。
・平均を出すタイミング適切でない。

乱数生成が１，２，３，４，５とされた場合
平均値の推移は

sum += data[i];
avg = sum / data.length;

とあるので、1/5、3/5．6/5、10/5、15/5 となります。
本来の平均値として正しいのは15/5じゃないでしょうか。

・平均の小数部が切り捨てられる
intをintで割ると小数部が切り捨てられます。
たとえば、1/2は、0.5でなく0になります。

Comments

[blotors]

ご回答ありがとうございます。小数点以下の部分は切り捨てられていますね。本来の平均値については現状の私のレベルではよくわかりません。いずれ理解できればと思います。