なんも考えずに"リテラル".equals(s)と書け - 2008-12-04 - きしだのはてな

テクノロジーカテゴリーの変更を依頼記事元:

nowokay.hatenablog.com

25 usersがブックマークコメント

なんも考えずに"リテラル".equals(s)と書け - 2008-12-04 - きしだのはてな

"".equals(stringVariable) は改悪だろうについて。まず、空文字との比較は場合による。というか、空文... "".equals(stringVariable) は改悪だろうについて。まず、空文字との比較は場合による。というか、空文字との比較なら s != null && s.isEmpty() か s == null || s.isEmpty() などと書くほうがいい。あと、件の問題は改悪とは言わないまでも、問題文だけをみてこれを改善といえるほどでもないので、あまり良くない。中身があるリテラルとの比較。基本的な方針として「二通りの書き方があるとして、実行コストや記述コストが同じならより安全で判断の必要がない書き方を選ぶべき」というのがある。というか、これはコードの書き方だけではなく、すべての行動にあてはまる。判断不要で安全な方法があるのなら、それを選ぶべきだ。シャワーをまっすぐにシャワー置きにかけずに横向きにかけるというのは、これにあてはまる。確かにシャワー置きにはシャワーをまっすぐ置き

idesaku この記法でNPEを先送りしても、それは"たまたま動いている"状態なわけで、品質面から考えるといただけない。また、一般的にミスは早く見つかるほど被害が小さくなるので、ごまかしを考えるべきではない。