Java で引数の null チェックで迷った話 - yukungのブログ

これは Java Advent Calendar 2015 の 15 日目の記事です。 昨日は @opengl_8080 さんの Byteman 使い方メモ＋α でした。明日は @irof さんです。 前置き ついこないだチームでちょっとだけ話題に上って、みんなある程度指針は持っているものの、割と悩みつつ明確に答えを出せなかったので、もっと良い意見があればと思って晒してみます。まぁよくある話だし、Java 8 で Optional が使えるようになって null について語られるケースが増えたと思うので、再考するちょうどよい機会になればいいなーと思います。初心者向けです。 どう処す？処す？ こんな状況の時にあなたならどうしますか？ // Generics なのは例です。String でもなんでもいいです public T doSomething(T input) { // input が

[yukung]

Java Advent Calendar 15 日目、書きました！

[homarara]

NPEかな。主観になるがIAEは「想定内だがその引数はダメ」を咎める例外だと思う。空を返すのは、引数がnullのケースも正常な操作として扱う意味になるので、null不可のメソッドならやるべきではない。

[umaemong]

標準APIだとメソッドの先頭で引数を触ることでNPEを投げてることが多い。効率のために意図的にそうなってる。.NETだとArgumentNullExceptionがあるので(・∀・)ｲｲ!!。Commonsに似たようなのがあるよ。

[warufuzaketaichi]

チーム内で合意が取れて一貫性のあるやり方をするなら、どちらでもいいです。

[knjname]

例外は型階層における型というよりは漠然としたメッセージングなんですよねえ

[oinume]

自分もIllegalArugmentException派だった

[HHR]

Optionalとか便利なメソッドとか

[naga_sawa]

== null で IAE 派だったけど標準ライブラリが NPE 返すからそっちに転向/Objects.requireNotNull 知ってからはそっち常用/贅沢言えばエラーメッセージのテンプレートを仕込めればいいのだが…> requireNotNull

[yumiko-deguchi]

“ こんな状況の時にあなたならどうしますか？”

[quabbin]

私は基本JSR-305任せ派だけど、いまのところ、結構チェックが漏れるんですよね…。

[hs_hachi]

自分もillegalArgumentExceptionかなー 引数が不正という意味でわかりやすいから