f多次元配列の１行ごとの合計の出し方・成績表を作りたい

Question

前提・実現したいこと

２次元配列を使った成績表を作っています。
完成イメージはこのように名前と平均成績の表を出したいです。

発生している問題・エラーメッセージ

エラーコードは出ていないのですが、何も表示さないです。

該当のソースコード

package jp.learningdesign.javastudy;

	 class Task2_Lab5 {
		
		public static void main(String[] args) {
	//1. input
	    String[][] names = {
	        {"Ewell ", "Nate"},
	        {"Smurthwaite ", "Adelina"},
	        {"Bernollet ", "Maryjane"},
	        {"Chalfain ", "Karlis"},
	        {"Huckabe ", "Almire"}
	    };
	    double[][] grades = {
	            {3.83, 9.21, 9.25, 0.84, 1.40, 7.90, 1.41, 5.15},
	            {5.09, 3.66, 3.04, 3.02, 8.59, 2.34, 8.86, 7.40},
	            {5.35, 0.85, 8.56, 7.37, 2.78, 3.19, 2.04, 5.70},
	            {9.25, 9.62, 7.08, 7.27, 9.86, 5.88, 9.86, 9.70},
	            {3.39, 2.75, 1.23, 1.04, 3.87, 8.72, 4.60, 6.21}
	        };
	    
	    }
	    public double classAverage(double[][] grades, double classAverage) {

	        // avg of per line
	        double[] personAvg = new double[grades.length];
	        for (int i = 0; i < grades.length; i++) {
	            double Sum = 0;
	            for (int j = 0; j < grades[i].length; j++) {

	                double classSum = 0;
	                double classAvg = 0;
	            }
	            
	            double classAvg = ((Sum / grades.length) / grades[0].length);

	            personAvg[i] = Sum / grades[i].length;// １行の平均
	            System.out.println("each students average " + personAvg[i]);
	            System.out.println("class average " + classAvg);

	        }
	        return classAverage;
	    }
	    
	    public void  printNames(String[][] names, double[] personAvg) {
	        String students = " ";
	        for (int i = 0; i < names.length; i++) {
	            for (int j = names[i].length - 1; j >= 0; j--) {
	                students += " " + names[i][j];
	            }
	            students += "\t" + personAvg[i];
	        }
	            System.out.print(students + "\t\n");
	            students = "";
	        
	        
	    }
	}

試したこと

classAverage(grades);
printNames(names) ;
や
classAverage();
printNames() ;
をメインにおきましたがエラーが出ます。

The method classAverage(double[][], double) in the type Task2_Lab5 is not applicable for the arguments (double[]
 [])

Comments

[yambejp]

javaかなにか？使用プログラム言語のタグ付けしてください

[Gingerale09]

失礼しました。Javaです。

Answer 1

Java8以降を使っている前提ですが、この手の配列の処理は
streamを使用すると簡単に算出する事ができます。

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;

public class Test {

  public static void main(String[] args) {
    String[][] names = {
        {"Ewell ", "Nate"},
        {"Smurthwaite ", "Adelina"},
        {"Bernollet ", "Maryjane"},
        {"Chalfain ", "Karlis"},
        {"Huckabe ", "Almire"}

    };

    double[][] grades = {
        {3.83, 9.21, 9.25, 0.84, 1.40, 7.90, 1.41, 5.15},
        {5.09, 3.66, 3.04, 3.02, 8.59, 2.34, 8.86, 7.40},
        {5.35, 0.85, 8.56, 7.37, 2.78, 3.19, 2.04, 5.70},
        {9.25, 9.62, 7.08, 7.27, 9.86, 5.88, 9.86, 9.70},
        {3.39, 2.75, 1.23, 1.04, 3.87, 8.72, 4.60, 6.21}
    };

    var obj = new Test();
    List<Double> averageList = new ArrayList<>();
    for (int i = 0; i < grades.length; i++) {
      double[] grade = grades[i];
      double average = obj.average(grade);
      // 1行毎の平均値を保存
      averageList.add(average);
      System.out.println(obj.round(average));
    }
    // 1行毎の平均からクラス全体の平均を算出
    var average = obj.average(averageList);
    System.out.println(obj.round(average));

    // 2次元配列から一括でクラス平均を算出
    var classAverage = obj.classAverage(grades);
    System.out.println(obj.round(classAverage));
  }

  /**
   * 平均値を算出する.
   *
   * @param grade 成績
   * @return 平均値
   */
  public double average(double[] grade) {
    return Arrays.stream(grade).average().getAsDouble();
  }

  /**
   * 平均値を算出する.
   *
   * @param grade 成績
   * @return 平均値
   */
  public double average(List<Double> grade) {
    return grade.stream()
        .mapToDouble(Double::doubleValue)
        .average()
        .getAsDouble();
  }

  /**
   * 2次元配列の配列要素毎の平均値を算出する.
   *
   * @param grades 成績
   * @return 平均値
   */
  public double classAverage(double[][] grades) {
    return Arrays.stream(grades)
        .map(grade -> this.average(grade))// 配列要素毎に平均値を計算[4.87 , 5.25 , 4.48 , 8.57, 3.98]
        .mapToDouble(Double::doubleValue) // 型変換
        .average() // 平均値の平均を計算
        .getAsDouble();
  }

  /**
   * 小数点第3位を四捨五入した文字列を返す.
   *
   * @param value double値
   * @return 文字列
   */
  public String round(double value) {
    return String.format("%.2f", value);
  }
}

Comments

[Gingerale09]

すみません。streamについて調べたところstreamを使うときれいにコードが表示できるようなのですが、私、勉強を初めたばかりでして、streamの使い方が全く分からないです。

Answer 2

＞ １行ごとの合計の出し方につまずいている

１行ごとの合計を保存する変数を用意します。
gpaや、classSumと同様に、その変数に対して合算します。
必要に応じて、平均を出すためにlengthで割ります。

途中ぬかして上記だけ書くと、こんな感じ。

for (int i = 0; i < grades.length; i++) {
	double personSum = 0.0;	// 1行の合計保存用
	for (int j = 0; j < grades[i].length; j++) {
		personSum += grades[i][j];	// 合計する
	}
	// ループを抜けた（１行の合計が終わった）あとに平均を求める
	double personAvg = personSum / grades[i].length;  // １行の平均
}

Comments

[momon-ga]

そいや、クラス平均のだしかた間違ってない？
つど割ってると、めっちゃ小さな値になりそうだけど・・・

[Gingerale09]

ありがとうございます。
double personSum = 0.0;は j=0のfor の前に置くから機能するのですか？
結果は5.3400000000003 みたいになっています。。。

[momon-ga]

機能するとは？
変数を参照する前に定義すればよいですし、（j=0のforの前）
１行分ごとに０に初期化しないと総合計(classSum？)と同じになります。
※定義する位置をいろいろ変えてみれば、挙動が理解できるかと思います

> 結果は5.3400000000003 みたいになっています。。。
２進数で小数部を扱うと、そんな感じになります。

[Gingerale09]

私はfor ループの前にdouble sum=0; を置いていたため、２行目からが前行の値がどんどん足されていく状態になっていました。理解できました。ありがとうございます。

[momon-ga]

なるほど。ありがちですね。
i=0のforループの前に書いた場合は、j=0のforループの前で
double sum=0;、でなく sum=0; とすれば、同様の結果になります。（コンパイルエラーにならない）
不必要に変数スコープを大きくしないように、ループの中で定義するのが良い場合が多いです。