java キーボード入力を用いた計算プログラム

Question

##概要
キーボードから入力したテストの結果から、合計・平均・順位（バブルソート）分散・標準偏差を導出するプログラムを作っています。
具体的には
生徒数を入力
↓
生徒数の応じた点数を入力
↓
使いたい機能を番号で選択
↓
結果の表示
みたいな感じです。コンパイルのエラーは出ないのですが、結果の表示の部分で合計は０、平均は０のような羅列が無限に表示されます。入力した結果を配列の中に一時保存して、その配列を引数として別のクラスで計算してからメインクラスで表示したいのですが、結果がうまく配列化されず引数が０であることが原因だと考えています。色々な方からアドバイスを受けるのですが自力ではどうにもなりませんでした。お願いします。条件としてはクラスを二つ使うこと、キーボード入力をすることです。

  class Main {
    public static void main(String[] args) throws IOException{
    new Main();
}

    Main() {
    String ss=null;
    int t=0,j=0,ninzu=0;

    BufferedReader k=new BufferedReader(new InputStreamReader(System.in));
  	System.out.println("こんにちは！");
  	System.out.println("ここではテスト結果の合計・平均、順位、分散、標準偏差を算出できます。");
    System.out.println("はじめに生徒数を教えてね。");
    try{
    //生徒数の確認
    ss=k.readLine();
    ninzu=Integer.parseInt(ss);
  }catch(IOException e) {
        System.out.println("error");
      }

    System.out.println("人数分の点数を入力してね。終わったらendと入力して！");
    //入力した結果を配列化する
    try{
    for(int i=0;i<ninzu;i++){
        ss=k.readLine();
        int[] score=new int[ninzu];//この部分が機能していない？
        score[i]=Integer.parseInt(ss);
          }
      }catch(Exception e) {
            System.out.println("エラーが発生しました");
        }

    System.out.println("次にどの機能を使うのか教えてね。");
    System.out.println("合計・平均なら１、順位なら２、分散・標準偏差なら３と入力してね");
    //機能の選択
    try{
      ss=k.readLine();
      t=Integer.parseInt(ss);
    }catch(IOException e) {
          System.out.println("error");
      }

        while(true) {
            switch(t) {
                case 1:
                     System.out.println("合計は"+TestResult.getTotal(score)+","+"平均は"+TestResult.getAverage(score));
                     break;

                case 2:
                     System.out.println("順位は"+TestResult.getSortedResult(score));
                     break;

                case 3:
                     System.out.println("分散は"+TestResult.getVar(score)+","+"標準偏差は"+TestResult.getStddev(score));
                     break;
            }
        }
    }
}

class TestResult {
    private static int[] result;
    private static int[] sortedResult;

    private static int total;
    private static double ave, var, stddev;

    public TestResult() {
        this.result = result;

        total = computeTotal(result);
        ave = computeAverage(result);
        var = computeVar(result);
        stddev = computeStddev(result);
    }

    public static int getTotal() {
        return total;
    }
    public static double getAverage() {
        return ave;
    }
    public static double getVar(){
        return var;
    }
    public static double getStddev(){
        return stddev;
    }
    public static int[] getResult(){
        return result;
    }
    public static int[] getSortedResult(){
        return sortedResult;
    }

// Private Methods
    private static int computeTotal(int[] x) {
    for(int i=0;i<x.length;i++){
     total=total+x[i];
    }
    return total;
  }
    private static double computeAverage(int[] x) {
      for(int i=0;i<x.length;i++){
       total=total+x[i];
    }
    return (total/x.length);
}

    private static double computeVar(int[] x){
      for (int j=0; j<x.length; j++) {
          total = total+x[j];
    }
              double e=0;
             ave=(total/x.length);

             for(int j=0; j<x.length; j++) {
     				e= (ave-x[j])*(ave-x[j]);
     }
            var=(e/x.length);
            return var;
     }

    private static double computeStddev(int[] x){
      double e=0;
      for (int j=0; j<x.length; j++) {
      total = total+x[j];

    }
             ave=(total/x.length);
             for(int j=0; j<x.length; j++) {
     				e= (ave-x[j])*(ave-x[j]);
     }
            var=(e/x.length);
            stddev=Math.sqrt(var);
            return stddev;
    }
    private static int[] sortedResult(int[] x){
      int tmp=0;

      for(int j=x.length-1;j>0;j--){
        for(int l=0;l<j;l++){
          if(x[l]<x[l+1]){
            tmp=x[l];
            x[l]=x[l+1];
            x[l+1]=tmp;

        }
      }
    }
    return sortedResult;
  }
}

Answer 1

ともかく、それなりに動作(コンパイルエラーを取る、無限に出力がくりかえされないようにする) ようにしたものを示します。
（別回答として、全体をかきなおしたものを投稿する予定です)

teratail/Main.java

package teratail;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.Arrays;

class Main {
    public static void main(String[] args) throws IOException {
        new Main();
    }

    Main() {
        String ss = null;
        int t = 0, j = 0, ninzu = 0;

        BufferedReader k = new BufferedReader(new InputStreamReader(System.in));
        System.out.println("こんにちは！");
        System.out.println("ここではテスト結果の合計・平均、順位、分散、標準偏差を算出できます。");
        System.out.println("はじめに生徒数を教えてね。");
        try {
            // 生徒数の確認
            ss = k.readLine();
            ninzu = Integer.parseInt(ss);
        } catch (IOException e) {
            System.out.println("error");
        }

        System.out.println("人数分の点数を入力してね。終わったらendと入力して！");
        // 入力した結果を配列化する
        int[] score = new int[ninzu];// この部分が機能していない？
        try {
            for (int i = 0; i < ninzu; i++) {
                ss = k.readLine();
                score[i] = Integer.parseInt(ss);
            }
        } catch (Exception e) {
            System.out.println("エラーが発生しました");
        }

        TestResult testResult = new TestResult(score);
        while (true) {

            System.out.println("次にどの機能を使うのか教えてね。");
            System.out.println("合計・平均なら１、順位なら２、分散・標準偏差なら３と入力してね");
            // 機能の選択
            try {
                ss = k.readLine();
                t = Integer.parseInt(ss);
            } catch (IOException e) {
                System.out.println("error");
            }
            
            switch (t) {
            case 1:
                System.out.println("合計は" + testResult.getTotal() + "," + "平均は" + testResult.getAverage());
                break;

            case 2:
                System.out.println("順位は");
                System.out.println(Arrays.toString(testResult.getSortedResult()));
                break;

            case 3:
                System.out.println("分散は" + testResult.getVar() + "," + "標準偏差は" + testResult.getStddev());
                break;
            }
        }
    }
}

teratail/TestResult.java

package teratail;

class TestResult {
    private int[] result;
    private int[] sortedResult;

    private int total;
    private double ave, var, stddev;

    public TestResult(int[] result) {
        this.result = result;

        this.total = computeTotal(result);
        this.ave = computeAverage(result);
        this.var = computeVar(result);
        this.stddev = computeStddev(result);
        this.sortedResult = sortedResult(result.clone());
    }

    public int getTotal() {
        return this.total;
    }

    public double getAverage() {
        return this.ave;
    }

    public double getVar() {
        return this.var;
    }

    public double getStddev() {
        return this.stddev;
    }

    public int[] getResult() {
        return this.result;
    }

    public int[] getSortedResult() {
        return this.sortedResult;
    }

    // Private Methods
    private static int computeTotal(int[] x) {
        int total = 0;
        for (int i = 0; i < x.length; i++) {
            total = total + x[i];
        }
        return total;
    }

    private static double computeAverage(int[] x) {
        int total = computeTotal(x);
        return (total / x.length);
    }

    private double computeVar(int[] x) {
        double e = 0;
        double ave = computeAverage(x);
        for (int j = 0; j < x.length; j++) {
            e = (ave - x[j]) * (ave - x[j]);
        }
        double var = (e / x.length);
        return var;
    }

    private double computeStddev(int[] x) {
        double var = computeVar(x);
        double stddev = Math.sqrt(var);
        return stddev;
    }

    private static int[] sortedResult(int[] x) {
        int tmp = 0;

        for (int j = x.length - 1; j > 0; j--) {
            for (int l = 0; l < j; l++) {
                if (x[l] < x[l + 1]) {
                    tmp = x[l];
                    x[l] = x[l + 1];
                    x[l + 1] = tmp;

                }
            }
        }
        return x;
    }
}

実行例：

Comments

[kazuo-maru]

ありがとうございます。解決できました。