javaのループで配列に入力した数値を格納して計算していきたいのですが上手くいきません

Question

入力した数値を配列に格納して総合値や平均値を求めていきたいです
最後にsubjectごとの平均を求めたいのですが求めたい数値と異なった値が出てきますので教えていただきたいです

public class Sample1 {
     public static void main(String[] args) {
         final int MEMBER = 4;
         final int SUBJECT = 3;
         int[] total = new int[MEMBER];
         int[][] score = new int[MEMBER][SUBJECT];
         int avg = 0;
         int ave = 0;
         int ave2 = 0;
         int ave1 = 0;
         int[] point = new int[5];
         for (int i = 0; i < MEMBER; i++) {
             System.out.println("****Member" + (i + 1) + "****");
             for (int j = 0; j < SUBJECT; j++) {

                 System.out.print("Input point of subject" + (j + 1) + ":");
                 score[i][j] = new java.util.Scanner(System.in).nextInt();
                 total[i] += score[i][j];
             }
         }
System.out.println("*Name****S1*****S2*****S3*****Total***");

         for (int i = 0; i < MEMBER; i++) {
             System.out.print("  ");
             System.out.print("Members" + (i + 1));
             for (int j = 0; j < SUBJECT; j++) {
                 System.out.print(" ");
                 System.out.print(score[i][j]);
                 point[0] += score[i][0];
                 point[1] += score[i][1];
                 point[2] += score[i][2];
                 System.out.print("  ");
             }
             avg = total[i] / SUBJECT;
             System.out.print(total[i]);
             System.out.print("  ");
             System.out.println(Math.ceil(avg));

         }
         ave = point[1] / MEMBER;
         ave1 = point[2] / MEMBER;
         ave2 = point[3] / MEMBER;
System.out.println("-----------------------------------------");
         System.out.print("AVG");
         System.out.print("  ");
         System.out.print(ave);
         System.out.print("  ");
         System.out.print(ave1);
         System.out.print("  ");
         System.out.print(ave2);
     }
}

Comments

[y_waiwai]

うまくいかないとはどういうふうにうまくいかないのか説明を追記しましょう
また、このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

Answer 1

point[0]、point[1]、point[2] に値を設定しているのに、
point[1]、point[2]、point[3] の値を使っています。

for (int j のループで、point[0]、point[1]、point[2] に値を設定して
いますが、ループの外の i のループで設定しないといけません。というよりも、
total を読み込み時に行っているのだから、point もそうすればよいでしょう。

数値を1個読むごとに Scannerオブジェクトを new していますが、
最初に 1回生成するだけにしましょう。

import java.util.Scanner;

public class Sample1 {
	public static void main(String[] args) {
		final int MEMBER = 4;
		final int SUBJECT = 3;
		int[][] score = new int[MEMBER][SUBJECT];
		int[] total = new int[MEMBER];
		int[] point = new int[SUBJECT];
		Scanner sc = new Scanner(System.in);

		for (int i = 0; i < MEMBER; i++) {
			System.out.println("****Member" + (i + 1) + "****");
			for (int j = 0; j < SUBJECT; j++) {
				System.out.print("Input point of subject" + (j + 1) + ":");
				score[i][j] = sc.nextInt();
				total[i] += score[i][j];
				point[j] += score[i][j];
			}
		}
		System.out.println("*Name****S1*****S2*****S3*****Total***");
		for (int i = 0; i < MEMBER; i++) {
			System.out.print("  Members" + (i + 1));
			for (int j = 0; j < SUBJECT; j++) {
				System.out.print(" " + score[i][j] + "  ");
			}
			double avg = (double)total[i] / SUBJECT;
			System.out.println(total[i] + "  " + avg);
		}
		double ave0 = (double)point[0] / MEMBER;
		double ave1 = (double)point[1] / MEMBER;
		double ave2 = (double)point[2] / MEMBER;
		System.out.println("-----------------------------------------");
		System.out.println("AVG  " + ave0 + "  " + ave1 + "  " + ave2);
	}
}

平均点は整数にしたほうがいいですか？

Comments

[okome88]

ありがとうございます
整数じゃなくても大丈夫です
TotalとAVGの平均値も出したいのですがどうしたらよろしいでしょうか？

[kazuma-s]

平均値は、平均を求めたい複数の値を足して、それを個数で割ることです。