.equals（）を使用して文字列変数を文字列リテラルと比較する場合、アイテムの順序に標準的な慣行はありますか？[閉まっている]

それぞれに利点があり、違いを理解していますが、ベストプラクティス/標準プラクティスは何ですか？なぜ？

例えば ​​：

"myString".equals(myStringVar)

• 潜在的なNPEを回避し、nullチェックを必要としません。（良いこと？）
• nullチェックが必要ないため、読みやすくなります。
• nullが予期される値でない場合、プログラムは賢明でなくても壊れている可能性があります。

しかしながら

myStringVar.equals("myString")

• nullが期待値である場合、nullチェックが必要です。（良いこと？）
• nullチェックで複合条件を乱雑にすることができます。
• NPEが何かが壊れているかどうかを知らせることができます。

Javaで使用する標準と見なされるバリエーションはどれですか？

これは、古いC（またはC ++）でプログラミングする際に使用される安全上の予防策に起因すると思われます。Cでは、等価性をテストする場合に誤って値を割り当てることができます。

if (x = 3)

xが3に等しいことをテストするのではなく、割り当てxaの値が3であるため、この条件は常に真になります。

if (3 = x)

これには同じ「バグ」が含まれていますが、コンパイラエラーが生成されるため、より安全です。

最近の言語ではこの問題は発生しません。これらの種類のことをしようとすると、簡単に警告することができます。そのため、純粋な好みではないため、1つを選択して一貫して使用してください。

• 標準および優れた実践は、あなたが働いている組織の文化によって異なります

• .NETの私たちの標準は、myStringVar == "myString"単に私たちがそれに同意したからです。つまり、私たちはそれがクリーンで簡潔であると信じています

注：==オブジェクト自体ではなく参照を比較するため、これはJavaには適用されません。Javaではを使用する必要がありますmyStringVar.equals("myString")。

変数のnullnessセマンティクスに依存するという点であなたは正しいと思います。nullであると思わず、チェックして処理する必要がない場合は、最初のフォームがきれいです。

とは言っても、文字列リテラルを埋め込むよりも、おそらく次のような選択肢があります。

private final String MY_STRING = "myString";
if(MY_STRING).equals(myStringVar);

特に、複数の場所で文字列を使用する可能性がある場合。