常に一定の値が最初に比較に来るべきです
通常の練習
比較では定数値が2番目になります。
private static final String COMPARE__VALUE = "VALUE123"; public boolean compareIt(String input){ if(input.equals(COMPARE__VALUE)){ return true; }else{ return false; } }
問題
上記のメソッドと定数値を比較するのは問題ありませんが、ユーザーが比較のために “null”値を渡すと、
NullPointerException
が発生する可能性があります。例えば :
if(input.equals(COMPARE__VALUE))//hit NullPointerException if input is "null"
ベストプラクティス
常に一定の値を比較する必要があります。
private static final String COMPARE__VALUE = "VALUE123"; public boolean compareIt(String input){ if(COMPARE__VALUE.equals(input)){ return true; }else{ return false; } }
リンク://タグ/ベストプラクティス/[ベストプラクティス]リンク://タグ/java/[java]