Java HashMap找不到密钥,但它应该

Question

我的应用程序中出现了一个奇怪的问题,我将快速解释全局架构,然后深入解释我的问题.

我使用一个服务来填充HashMap<DomainObject,Boolean>来自我的数据库(JPA驱动)的数据,然后通过EJB远程方法调用(使用Apache Wicket)返回到我的视图.在这部分中,我DomainObject为返回的地图添加了一个新内容,以便存储来自最终用户的任何新值.

当用户在其浏览器中点击"添加"按钮时,会出现问题,我尝试在地图中检索新创建的项目,但它失败了.通过使用调试器,我面临以下事情.

假设HashMap<DomainObject, Boolean> map并且DomainObject do两个变量很有趣,我在调试器中得到以下结果

map.keySet();给我一个对应的对象do(甚至@whatever simili-reference是相同的),hashcode()在两个对象上返回相似的值并equals()在两个返回之间true

map.containsKey(do); 回报 false

map.get(do); 回来null,很奇怪,因为我的钥匙似乎在map.

假设我新创建的项是枚举的第一个键keySet(),我执行以下操作: map.get(new ArrayList(map.keySet()).get(0)),它返回null.

如果它可以帮助,通过将断点附加到我DomainObject.equals()和DomainObject.hashcode()方法,我发现map.get()只是调用hashcode()而不是equals().

我找到的唯一解决方法是在现有的地图上重新创建一个新地图new HashMap(map),在这个新地图中,我根本没有问题通过其键查找对象.

我希望这里有人可以指出我发生了什么,谢谢.

使用环境:

• OS X 10.7.1下的Sun Java 1.6.0_26 x64
• Debian 6.0.2下的OpenJDK 1.6.0_18 x64(2.6.32)
• Apache Wicket 1.4.17
• Oracle Glassfish 3.1.1
• JBoss Hibernate 3.6.5

DomainObject 代码:

public class AssetComponentDetailTemplate extends BaseEntite<Long> {
public enum DataType {
    TXT,
    DATE,
    INT,
    JOIN,
    LIST,
    COULEURS,
    REFERENCE
}

public enum Tab {
    IDENTITE,
    LOCALISATION,
    CYCLE_DE_VIE,
    FINANCE,
    RESEAU,
    DETAIL
}

@Id
@GeneratedValue(strategy = GenerationType.AUTO)
private Long id;
@Column(nullable = false)
private String name;
@Column(nullable = false)
@Enumerated(EnumType.STRING)
private DataType dataType;
private Integer classNameId;
private Long orderId;
private Long nextAssetComponentDetailTemplateId;
private String unit;
@Enumerated(EnumType.STRING)
private Tab tab;

@Column(nullable = false)
private Long uniqueOrganizationId;

@OneToMany(fetch = FetchType.LAZY)
@JoinColumn(name = "idAssetComponentDetailTemplate", insertable = false, updatable = false)
private List<AssetComponentDetailJoin> assetComponentDetailJoins;

private Boolean mandatory = false;

public AssetComponentDetailTemplate() {
}

public Long getId() {
    return id;
}

public void setId(final Long id) {
    this.id = id;
}

public String getName() {
    return name;
}

public void setName(final String name) {
    this.name = name;
}

public DataType getDataType() {
    return dataType;
}

public void setDataType(final DataType dataType) {
    this.dataType = dataType;
}

public Integer getClassNameId() {
    return classNameId;
}

public void setClassNameId(final Integer classNameId) {
    this.classNameId = classNameId;
}

public Long getUniqueOrganizationId() {
    return uniqueOrganizationId;
}

public void setUniqueOrganizationId(final Long uniqueOrganizationId) {
    this.uniqueOrganizationId = uniqueOrganizationId;
}

public Long getNextAssetComponentDetailTemplateId() {
    return nextAssetComponentDetailTemplateId;
}

public void setNextAssetComponentDetailTemplateId(final Long nextAssetComponentDetailTemplateId) {
    this.nextAssetComponentDetailTemplateId = nextAssetComponentDetailTemplateId;
}

public String getUnit() {
    return unit;
}

public void setUnit(final String unit) {
    this.unit = unit;
}

public Tab getTab() {
    return tab;
}

public void setTab(final Tab tab) {
    this.tab = tab;
}

public Long getOrder() {
    return orderId;
}

public void setOrder(final Long order) {
    this.orderId = order;
}

public Boolean isMandatory() {
    return mandatory;
}

@Override
public String toString() {
    return name;
}

@Override
public boolean equals(final Object o) {
    if (this == o) {
        return true;
    }
    if (o == null || getClass() != o.getClass()) {
        return false;
    }

    final AssetComponentDetailTemplate that = (AssetComponentDetailTemplate) o;

    if (classNameId != null ? !classNameId.equals(that.classNameId) : that.classNameId != null) {
        return false;
    }
    if (dataType != that.dataType) {
        return false;
    }
    if (id != null ? !id.equals(that.id) : that.id != null) {
        return false;
    }
    if (name != null ? !name.equals(that.name) : that.name != null) {
        return false;
    }
    if (nextAssetComponentDetailTemplateId != null ?
        !nextAssetComponentDetailTemplateId.equals(that.nextAssetComponentDetailTemplateId) :
        that.nextAssetComponentDetailTemplateId != null) {
        return false;
    }
    if (orderId != null ? !orderId.equals(that.orderId) : that.orderId != null) {
        return false;
    }
    if (tab != that.tab) {
        return false;
    }
    if (uniqueOrganizationId != null ? !uniqueOrganizationId.equals(that.uniqueOrganizationId) :
        that.uniqueOrganizationId != null) {
        return false;
    }
    if (unit != null ? !unit.equals(that.unit) : that.unit != null) {
        return false;
    }

    return true;
}

@Override
public int hashCode() {
    int result = id != null ? id.hashCode() : 0;
    result = 31 * result + (name != null ? name.hashCode() : 0);
    result = 31 * result + (dataType != null ? dataType.hashCode() : 0);
    result = 31 * result + (classNameId != null ? classNameId.hashCode() : 0);
    result = 31 * result + (orderId != null ? orderId.hashCode() : 0);
    result = 31 * result +
             (nextAssetComponentDetailTemplateId != null ? nextAssetComponentDetailTemplateId.hashCode() : 0);
    result = 31 * result + (unit != null ? unit.hashCode() : 0);
    result = 31 * result + (tab != null ? tab.hashCode() : 0);
    result = 31 * result + (uniqueOrganizationId != null ? uniqueOrganizationId.hashCode() : 0);
    return result;
}

Answer 1

[这基本上扩展了 Jesper 的答案，但详细信息可能对您有帮助]

由于使用重新创建地图new HashMap(map)能够找到该元素，我怀疑hashCode()DomainObject 在将其添加到地图后发生了变化。

例如，如果您的 DomainObject 如下所示

class DomainObject {
    public String name;
    long hashCode() { return name.hashCode(); }
    boolean equals(Object other) { /* compare name in the two */'
}

然后

   Map<DomainObject, Boolean> m = new HashMap<DomainObject, Boolean>();
   DomainObject do = new DomainObject(); 
   do.name = "ABC";
   m.put(do, true); // do goes in the map with hashCode of ABC
   do.name = "DEF";
   m.get(do); 

上面的最后一条语句将返回null. 因为do地图内的对象位于"ABC".hashCode();的存储桶下 桶里什么也没有"DEF".hashCode()。

Map 中的对象一旦添加到 Map 中，其 hashCode 就不应更改。确保它的最好方法是 hashCode 所依赖的字段必须是不可变的。