配列の平均値を出すメソッドの作成 JAVA

Question

//* クラスの出席番号順にString型配列nameに名前が、
// * int型配列scoreにテストの点数が格納されています。
// *
// * 出席番号１ 大悟 74点
// * 出席番号２ 琴音 70点
// * 出席番号３ 勇輝 88点
// * ・
// *
// * このクラスの平均点を算出するメソッドを作成。

public class MethodAdd07 {
public static void main(String[] args) {
String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};

	 int[] score = {74,70,88,82,96,64,80,61,72,79};

↓ここから自分
★double ave =average(74,70,88,82,96,64,80,61,72,79);★

	System.out.println("平均は" + ave + "です");

}

★private static double average(int i, int j, int k, int l, int m, int n, int o, int p, int q, int r){★

	★int ave = (i + j + k + l + m + n + o + p + q + r)/ 10 ;★

	return ave;

}
}

それぞれの★で囲んだ3行を上手く最適化したいのですが、
double ave =average(score[]);
これを用いて効率化する方法をどなたか教えて頂けませんか？

Answer 1

プログラマー歴1年でjavaをほとんどもうかいてませんが
なんとなくjavaを書きたくなったのでためしに最適化しました。
もっといい方法があるかもしれません。

public class Main {

	public static void main(String[] args)
	{
		String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};
		int[] score = {74,70,88,82,96,64,80,61,72,79};

		System.out.println("平均は" + average(score) + "です");
	}

	private static double average(int[] score)
	{
               
		double ave = 0.0;

		for(int i:score)
		{
			ave += i;
		}

		return ave / score.length;
	}
}

学校の課題だと思いますががんばってくださいね。

4/09
コメントでお二人に指摘していただいたところを改善してみました。
私も勉強になりました。
あと変数の名前をaveとしていましたが
よく考えたらsumのほうが適切でしたOTL

改善したコード

public class Main {

	public static void main(String[] args)
	{
		String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};
		int[] score = {74,70,88,82,96,64,80,61,72,79};

		System.out.println("平均は" + average(score) + "です");
	}

	private static double average(int... score)//修正箇所１
	{
			double sum = 0.0;

			for(int i:score)
			{
				sum += i;
			}

			return score.length != 0 ? sum / score.length : sum ;//修正箇所２

	}
}

Comments

[R.lawliet]

正にこういう無駄の無いコードを求めていました！
ありがとう御座います。
初めて1週間の見習いです。

もうしばらく書いていないとの事ですが、こんなにスラスラ出てくるのですね！
はやくそのレベルに達したいものです。頑張ります！！

[katoy]

> ave / score.length

score.length が 0 の場合の対処をしておいたほうが良いかもしれません。

score = []
average(score)
として、よばれる可能性があるからです。(対象の学生が 0 人というケース)

Answer 2

標準APIで平均を求めるものがあるので追求するとそこに行き着いてしまいます。

int[] score = {74,70,88,82,96,64,80,61,72,79};
double result = IntStream.of(score).average().getAsDouble();
System.out.println(result);

Java8 のStream APIというもので、マルチコアCPUでの処理効率がよい点もポイントです。
しかし、こうなると自分でプログラムを考えるというよりもAPIを知っているかどうかという話になってしまって趣旨が違ってきてしまいますね。

Comments

[R.lawliet]

知識として新たな視野が開けました。ありがとうございます♪

Answer 3

基本的には先の方の回答でほぼ全てですが，より汎用的にするには，

    private static double average(int[] score)

この部分を，

    private static double average(int... score)

こう書く方法もあります．(可変長引数といいます)
こうすることで，引数にint配列だけでなく，

    average(74, 70, 88, 82, 96, 64, 80, 61, 72, 79)

直接intを列挙することもできます．
scoreは内部的に配列として扱われるので，内部の処理を変更する必要はありません．

Comments

[R.lawliet]

これは今後も大活躍しそうなコードですね！
知識が増えました、ありがとうございます♪