| 定义。该方法返回 false，因为 p 不是 ColoredPoint。该方法返回 false，因为 p 不是 ColoredPoint。 因此，equals定义的关系不是对称的。
对称的缺失将为集合造成意想不到的后果。下面为一个示例：
Set< Point> hashSet1 = new java.util.HashSet< Point>();
hashSet1.add(p);
System.out.println(hashSet1.contains(cp)); // 输出 false
Set< Point> hashSet2 = new java.util.HashSet< Point>();
hashSet2.add(cp);
System.out.println(hashSet2.contains(p)); // 输出 true
Java：所有的equals方法实现都是错误的？(6)
时间:2011-01-05 司马牵牛译
因此，即使 p 和 cp 是相等的，一个 contains 测试成功，而另一个却失败。
如何更改equals的 定义可以让它变为对称？基本上有两种方式。您可以使关系更一般或更严格。使它更加一般意味着对两个对象，a 和 b 被认为是相等的，如果比较 a 和 b 或 b 和 a 输出 true。以下为完成该功能的代码：
public class ColoredPoint extends Point { // 有问题：equals 不具有传递性
private final Color color;
public ColoredPoint(int x, int y, Color color) {
super(x, y);
this.color = color;
}
@Override public boolean equals(Object other) {
boolean result = false;
if (other instanceof ColoredPoint) {
ColoredPoint that = (ColoredPoint) other;
result = (this.color.equals(that.color) && super.equals(that));
}
else if (other instanceof Point) {
Point that = (Point) other;
result = that.equals(this);
}
return result;
}
}
在 ColoredPoint 中，equals的新定义比旧版本多了一种情况的检查：如果其他对象是 Point 而不是 ColoredPoint，该方法将使用 Point 的equals方法。这样就可以取得预期的效果，使equals具有对称性。现在，“cp.equals(p)”和“p.equals(cp)”都返回 true。 然而，equals的约定还是被打破了。现在的问题是，新的关系不再具有传递性！为了演示这个问题，下面进行一系列的声明。定义一个点和两个不同色的颜色点，所有点在同一位置：
ColoredPoint redP = new ColoredPoint(1, 2, Color.RED);
ColoredPoint blueP = new ColoredPoint(1, 2, Color.BLUE);
单独来看，redp 等于 p 并且 p 等于 bluep：
System.out.println(redP.equals(p)); // 输出 true
System.out.println(p.equals(blueP)); // 输出 true
然而，比较 redP 和 blueP，输出 false：
System.out.println(redP.equals(blueP)); // 输出 false
因此，这违反了equals约定中的传递性子条款。
让equals关系更一般看来是死路一条。下面我们试试让它更严格。使equals更严格的一个方法是：不同类的对象，不同地对待。通过修改类 Point 和 ColoredPoint 中的equals方法来实现。 在类 Point 中，可以添加一个额外的比较，用于检查其他点的运行时类是否与这个点的类相同，代码如下：
// 技术上有效，但仍不能令人满意的 equals 方法
public class Point {
private final int x;
private final int y;
public Point(int x, int y) {
this.x = x;
this.y = y;
}
public int getX() {
return x;
}
public int getY() {
return y;
}
@Override public boolean equals(Object other) {
boolean result = false;
if (other instanceof Point) {
Point that =
|
