FindBugs反模式.docx

FindBugs反模式制定者：cantellow使用方法：复制你的Pattern id，然后查找，即可快速定位。如有疑问，请当面询问，也可以发邮件：kakashixuze@我的博客：所有的解释和解决方式只是可能和建议，大多数时候需要根据实际情况而定。Bug: Call to equals() comparing different typesPattern id: EC_UNRELATED_TYPES, type: EC, category: CORRECTNESS解释：两个不同类型的对象调用equals方法，如果equals方法没有被重写，那么调用object的==，永远不会相等；如果equals方法被重写，而且含有instanceof逻辑，那么还是不会相等。解决方法：应该改为str.toString()Bug: Check for oddness that wont work for negative numbersPattern id: IM_BAD_CHECK_FOR_ODD, type: IM, category: STYLE解释：如果row是负奇数，那么row % 2 == -1，解决方法：考虑使用x 1 == 1或者x % 2 != 0Bug: 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方法返回trueBug: 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方法。Bug: Field PDHSubCardInstanceDialogCommand.m_instance masks field in superclass ViewNEProperityPattern id: MF_CLASS_MASKS_FIELD, type: MF, category: CORRECTNESS解释：这是什么意思呢？想要字段也能够具有多态性吗？太迷惑了。当你想要更新一个m_instance时，你要更新哪个？你用到它时，你知道哪个又被更新了？解决方法：要么去掉其中一个字段，要么重新命名。Bug: The class name crossConnectIndexCollecter doesnt start with an