java - Java equals 方法的错误实现模式

Question

这个问题已经有答案了:

Any reason to prefer getClass() over instanceof when generating .equals()? (11 个回答)

已关闭 9 年前。

我见过很多 java equals() 方法的实现，它们都遵循以下几行:

public boolean equals(Object other){
    if (this == other) 
        return true;

    //this if code
    if (!(other intanceof MyClass))
        return false;
    //ends here

    otherMyClass = (MyClass)other;
    //check all the attribute of this and otherMyClass and return true or false
    //accordingly
}

if 中的代码是否有问题，因为它会为 o1.equals(o2) 返回 true(使用 MyClass 的 o1 对象和 MyClass 子类的 o2 对象)？在大多数情况下，这不是预期的行为。

不会 other.getClass() != this.getClass()是一个更好的比较，而不是上面的粗体 if 吗？

Answer 1

o.getClass() != getClass()) 违反了里氏替换原则。

引用一位伟人的话:

The Liskov substitution principle says that any important property of a type should also hold for its subtypes, so that any method written for the type should work equally well on its subtypes.

《Effective Java》一书有关于此主题的更多详细信息。