重写hashCode（）和equals（）以将对象存储在不能正常工作的hashMap中

Question

我重写了类（Dog）中的hashCode（）和equals()，以便从hashMap中存储和检索它的实例，代码如下：

class Dog {

public Dog(String n) {
    name = n;
}
public String name;

public boolean equals(Object o) {
    if ((o instanceof Dog)
            && (((Dog) o).name == name)) {
       return true;
    } else {
       return false;
    }
}

public int hashCode() {
    return name.length();
   }
}

hashMap代码如下所示：

public class MapTest {

public static void main(String[] args) {
    Map<Object, Object> m = new HashMap<Object, Object>();
    m.put("k1", new Dog("aiko"));
    Dog d1 = new Dog("clover");
    m.put(d1, "Dog key");    // #1
    System.out.println(m.get("k1"));
    String k2 = "k2";
    d1.name = "arthur";     // #2
    System.out.println(m.get(d1)); #3 
    System.out.println(m.size()); 
  }
}

问题是，在2时，我更改了存储在hashMap中的dog对象的名称，在1时，预期的输出在3时为NULL，但实际的是dog键！！我希望它在equals（）方法中失败，就像Clover！=Arthur一样，但它成功了！！我注意到，当hashCode成功（即lengh==6)时，即使equals（）方法失败，也会检索存储在映射中的值，我更改了==并使用equals()，但没有发生任何更改，问题仍然存在。

Answer 1

您希望使用。equals（）not==比较字符串，它比较引用。

public boolean equals(Object o) {
    if ((o instanceof Dog)
            && (((Dog) o).name.equals(name))) {
       return true;
    } else {
       return false;
    }
}

另外，equals方法也有点偏离。如果name为null怎么办？您将得到一个空指针异常。您需要为那个特殊情况添加另一张支票。