对象的 equals 方法

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 似乎是 Color，这是一个类，因此是一个引用type，这意味着您需要使用equals()来比较颜色。

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

正如注释中所指出的，使用 == 来比较引用类型实际上是在 Java 中比较内存地址。仅当它们都引用内存中的同一对象时，它才会返回 true。

---

akf 指出您需要使用基本对象 类作为参数，否则您不会覆盖 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 并根据需要进行强制转换。