对象的等于方法

Question

我正在尝试为对象编写一个 equals 方法来比较它们的字段并在它们相等时返回 true。

private int x, y, direction;
private Color color;

public boolean equals(Ghost other){
   if (this.x == other.x && this.y == other.y &&
       this.direction == other.direction && this.color == other.color)
      return true;
   else 
      return false;
}

这有什么问题？

Answer 1

由于color appears to be 和Color，这是一个类，因此是一个引用类型，这意味着您需要使用equals() 来比较颜色。

if (/* ... && */ this.color.equals(other.color)) {

如 cmets 中所述，使用 == 比较引用类型实际上是比较 Java 中的内存地址。如果它们都引用内存中的同一个对象，它只会返回true。

---

akf points out，您需要使用基类 Object 作为参数，否则您不会覆盖 Object.equals()，而是实际上重载它，即提供调用同名方法的不同方式。如果您碰巧传递了一个完全不同类的对象，则可能会发生意外行为（尽管如果它们属于不同类，它仍然会正确返回 false）。

@Override
public boolean equals(Object obj) {
    if (!(obj instanceof Ghost))
        return false;

    // Cast Object to Ghost so the comparison below will work
    Ghost other = (Ghost) obj;

    return this.x == other.x
        && this.y == other.y
        && this.direction == other.direction
        && this.color.equals(other.color);
}

Answer 2

原则上，这看起来不错。

但请注意，您使用== 进行比较。对于原语，这没有问题，但对于对象，它将检查相同的实例，而不是相同的值。这可能是也可能不是您想要的。如果您正在比较例如java.lang.Strings，你会想用equals代替（并检查null）。

Answer 3

如果您要比较对象变量而不是原始类型，则应该使用this.color.equals(other.color) 比较。

在您的情况下，它还取决于您如何创建 Color 对象。如果您使用静态实例（例如 Color.BLUE），那么实际上，它应该没关系。如果您从 rgb 值创建了 Color 对象，那肯定很重要。无论哪种方式，最好习惯使用 .equals() 作为对象变量。

Answer 4

要考虑的一件事是，您没有从Object 覆盖equals 方法，因为您正在更改参数类型。您可能会发现此方法不会像您预期的那样在所有情况下都使用。而不是：

public boolean equals(Ghost other){

你应该有：

public boolean equals(Object other){

然后在内部测试 other 参数是否为 instanceof Ghost 并根据需要进行强制转换。