奇怪的 Findbugs 错误与 equals

Question

我有 equals 这个方法，但 Findbugs 报告错误，知道吗？

@Override
public boolean equals(final Object obj) {
    return obj instanceof String && this.value != null  
                              && this.value.equals(obj);      // this.value is a String
}

错误是：

Myclass.equals(Object) 检查操作数是否为字符串

Answer 1

您对 MyClass 实现 equals 至少会破坏 equals-contract：

对称：

对于任何非空
参考值x和y，x.equals(y)
应该返回 true 当且仅当
y.equals(x) 返回 true。

在您的情况下：

MyClass A 例如 value="a"：
A.equals("a") 将为 true，但 "a".equals(A) 为 false。这违反了对称性。

反身性：

对于任何非空引用值 x，x.equals(x) 应返回 true。

但是您的实现将返回 false，

A.equals(A) 

但必须返回 true。

ETC。

Answer 2

你的 equals 实现确实很奇怪。

其一，它看起来很像违反了要求

a.equals(a) == true

=== 更新以响应评论 ===

这是平等合同的一部分： http://download.oracle.com/javase/1.5.0/docs/ api/java/lang/Object.html#equals%28java.lang.Object%29

当您将对象放入 Set 或 Map 时，这种行为非常重要。如果没有提到的属性，您会得到奇怪的行为，您可以将实例添加到 Set 中，然后使用完全相同的对象作为参数调用 Set 上的 contains 将导致 false。

=== 针对您更改的问题的另一个更新 ===

由于您检查操作数是 String，但您的类不是 String 的子类，因此根据您的定义，您的类的实例永远不会等于自身的平等。正如另一个答案所述，对称性将被打破。

这也可能有帮助：

Answer 3

 @Override
 public boolean equals(final Object obj) {
    return (obj instanceof YourClass) && (this.value.equals(((YourClass)obj).value));      // this.value is a String
}

据我了解，Findbugs 指出了潜在的错误。