正确重写 equals 方法

Question

我的老师给了我一个 equals 重写示例的解决方案，它是这样的：

@Override
public boolean equals(Object o)
{
    if (this == o) return true;
    boolean result = false;
    if (o instanceof Product)
    {
        Product other = (Product)o;
        result = this.id == other.id;
    }
    return result;
}

该方法被 Product 类重写，它有一个对于每个产品都是唯一的属性 id。但我不明白第一个if的含义，我认为第二个if已经检查了第一个if的限制。谁能给我一个这个代码工作的例子，而下面这个代码不行？谢谢！

@Override
public boolean equals(Object o)
{
    boolean result = false;
    if (o instanceof Product)
    {
        Product other = (Product)o;
        result = this.id == other.id;
    }
    return result;
}

Answer 1

这两个代码示例都有效。第一个示例中的 if (this == o) return true; 是性能优化^{（很可能是过早优化 - 始终首先分析）}，它检查对象是否是与自身进行比较。在 Java 中，== 运算符比较两个对象是否是同一个实例，而不是比较它们是否是具有相同数据的不同实例。

equals 方法有多种编写风格。我通常是这样做的：

public boolean equals(Object obj) {
    if (obj instanceof Product) {
        Product that = (Product) obj;
        return this.id == that.id;
    }
    return false;
}

如果我知道我的代码永远不会将该对象与其他类型的对象或 null 进行比较，那么我什至可以编写如下所示的代码，以便无论如何它都会抛出异常发生 - 这意味着我的代码有一个错误，所以通过尽早失败，我会发现它并尽快修复它。

public boolean equals(Object obj) {
    Product that = (Product) obj;
    return this.id == that.id;
}

Answer 2

你是对的。第一个 if 语句是多余的，因为 this == o 意味着 o instanceof Product 和 this.id == other.id。

如果争论的是性能，我会说它有点过早优化的味道。

Answer 3

if (this == o) return true;

上述说法是多余的。

更具体地说，它只是检查您是否正在将对象与自身进行比较...因此它可以跳过其下面的代码。例如：

Foo f = new Foo();
f.equals(f); //the if (this == o) would be true. References the same thing.

注意：顺便说一句，如果一个人重写 equals，则应该重写 hashcode() 以维护 hashcode() 的一般契约 - 相等的对象必须具有相同的 hashcode（反之则不然，因为两个对象可以具有相同的 hashcode）散列但不相等。）

http://download.oracle.com/javase/6/docs/api/java/lang/Object.html#equals(java.lang.Object)

Answer 4

在 Eclipse 中，您可以选择“生成 hashCode() 和 equals()...”（菜单源）