mamatumo 2012-08-24 06:04:43

[Java] まったくもって意味不明 このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

nullをスローするので、StackTraceが出るときにヌルポ。 コンパイラも通る。 Findbugsで検知。

private Hoge doHoge(List<String> foo) {
   for (String  str : foo) {
      if (...) {
         ...
      }
   }
   throw null;  // ← なんだコレ
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(8)

#1 Verna_Velna 2012-08-24 06:11:41  

Null 許容しちゃうんだな、というかなぜそこで投げたし。

#3 math_neko 2012-08-24 09:34:21  

> そこまで処理がきてしまったら例外なんです

という実装もおかしいと思うけど、だったら普通に例外投げろよ、って思いません ? なんで null を投げた…。

#4 InternalServerE 2012-08-24 11:05:44  

おおお、
これ、どうやったらcatch できるだろう。。

#5 CoolDriverJPN 2012-08-24 13:26:31  

この発想はなかった。

#7 double_standard 2012-11-02 07:47:33  

false返したら、コンパイル通らなかったから、nullにしたとか?

#8 tacticsrealize 2013-04-14 10:34:59  

何かを知らせたいからthrowするはずなのに何も知らせないという常識に囚われないコード、

作者は一休さんの生まれ変わりなんじゃないだろうか

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[その他] HOW TO ABC..

このエントリーをはてなブックマークに追加

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »

[Java] フルHD推奨。

このエントリーをはてなブックマークに追加

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[Java] 内部的にunicodeだから日本語つかえ...

このエントリーをはてなブックマークに追加

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...

鑑賞する »