1

我有最终字符串作为唯一 ID 的课程。当然我想覆盖equals所以比较只基于ID。那么只返回ID的哈希码是正确的做法吗,如下所示?

class ItemSpec{
    final String name;

    ...

    @Override
    public boolean equals(Object o){
        if(o != null && o instanceof ItemSpec){
            return name.equalsIgnoreCase(((ItemSpec)o).name);
        } else{
            return false;
        }
    }

    @Override
    public int hashCode(){
         if(name == null){
             return 0;
         } else{
             return name.hashCode();
         }
    }
}
4

2 回答 2

12

如果您的 equals 不区分大小写,则不会。您可以有两个ItemSpec相同但具有不同哈希码的结果。这打破了哈希码最关键的要求。

equals必须同意您的hashCode. 因此,如果您要不区分大小写地比较它们,则必须hashCode不区分大小写。

@Override
public int hashCode(){
     if (name == null){
         return 0;
     } else{
         return name.toLowerCase().hashCode();
     }
}

您的hashCode方法也暗示name可能为空。如果是这样,您也应该在您的equals方法中对其进行空检查。

于 2014-09-19T15:05:45.773 回答
-1

我认为您的方法没有任何问题。

但是,我经常看到以下实现:

public int hashCode() {
    final int prime = 31;
    int result = super.hashCode();
    result = prime * result + ((name == null) ? 0 : name.hashCode());
    return result;
}

更新:这是一个有更好解释的链接: hashCode 方法的最佳实现

于 2014-09-19T15:11:24.127 回答