フィールドで宣言したMapにアクセスしたい

Question

前提・実現したいこと

じゃんけんをするプログラムを書いています.下記のコードはその一部です.
実現したいことは, Janken.javaのstartGameメソッドで, フィールドのMapに保存した
データを, 同クラスのjudgeメソッドで利用することです.

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

実行すると、こう出ました.
Exception in thread "main" java.lang.NullPointerException
	at Judgeman.judge(Judgeman.java:55)
	at Main.main(Main.java:22)

55というのはjudgeメソッドのif文、22というのはman.judge()をさしています.
judgeメソッド内の最初の出力ではアクセスできているのに（表示されるので)
なぜif文に入るとエラーが出るのかが分かりません.

該当のソースコード

//////////////////////////////Main.javaです
import java.util.*;

public class Main{
  public static void main(String[] args){
    Scanner sc = new Scanner(System.in);
    int n = sc.nextInt();
    int m = sc.nextInt();
    Player[] p = new Player[n];
    for(int i = 0; i < n; i++){
      p[i] = new Player(i);
    }

    Judgeman man = new Judgeman();
    Counter cnt = new Counter(n, m);

    //試しに一回じゃんけんしてみる//clear
    man.startGame(p, cnt);
    //格納された手から、勝ち手を判断できるか試してみる
    man.judge();

  }
}
/////////////////////////////////////Judgeman.javaです
import java.util.*;

public class Judgeman{
  public final int ROCK = 0;
  public final int PAPER = 1;
  public final int SCISSORS = 2;
  public final int TIE = -1;

  Map<Integer, Integer> map = new LinkedHashMap<>();


  public void startGame(Player[] p, Counter cnt){
    for(Player player : p){
      int result = player.showHand();
      this.map.put(result, this.map.getOrDefault(result, 0) + 1);
      //System.out.println("キー" + result + "の値は" + this.map.get(result));
    }
  }

  public int judge(){

    System.out.println("キー0" + "の値は" + this.map.get(0));//これは表示される
    if((this.map.get(ROCK)>=1 && this.map.get(PAPER)>=1 && this.map.get(SCISSORS)>=1) ||
    (this.map.get(ROCK)>=1 && this.map.get(PAPER)==0 && this.map.get(SCISSORS)==0) ||
    (this.map.get(ROCK)==0 && this.map.get(PAPER)>=1 && this.map.get(SCISSORS)==0) ||
    (this.map.get(ROCK)==0 && this.map.get(PAPER)==0 && this.map.get(SCISSORS)>=1)){
      return  TIE;
    }else{
      if((map.get(ROCK)>=1 && map.get(PAPER)>=1 && map.get(SCISSORS)==0)){
        return  PAPER;
      }
      if((map.get(ROCK)==0 && map.get(PAPER)>=1 && map.get(SCISSORS)>=1)){
        return  SCISSORS;
      }
      if((map.get(ROCK)>=1 && map.get(PAPER)==0 && map.get(SCISSORS)>=1)){
        return  ROCK;
      }
    }
    return 100;
  }
}

補足情報（FW/ツールのバージョンなど）

java 10.0.2 2018-07-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.2+13)
Java HotSpot(TM) 64-Bit Server VM 18.3 (build 10.0.2+13, mixed mode)

Comments

[azuapricot]

再現がめんどくさいのでPlayerクラスも貼りつけてほしいな～というところですね( )

[退会済みユーザー]

他の方の回答で解決致しましたので補足はしませんが, 次回から貼るようにします... Counterクラスはともかく, 呼び出し先のメソッドでガンガン使っているPlayerクラスは貼るべきでした. 配慮が足りず申し訳ありません.

Answer 1

    if((this.map.get(ROCK)>=1 && this.map.get(PAPER)>=1 && this.map.get(SCISSORS)>=1) ||
    (this.map.get(ROCK)>=1 && this.map.get(PAPER)==0 && this.map.get(SCISSORS)==0) ||
    (this.map.get(ROCK)==0 && this.map.get(PAPER)>=1 && this.map.get(SCISSORS)==0) ||
    (this.map.get(ROCK)==0 && this.map.get(PAPER)==0 && this.map.get(SCISSORS)>=1)){

Integerとintを==などで比較する場合、オートアンボクシングが働きます。Integerがnullである場合、NullPointerExceptionが発生します。つまり、mapに登録のない値である場合は例外が発生してしまいます。
getではなくgetOrDefaultをここでも使うなどの処置が必要ですが、面倒になるのでお勧めしません。

ここは「出た手の種類の数」が参照できればいいので、

if (this.map.size() == 3 || this.map.size() == 1)

とすればいいのではないでしょうか。

全体を書いてみると

if (this.map.size() == 3 || this.map.size() == 1) {
    return TIE;
}
if (!map.containsKey(ROCK)) {
    return SCISSORS;
}
return map.containsKey(PAPER) ? PAPER : ROCK;

こう書くと、「その手が出ているか出ていないか」を見ることになるので、MapではなくSetでいいということにもなります。

ついでに

startGame()内の

      this.map.put(result, this.map.getOrDefault(result, 0) + 1);

は、こっちのほうが読みやすくなると思います。

      this.map.merge(result, 1, Integer::sum);

Comments

[退会済みユーザー]

前回に引き続き, 今回も回答していただきありがとうございます.
オートアンボクシングについて, get(手) == nullにしたら通りました.
ただ, 数字の中にnullが混じるのも気持ち悪いし, 仰せのようにsize()で分ける方が綺麗なのでsize()を
使いたいと思います.
size()を使った時のあいこでないときの処理について, 拡張for文でキーを取り出してその手を変数なりに格納してどの組みか見つけるという面倒な方法しか思いつきませんでしたが, containsKey()をびっくりマークをつけることでうまく処理されていて勉強になりました.
setを使えば, 3つ目の四角だけで事足りますね...
「ついでに」のmergeメソッドについて, result先がnullなら1, nullでなければresult先の値に1をプラス(この「1」は自分のプラスしたい値)という理解ですが, Integer::sumというのは何か名前のついた文法なのでしょうか?

[退会済みユーザー]

揚げ足ではなく気づいていますので訂正不要ですという意味で一応書いておきますが, 3つ目の四角の最後の箇所は
return map.containsKey(PAPER) ? PAPER : ROCK;だと気づいております.

[swordone]

ミス指摘ありがとうございます。

Integer::sumは「メソッド参照」という書き方です。
mergeメソッドは第1引数のキーのマッピングがなければputメソッドと同じなのですが、マッピングがあった場合は、既存の値と新たな値(第2引数)とを第3引数の関数を使って計算し、putします。

[退会済みユーザー]

教えていただきありがとうございます.