Javaでnullを受け付けないメソッドにnullを渡したらどうなるべきか - Togetter
Twitterのつぶやきマッシュアップメディア!
まとめ
メニューを開く-
事の起こり
-
今更commons-lang3を使い始めようとしている。 -
しかし重大な事実発覚。commons-langで俺が一番使っているValidate#notNullの仕様が変わっとるｗ IAEだったはずがNPEに変わっているではないかっ。…世の中そういう流れなのかな…。
-
Content from Twitter
-
そして問題提起してディスカッション
-
引数としてnullを受け付けないメソッドにnullを渡した時に飛ぶべきはNullPointerExceptionかIllegalArgumentExceptionか。って哲学っぺーよなｗ 俺は今までIAE派だったのだが。
-
@daisuke_m NullPointerに一票。 -
@daisuke_m そちらのほうが直感的だからですかね。汎用的な変な引数ですよりも、引数がNullです。のほうが後で利用するときに判りやすいとおもってます。
-
@daisuke_m commonsつかわないならNullPointerExceptionであってほしい。commons使うならNullArgumentExceptionであってほしい -
NullArgumentExceptionはIlegalArgumentException継承してんのか。
-
@daisuke_m 3系からかわったのか・・・引数がNullか否かだから引数が不正というわけじゃないんだよなぁ。NPEが文脈にあってるきがする。NPEに変わってしまったら以前のIAEに違和感感じる。
-
@daisuke_m なるほど。そうであればパラメータ不正としてIAEでいいとおもうなぁ
-
@daisuke_m 僕のイメージ的にNPEはnull参照しちゃった例外、IAEはその引数はアカンよ例外なのでIAEの感覚あります。 -
@daisuke_m AssertionError まぁ、実際に使うことはないと思うけど... -
俺がNPEではなくIAEを選択した理由はNPEはコード上でthrowしなくても飛ぶものだから。null参照にメッセージを送ってしまったのか、明示的にthrowされたのかがぱっと見分からないから。IAEに統一しておくと、NPEが飛んだときはnull参照にメッセージしたと確定できる。
-
assertionって「ここはこうなる訳が無い」という、まさに "表明” なんすよね。だから、コード書いた人本人は「ミスでもない限り絶対起こらない」と思ってる。
-
@daisuke_m ミスというか、将来別の個所を変更する際にそうあってはならないという前提をコード内に埋めてる感じだなぁ -
@daisuke_m 実際には、IAEを使うことが多いです。もしくは、アプリケーションによるのですが、nullでもなんとなく動いてしまうようにするという設計にする場合もあります。GUIとかはnullでも動くように作ってしまうことが多いです。
-
1) commons-lang2系に甘んじる、 2) lang3に移行してNPE派に改宗する、 3) Validateの利用を諦めて自作のチェックロジックを作る …瀬戸際だなぁｗ
-
Content from Twitter
コメント