Обычная практика
Постоянное значение занимает второе место в сравнении.
private static final String COMPARE_VALUE = "VALUE123"; public boolean compareIt(String input){ if(input.equals(COMPARE_VALUE)){ return true; }else{ return false; } }
Проблема
Это нормально для сравнения постоянного значения с описанным выше методом, однако это может привести к исключению 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; } }
Оригинал: “https://mkyong.com/java-best-practices/constant-value-should-always-come-first-in-comparison/”