Постоянное значение всегда должно быть первым в сравнении
Нормальная практика
Постоянное значение занимает второе место в сравнении.
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;
}
}