equals()
being called to compare a String
with an empty string.
In this case, using .isEmpty()
is better as it shows you exactly what you're checking.
Example:
void checkString(String s){
if ("".equals(s)) throw new IllegalArgumentException();
}
After the quick-fix is applied:
void checkString(String s){
if (s != null && s.isEmpty()) throw new IllegalArgumentException();
}
"".equals(str)
returns false when str
is null. For safety, this inspection's quick-fix inserts an explicit
null-check when
the equals()
argument is nullable. Use the option to make the inspection ignore such cases.