合計点が表示されない

Question

以下のコードでSumScoreを実行した時に「====== 合 計 点 一 覧 =====」の下に以下の形式で名前と各教科の点数の合計を表示させたいのですが、名前と点数入力後、何も表示されません。

---

名前:◯◯◯◯, 合計:△ 点

実行例

何人分の成績を入力しますか(数字で入力): 2
1人目の名前: qqq
国語: a
得点を入力してください。
国語: a
得点を入力してください。
国語: 1
算数: 1
理科: 1
社会: 1
2人目の名前: w
国語: 1
算数: 1
理科: 1
社会: 1

====== 合 計 点 一 覧 =====
[ここに表示させたいが何も出ません。]

色々試してみたのですが、うまく出力されないので知恵をお貸しください。。
よろしくお願いします

import java.util.HashMap;

/**
 * 個人の成績を格納し、合計点を計算する機能を提供する。
 *
 * @author 
 */
public class Person {

    //デフォルトコンストラクターの宣言
    public Person() {
    }

    /** 氏名 */
    String name;

    /** 成績を格納(科目名, 点数) */
    HashMap<String, Integer> recordMap;

    /**
     * 引数に与えられた名前をもとに、新しいPersonオブジェクトを生成する。
     * @param name 名前
     */
    public Person(String  name) {
        this.name = name;
    }

    /**
     * 合計点を計算する。
     * @return 合計点
     */
    public int calcSum() {
        int kokugo = (recordMap.get("国語")).intValue();
        int sansu  = (recordMap.get("算数")).intValue();
        int rika   = (recordMap.get("理科")).intValue();
        int syakai = (recordMap.get("社会")).intValue();
	int sum = kokugo + sansu + rika + syakai;
        return sum;
    }
}

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.HashMap;

/**
 * 氏名と各科目の点数を入力させ、合計値を表示する。
 *
 * 
 */
public class SumScore {

    /** 新しいSumScoreオブジェクトを生成する。 */
    public SumScore() {
    }

    /**
     * mainメソッド
     * @param args コマンドライン引数(今回は使用しない)
     */
    public static void main(String[] args) {

        try (InputStreamReader inStream = new InputStreamReader(System.in);
            BufferedReader bufReader = new BufferedReader(inStream)) {

            int kazu = 0;
            System.out.print("何人分の成績を入力しますか(数字で入力): ");
            try {
                String inParam = bufReader.readLine();
                kazu = Integer.parseInt(inParam);
                if (kazu <= 0) {
                    System.out.println("人数は1以上にしてください。");
                    System.exit(1);
                }
            } catch (NumberFormatException e) {
                System.out.println("人数は整数で指定してください。");
                System.exit(1);
            }

            // 人数分だけPerson配列を生成する。
            Person[] person = new Person[kazu];
            // 人数分だけ点数を入力させる。
            for (int i = 0; i < kazu; i++) {
                System.out.print((i + 1) + "人目の名前: ");
		
                String inName = bufReader.readLine();
                while (true) {
                    if (!inName.isEmpty()) break;
                    System.out.println("名前を入力してください。");
                }

                // 名前を引数にPersonオブジェクトを生成する。
                person[i] = new Person(inName);
                // 成績を格納するHashMapを生成する。
                HashMap<String, Integer> recordMap = new HashMap<String, Integer>(4);

                // 国語の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("国語", getScore("国語", bufReader));

                // 算数の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("算数", getScore("算数", bufReader));

                // 理科の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("理科", getScore("理科", bufReader));

                // 社会の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("理科", getScore("社会", bufReader));

                // recordMap をPersonオブジェクトに代入する。
                person[i].recordMap = recordMap;
	}
            // 合計点を計算し出力する。
            System.out.println();
            System.out.println("====== 合 計 点 一 覧 =====");
            for (int i = 0; i < kazu; i++) {
                int sum = person[i].calcSum();
                System.out.println("名前：" + person[i].name + ", 合計：" + sum + "点");
            }
        } catch (Exception e) { }
    }

    static Integer getScore(String subject, BufferedReader bufReader) {
        while (true) {
            try {
                System.out.print(subject + ": ");
                String line = bufReader.readLine();
                if (!line.isEmpty()) {
                    try {
                        Integer score = Integer.valueOf(line);
                       	if (score >= 0 && score <= 100) return score;
                       	System.out.println("得点は 0~100点の範囲で入力してください。");

                    } catch (NumberFormatException e) { }                }
            } catch (Exception e) { }
            System.out.println("得点を入力してください。");
        }
    }
}

Comments

[y_waiwai]

うまく出なくて、どういうふうに出てくるのでしょうか

[icecleam]

ご質問ありがとうございます。
名前と合計点の部分が出力されないです。

Answer 1

              recordMap.put("理科", getScore("社会", bufReader));

社会を入力するつもりが、理科を上書きし、社会の点数は未登録のままになるので、

        int syakai = (recordMap.get("社会")).intValue();

社会の点数を取得し、intに変換する段階でNullPointerExceptionになります。

Answer 2

try catch のcatch部で例外をcatchしていますが、例外処理がないので例外が発生していてもわかりません。
その部分を修正すると、合計点一覧を表示しようとした際に以下のように例外が発生しています。

====== 合 計 点 一 覧 =====
java.lang.NullPointerException
        at Person.calcSum(Person.java:36)
        at SumScore.main(SumScore.java:76)

これは、SumScore内で、社会の成績を聞いているにもかかわらず、理科の項目にセットしているのが原因です。

修正したSumScore.javaは以下の通りです。修正箇所は ★ を入れておきました

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.HashMap;

/**
 * 氏名と各科目の点数を入力させ、合計値を表示する。
 *
 * 
 */
public class SumScore {

    /** 新しいSumScoreオブジェクトを生成する。 */
    public SumScore() {
    }

    /**
     * mainメソッド
     * @param args コマンドライン引数(今回は使用しない)
     */
    public static void main(String[] args) {

        try (InputStreamReader inStream = new InputStreamReader(System.in);
            BufferedReader bufReader = new BufferedReader(inStream)) {

            int kazu = 0;
            System.out.print("何人分の成績を入力しますか(数字で入力): ");
            try {
                String inParam = bufReader.readLine();
                kazu = Integer.parseInt(inParam);
                if (kazu <= 0) {
                    System.out.println("人数は1以上にしてください。");
                    System.exit(1);
                }
            } catch (NumberFormatException e) {
                System.out.println("人数は整数で指定してください。");
                System.exit(1);
            }

            // 人数分だけPerson配列を生成する。
            Person[] person = new Person[kazu];
            // 人数分だけ点数を入力させる。
            for (int i = 0; i < kazu; i++) {
                System.out.print((i + 1) + "人目の名前: ");

                String inName = bufReader.readLine();
                while (true) {
                    if (!inName.isEmpty()) break;
                    System.out.println("名前を入力してください。");
                }

                // 名前を引数にPersonオブジェクトを生成する。
                person[i] = new Person(inName);
                // 成績を格納するHashMapを生成する。
                HashMap<String, Integer> recordMap = new HashMap<String, Integer>(4);

                // 国語の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("国語", getScore("国語", bufReader));

                // 算数の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("算数", getScore("算数", bufReader));

                // 理科の点数を入力させ、recordMapに点数を格納する。
                recordMap.put("理科", getScore("理科", bufReader));

                // 社会の点数を入力させ、recordMapに点数を格納する。
                // recordMap.put("理科", getScore("社会", bufReader));
                recordMap.put("社会", getScore("社会", bufReader)); // ★社会を入力しているのに理科になっている

                // recordMap をPersonオブジェクトに代入する。
                person[i].recordMap = recordMap;
            }

            // 合計点を計算し出力する。
            System.out.println();
            System.out.println("====== 合 計 点 一 覧 =====");
            for (int i = 0; i < kazu; i++) {
                int sum = person[i].calcSum();
                System.out.println("名前：" + person[i].name + ", 合計：" + sum + "点");
            }

        } catch (Exception e) {
            e.printStackTrace();  // ★例外を握りつぶさないように
        }
    }

    static Integer getScore(String subject, BufferedReader bufReader) {
        while (true) {
            try {
                System.out.print(subject + ": ");
                String line = bufReader.readLine();
                if (!line.isEmpty()) {
                    try {
                        Integer score = Integer.valueOf(line);
                           if (score >= 0 && score <= 100) return score;
                           System.out.println("得点は 0~100点の範囲で入力してください。");

                    } catch (NumberFormatException e) { }                }
            } catch (Exception e) { 
                e.printStackTrace();  // ★例外を握りつぶさないように
            }
            System.out.println("得点を入力してください。");
        }
    }
}

Answer 3

何も表示されない場合、処理としてどうなってるか考えてみると良いです。

            System.out.println("====== 合 計 点 一 覧 =====");
            for (int i = 0; i < kazu; i++) {
                int sum = person[i].calcSum();
                System.out.println("名前：" + person[i].name + ", 合計：" + sum + "点");
            }
        } catch (Exception e) { }

System.out.println("====== 合 計 点 一 覧 =====");が通っているのは確認できていると思うので
forの中のsysoutを取っていないことがわかります。

じゃあ、どこを通っているのか？
kazuが0ならfor文を通らない可能性があるが、0の場合は処理をすぐに中断しているので、
その可能性は違う。

では、どこだ？
ここです。
} catch (Exception e) { }

例外を握りつぶしてはいけない。の良い見本だと思いました。

        } catch (Exception e) { e.printStackTrace(); }

と、すればすぐに解決できると思います。
※わからなければ、質問を修正(エラーメッセージの追記など)してもらえればと思います。

Comments

[icecleam]

ご回答ありがとうございます。
上記のコードでうまく実行できました。

例外の処理部分がよくわかっていなかったみたいです。。
ありがとうございました！