FindBugs抱怨在此方法中可能在Comparator.compareStrings(String,String)中不可行的分支上str1的可能的空指针取消引用:
@H_301_2@private static int compareStrings(final String str1,final String str2) {
if ((str1 == null) && (str2 == null)) {
return COMPARE_ABSENT;
}
if ((str1 == null) && (str2 != null)) {
return COMPARE_DIFFERS;
}
if ((str1 != null) && (str2 == null)) {
return COMPARE_DIFFERS;
}
return str1.equals(str2) ? COMPARE_EQUALS : COMPARE_DIFFERS;
}
在Eclipse中,我也在最后一行看到一个警告(str1可能为null).
str1.equals(str2)在什么情况下可以str1为null? COMPARE_EQUALS:COMPARE_DIFFERS; (考虑到前两个if块覆盖的情况,当str1为null时)?
解决方法
您可以通过重新排列if语句来避免警告:
@H_301_2@private static int compareStrings(final String str1,final String str2) {
if (str1 == null) {
if (str2 == null)) {
return COMPARE_ABSENT;
} else {
return COMPARE_DIFFERS;
}
} else {
if (str2 == null)) {
return COMPARE_DIFFERS;
} else {
return str1.equals(str2) ? COMPARE_EQUALS : COMPARE_DIFFERS;
}
}
}