Question

我正在编写一些代码来演示equals和hashcode，对于我的实现，我使用了User example：

public class User {
    private String name;
    private String pass;
    //...
    @Override
    public boolean equals(Object obj) {
        if (obj == null) { return false; }
        if (obj == this) { return true; }
        if (obj.getClass() != this.getClass()) {
            return false;
        }
        User rhs = (User) obj;

        EqualsBuilder eb = new EqualsBuilder();
        eb.append(this.getName(), rhs.getName());
        //eb.append(this.getPass(), rhs.getPass());
        return eb.isEquals();
    }
    @Override
    public int hashCode() {
        HashCodeBuilder hcb = new HashCodeBuilder(17, 37);
        hcb.append(this.getName());
        hcb.append(this.getPass());
        return hcb.toHashCode();
    }
    //...
    public static void main(String[] args) {

        User u1 = new User("foo","foo1");
        User u2 = new User("bar","bar1");
        System.out.println(u1.equals(u2));
        System.out.println(u1.hashCode() + " ?= " + u2.hashCode());
        User u3 = new User("foo","foo1");
        User u4 = new User("foo","bar1");
        System.out.println(u3.equals(u4));
        System.out.println(u3.hashCode() + " ?= " + u4.hashCode());

    }
}

输出：

false
2128613651 ?= 2129111967
true
2128615478 ?= 2214545177

我认为我做错了，因为我的对象现在可以等于但是有一个不同的哈希码（我知道这很糟糕），但是我想让我的用户等于他们的名字也是等于...而不是当他们有名字并且通过等于。

我如何尊重这些惯例并拥有我想要实现的目标？感谢您的帮助/澄清：）

Answer 1

一般来说，如果您的equals()方法中没有字段，则不应在hashCode()方法中引用该字段。

这可确保您的hashcode()结果不会更频繁地更改equals()结果。考虑将您的方法更改为：

@Override
public int hashCode() {
    HashCodeBuilder hcb = new HashCodeBuilder(17, 37);
    hcb.append(this.getName());
    return hcb.toHashCode();
}

话虽如此，创建一个非直观的equals()方法并不是一个好主意。我自然会认为你的方法也会考虑密码，就像你未来的许多维护者一样。如果不进行真正的相等测试，单元测试可能更难写。考虑创建自己的方法是否更好，例如

public boolean usernameEquals(User other) {
  ...
}

Answer 2

所有等于的对象具有相同的哈希码。但并非所有具有相同哈希码的对象都是相同的。

您在hascode方法中使用的所有字段都应该在您的equals方法中使用。

Answer 3

我不会说你做错了。就合同而言，如果2个对象相等，那么它们应该具有相同的hashCode。如果2个对象具有相同的hashCode，则它们不必相等

如果将2个用户实例的相等定义为 - 它们相等，如果它们的名称相等，那么你就可以了。就hashCodes而言，你是对的，因为如果2个对象的hashCodes不相等，那么对象就不相等了。

BTW Duncan 完全正确地指出创建一个非直观的equals（）方法很少是一个好主意

哈希码＆amp;等于实施

3 个答案: