Javaの文字列が空白だった時の判定の条件式はどれが適切？

Question

文字列が空文字と同じか判断

if(str.equals("")){
 ...
}

文字列の長さが0か判断

if(str.isEmpty()){
 ...
}

よく使う処理なのでどちらが良いのかふと気になりました。

Answer 1

• 可読性ではどちらも大差ないと思います。equals("")は空文字""と一致というわかりやすい条件ですし、isEmpty()も空文字、という意味でそのままです。

しかし、ひねくれた見方をすると、リテラルで打ち込まれた""に実は見えない空白文字が入っているのではないか？という可能性はありそうです(どこかからコピペでもしない限りまずそんなこと起きないし、考える人もあまりいないでしょうが)。

• 内部の判定ロジックは、isEmpty()は文字列の長さが0かで判断しますが、equals("")ではいったんObjectに変換された""のinstanceofによる型チェック、そしてキャストが挟まり、そのあと文字列の長さチェックになるので、パフォーマンス面ではisEmpty()に軍配が上がります。
• 判定対象の文字列がnullである可能性があるのならnullチェックが必要ですが、equalsならリテラルと変数を逆にした"".equals(str)という書き方でNullPointerExceptionを回避できます(Tomohiro12さんがご指摘済み、ヨーダ記法というようだ)。isEmpty()でこの芸当はできません。

しかし、変数がnullになることを想定していないのであれば、nullである状態はバグなので、それを早く検知するために通常の書き方をするべきと思います。

以上の点を総合的に考えると、isEmpty()なのかなと私は思います。

Answer 2

if("".equals(str)){
 ...
}

少なくともこうあるべきです
出なければstrがnullの際NullPointerExceptionが発生します

また、StringUtils.isBlank()とisEmpty() は

StringUtils.isEmpty(null)      = true
StringUtils.isEmpty("")        = true
StringUtils.isEmpty(" ")       = false

StringUtils.isBlank(null)      = true
StringUtils.isBlank("")        = true
StringUtils.isBlank(" ")       = true

結果が異なるため、注意してください

Comments

[kurokoba]

割り込み失礼。同意見です。Apache Commonsは外部ライブラリを使っていると大概クラスパスが通っているので積極的に利用しちゃえというのはありっちゃありなんですが、ご指摘のとおり使い方間違えたら意味がないというのとフレームワークとかつかってると"StringUtils"とか"StringUtil"とかパッケージ違いのやつがボロボロ見えて気持ち悪いことになります。
なのでPJに合わせて一旦自作メソッド経由で使うのが個人的にはおすすめ。

Answer 3

よく判定に悩むところですし、コーディングの揺れが気になるところでもありますので、

Apache Commons-Langにある StringUtils.isBlank() や isEmpty() を使うと良いでしょうか。