FindBugs错误修改指南.docVIP

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方法被重写，而且含有instanceof逻辑，那么还是不会相等。 解决方法： 应该改为str.toString() 2. IM_BAD_CHECK_FOR_ODD Bug: Check for oddness that wont 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 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主题是，遵守约定(pareTo(y)==0) == (x.equals(y))，强烈建议，但不严格要求。 在return 0的时候，调用equals方法返回true，因为在PriorityQueue.remove方法中，1.5使用的是compareTo方法，而1.6使用的是equals方法，保证环境升级的时候，受影响最小。 解决方法： 在return 0的时候，调用equals方法返回true 7. NM_METHOD_NAMING_CONVENTION Bug: The method name MsmPlanDAOTest.TestViewMsmPlanList() doesnt start with a lower case letter Pattern id: NM_METHOD_NAMING_CONVENTI