文字列の良い比較方法について

Question

文字列の比較を行う際に、
str.equals("A")//自然な書き方 "A".equals(str)//倒置法(?)
の二通りの書き方を見かけますが、
どちらがいいコーディングなのでしょうか？

Answer 1

どちらも一理あるといえると思います。

ですが、どちらも選びがたい時に第三の選択肢として、
Java7以降のjava.util.Objectsクラスのクラスメソッドequals(Object, Object)や、
Apache Commons Lang のクラスメソッドStringUtils.equals(CharSequence, CharSequence)を使う
というのも有りかと思います。これなら、両辺が参照型変数の場合に、両方ともnullの場合にも使えるというメリットがあります。

Comments

[suminotky]

回答ありがとうございました！とても参考になりました！

Answer 2

前者だとnullポインターが発生する可能性があるので自分は後者の書き方をします。
ただ前者でエラーが出る場合、データや実装にミスがあるのでそれに気づける利点があるという考え方もあります。

Answer 3

PHPとか、C言語の数値みたいに、==で比較するような状況だと、代入と間違えたら動かなくなるようにと、
if('A'===$str)
のように、あえて定数を左辺に持ってくる事例があります。Javaのようにメソッドで比較する場合は、どっちで書いても文脈的な効果があるわけでもなさそうですね。