HashMapをget()する時のNullPointerException警告の回避方法

Question

前提・実現したいこと

AndroidStudioにてjavaをいじっています。(java系初心者です)
エディタ上でいじっている最中からエラーなりワーニングなりが出るので重宝しているのですが、どうしても消せないワーニングが気になっています。
ワーニングですので、理解しているなら放置でもいいのでしょうが、もっと綺麗/安全な書き方があるなら倣いたいと思っています。

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

Unboxing of 'map.get(0)' may produce 'NullPointerException'

該当のソースコード

説明のため、あり得ないコードになっている点は無視してください。
ワーニングの箇所
(1) line 4 のmap.get(0)
line 3の単純なget()では大丈夫なのに、+1みたいな加工を入れるとワーニングになります。
line 3と4を入れ替えてもダメ。

(2) line 5 のmap.get(0)
変数に取り出す事自体がワーニングになり
直前にnullチェックする以外の回避方法が思いつかないです。

また、
line 1でもnew HashMap<>()にワーニング出ていますが、それはいったん無視してください。
(SparseUbtArrayのほうがいいよ、というメッセージ)

1    HashMap<Integer, Integer> map = new HashMap<>();
2    if (map.get() != null) {
3        map.put(1, map.get(0));
4        map.put(2, map.get(0) + 1);
5        int xxxx = map.get(0);
6    }

試したこと

ちょっと条件混乱して分からなくなってしましましたが、直前でnullチェックすればワーニング出なくなるパターンもあったように記憶しています。。

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

AndroidStudio 3.5.3
javaのプロジェクト

Answer 1

エラーメッセージのとおりで、nullが来て困る操作にはワーニングを出しています。

line 3の単純なget()では大丈夫なのに、+1みたいな加工を入れるとワーニングになります。

仮にnullが返ってきてもそのままputすることは可能ですが、計算しようとするとアンボクシングが入ることになり、nullだとうまくいきません。

変数に取り出す事自体がワーニングになり

HashMap<Integer, Integer>となっているように、実際にHashMapに入っている値はintではなくIntegerです。intに入れるにはアンボクシングが必要ですが、nullに対しては行なえません。

Comments

[maisumakun]

intとIntegerの違いに着目してみると、よく分かるかと思います。

[InoueMitsuharu]

アドバイスありがとうございました。
intとInteger(ラッパークラス)の違いを再度 頭の中で整理してみます。

Answer 2

Java8が使える環境ならば、Map#computeIfPresentというメソッドもあったり。

map.computeIfPresent(2, (k, v) -> v + 1));

nullのとき(キーに対して値がまだ関連付けられていないとき)にも値を設定する場合は、mergeメソッドもあります。

map.merge(2, 1, Integer::sum);

Answer 3

原因については既にmaisumakunさんが回答しているので、回避法だけ言及します。

---

NullPointerException警告の回避方法

デフォルト値を適切に決定できるのであれば、
簡単には Map#get の代わりに Map#getOrDefault を使えば良いでしょう。

ただしバグの発見が遅れる恐れもありますので、逐一nullチェックをするのが丁寧かもしれません。