java - 查找错误 SIO_SUPERFLUOUS_INSTANCEOF

Question

在我的 findbugs 报告中，我有一个 SIO_SUPERFLUOUS_INSTANCEOF以下代码部分的正确性错误

/** Allow Comparison based on User-Labels */
public int compareTo(AbstractListItem o) {
    if ( !( o instanceof AbstractListItem ) ) {
        // Correctness - Unnecessary type check done using instanceof operator
        // : Method com.x.y.gui.topology.TopologyListNode.compareTo (AbstractListItem) 
        // does an unnecessary type check using instanceof operator when it 
        // can be determined statically
    return -1;
}

静态确定类型的正确方法是什么？

Answer 1

public intcompareTo(AbstractListItem o) - o 是一个AbstractListItem，你不需要检查。

如果您有 public intcompareTo(Object o) 那么您的 instanceof 将被需要并且不会产生警告。