結果の出力が上手くいかない

Question

以下は、入力した数値の平均を表示するプログラムです。平均は切り捨てです。
まず初めに、数値の個数(n)を入力し、その後に数値を入力していきます。最後に0と入力すると終了します。
例えば、
4
1
2
3
4
0
と入力すると、
2と出力されます。一番上の4は数字の個数です。
ここまでは良いのですが、入力データセットが複数あると、前の入力データセットの平均がリセットされてしまいます。
例えば、
4
1
2
3
4
6
1
2
3
4
5
7
0
と入力した場合、一番上の4と6が数値の個数を表し、データセットの境目になっているのですが、結果は2つめのデータセット(123457)の方の平均しか出力されず、(1234)の平均は出力されません。おそらく上書きされているからだとは思いますが、どう書いたらよいか困っています。0を入力した後に2つの結果を表示するようにしたいです。教えていただけないでしょうか。よろしくお願いします。

import java.util.Scanner;
public class Average{
    public static void main(String[]args){
	Scanner sc = new Scanner(System.in);
	int y=0;
	while(sc.hasNext()){
	   int n=sc.nextInt();
	      if(n==0) {System.out.println(y);
		  break;}
	 	int[]a=new int[n];
		for(int i=0; i<n; i++) a[i]=sc.nextInt();
                int x=0;
		for(int i=0; i<n; i++){
		   x=x+a[i];
		}
	        y=x/n;
	       }
	   }
}

Answer 1

処理フローを考え直しましょう。

　　開始
　　↓
　　平均を入れる配列またはListを用意
　　↓
→→処理個数入力
↑　↓
↑　0以外か？ → No → 配列またはListの内容を出力して終了
↑　↓ Yes
↑　処理個数分入力して平均を求める
↑　↓
↑←求めた平均を配列やListに入れる

こうしないと各回の平均を出力できません。
現状最後に終了するときだけ出力を行っているため、最後の平均の結果しか出力できません。

Answer 2

それなりに改造しました。

import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Mean{
    public static void main(String[]args){
        Scanner sc = new Scanner(System.in);
        List<Double> means = new ArrayList<>();

        System.out.println("何個の整数を入力するか入れてね");
        while(sc.hasNext()) {
            int inputLen = sc.nextInt();
            if(inputLen == 0) {
                break;
            }

            int sum = 0;
            System.out.println(inputLen + "個入力してね");
            for(int i = 0; i < inputLen; i++) {
                sum += sc.nextInt();
            }
            means.add(1.0 * sum / inputLen);

            System.out.println("何個の整数を入力するか入れてね");
        }

        System.out.println(means);
    }
}

/* 出力
何個の整数を入力するか入れてね
4
4個入力してね
1
2
3
4
何個の整数を入力するか入れてね
6
6個入力してね
1
2
3
4
5
7
何個の整数を入力するか入れてね
0
[2.5, 3.6666666666666665]
*/

本質的な回答とはちょっと違いますので、参考程度に見てみてください。
ただし、インデントはちゃんと揃えましょう。

---

元のプログラムをある程度維持するとこんな感じ。

import java.util.Scanner;
public class Average{
    public static void main(String[]args){
        Scanner sc = new Scanner(System.in);
        int[] result = new int[100];

        int resultNum = 0;
       while(sc.hasNext()){
            int n=sc.nextInt();
            if(n==0) {break;}
            int[]a=new int[n];
            for(int i=0; i<n; i++) a[i]=sc.nextInt();
                    int x=0;
            for(int i=0; i<n; i++){
               x=x+a[i];
            }

                result[resultNum] = x/n;
               result++;
        }

        // 出力
        for(int j = 0; j < 100; j++) { if(result[j] == 0)
           break;
            System.out.println(result[j]);
       }
   }
}

インデントを意図的に崩し、わざと冗長な方法を用いてみました。

Comments

[prog1717nm]

ありがとうございました！

[LouiS0616]

過去の質問を拝見しました。厳しいことを言うようですが...
インデントをそろえる／エラーメッセージをよく読む／格納されている値を適宜確認する／...
このあたりのテクニックは、毎回の反省の中で学んで欲しいものです。
「とりあえず動けばいい」「エラーが出たら聞けばいい」この考えから脱却してください。

[LouiS0616]

あと、書いているコードが全体的に古すぎます。
生産性を上げるためにも、基礎を固めるためにも、新しめの入門書を通読することをお勧めします。

[swordone]

古いというより、無駄が多い

[LouiS0616]

そうですね。
非拡張for文と生配列ばっかりに目が向いてしまって、古いという表現を使ってしまいました。
新しさ以前の問題であることについてはその通りだと思います。