FindBugs错误修改指南
使用方法：复制你的Pattern id，然后用快捷键Ctrl + F查，即可快速定位。
所有的解释和解决方式只是可能和建议，大多数时候需要根据实际情况而定。
Findbugs也有少量的误报，需要大家自己甄别。
1. EC_UNRELATED_TYPES
Bug: Call to equals() comparing different types
Pattern id: EC_UNRELATED_TYPES, type: EC, category: CORRECTNESS
解释：
两个不同类型的对象调用equals方法，如果equals方法没有被重写，那么调用object的==，
equals()方法
永远不会相等；如果equals方法被重写，而且含有instanceof逻辑，那么还是不会相等。
解决方法：
应该改为String()
2. IM_BAD_CHECK_FOR_ODD
Bug: Check for oddness that won't work for negative numbers
Pattern id: IM_BAD_CHECK_FOR_ODD, type: IM, category: STYLE
解释：
如果row是负奇数，那么row % 2 == -1，
解决方法：
考虑使用x & 1 == 1或者x % 2 != 0
Class doesn't override equals in superclass
3. NP_ALWAYS_NULL
Pattern: Null pointer dereference
id: NP_ALWAYS_NULL, type: NP, category: CORRECTNESS
A null pointer is dereferenced here.  This will lead to a NullPointerException when the code is executed.
4. RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
Bug: Redundant nullcheck of bean1, which is known to be non-null Pattern id: RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE, type: RCN, category: STYLE
This method contains a redundant check of a known non-null value against the constant null.
这种方法包含了一个称为非空对空值的不断重复检查。
修改为：
5. SS_SHOULD_BE_STATIC
Bug: Unread field: ADDRESS_KEY; should this field be static?
Pattern id: SS_SHOULD_BE_STATIC, type: SS, category: PERFORMANCE
This class contains an instance final field that is initialized to a compile-time static value. Consider making the field static.
解释：
final成员变量表示常量，只能被赋值一次，赋值后值不再改变。
这个类包含的一个final变量初始化为编译时静态值。考虑变成静态常量
解决方法：
增加static关键字
6. EQ_COMPARETO_USE_OBJECT_EQUALS
Bug: RsInterface defines compareTo(Object) and uses Object.equals()
Pattern id: EQ_COMPARETO_USE_OBJECT_EQUALS, type: Eq, category: BAD_PRACTICE
解释：
第一段代码，没有使用instanceof判断就直接转型，有抛出classcastexception异常的可能。
这个BUG主题是，遵守约定(xpareTo(y)==0) == (x.equals(y))，强烈建议，但不严格要求。在return 0的时候，调用equals方法返回true，因为在ve方法中，1.5使用的是compareTo方法，而1.6使用的是equals方法，保证环境升级的时候，受影响最小。解决方法：
在return 0的时候，调用equals方法返回true
7. NM_METHOD_NAMING_CONVENTION
Bug:The method name MsmPlanDAOTest.TestViewMsmPlanList() doesn't start with a lower case letter
Pattern id: NM_METHOD_NAMING_CONVENTION, type: Nm, category:
BAD_PRACTICE
Methods should be verbs, in mixed case with the first letter lowercase, with the first letter of each internal word capitalized.
解释：
方法应该是动词，与第一个字母小写混合的情况下，与每个单词的首字母大写的内部。
解决方法：
方法名称小写就通过了。
8. HE_EQUALS_USE_HASHCODE
Bug: PerfmSingleGraphPanel$RSCategory defines equals and uses Object.hashCode()
Pattern id: HE_EQUALS_USE_HASHCODE, type: HE, category: BAD_PRACTICE
解释：
重载了equals方法，却没有重载hashCode方法，如果使用object自己的hashCode，我们可以从JDK源代码可以看到object的hashCode方法是native的，它的值由虚拟机分配（某种情况下代表了在虚拟机中的地址或者唯一标识），每个对象都不一样。所以这很可能违反“Equals相等，hashcode一定相等；hashcode相等，equals不一定相等。”除非你保证不运用到HashMap/HashTable等运用散列表查值的数据结构中。否则，发生任何事情都是有可能的。
关于何时改写hashcode，请参考：在重写了对象的equals方法后，还需要重写hashCode 方法吗?
关于编写高质量的equals方法：
1.先使用==操作符检查是否是同一个对象，==都相等，那么逻辑相等肯定成立；
2.然后使用instanceof操作符检查“参数是否为正确的类型”；
3.把参数转换成正确的类型；
4.对于该类中的非基本类型变量，递归调用equals方法；
5.变量的比较顺序可能会影响到equals方法的性能，应该最先比较最有可能不一致的变量，或者是开销最低的变量。
当你编写完成equals方法之后，应该问自己三个问题：它是否是对称的、传递的、一致的？
解决方法：
除非你保证不运用到HashMap/HashTable等运用散列表查值的数据结构中，请重写hashcode方法。
9. NM_CONFUSING

版权声明：本站内容均来自互联网，仅供演示用，请勿用于商业和其他非法用途。如果侵犯了您的权益请与我们联系QQ:729038198，我们将在24小时内删除。