Hibernate @OneToMany关联在保存父实体时会创建重复项

时间:2014-04-23 20:32:10

标签: java database hibernate orm cascade

我有一个涉及品牌(动物)的协会,然后是品牌可以应用的动物物种。在我们的例子中,有Brand模型,Species模型和关系模型BrandSpecies。与我的问题相关的两个是品牌模型和BrandSpecies模型。

表单将为可能已与品牌相关的物种发送ID。我希望避免不得不在集合中循环并检查物种是否已被考虑。

我尝试在物种集上调用clear()并添加从服务器发送的每个ID,但是一旦对象被持久化,旧的DB记录仍然存在。

我的品牌协会:

public class PendingBrandModel implements Serializable, Comparable<PendingBrandModel> {
    .
    .
    .
 @JsonBackReference
 @OneToMany(mappedBy="pending_brand", cascade = {CascadeType.ALL})
 private Set<PendingBrandSpeciesModel> selected_species;

    ...
}

品牌种协会:

public class PendingBrandSpeciesModel implements Serializable {
...
@JsonManagedReference
@ManyToOne()
@JoinColumn(name="pending_brand", referencedColumnName="id", nullable = false)
private PendingBrandModel pending_brand;

@JsonManagedReference
@ManyToOne()
@JoinColumn(name="species", referencedColumnName="id", nullable = false)
private BrandSpeciesModel species;
// below here are the hashCode and equals override methods...
@Override
public int hashCode() {
    final int prime = 31;
    int result = 1;
    result = prime * result + ((id == null) ? 0 : id.hashCode());
    return result;
}

/* (non-Javadoc)
 * @see java.lang.Object#equals(java.lang.Object)
 */
@Override
public boolean equals(Object obj) {
    if (this == obj)
        return true;
    if (obj == null)
        return false;
    if (getClass() != obj.getClass())
        return false;
    PendingBrandSpeciesModel other = (PendingBrandSpeciesModel) obj;
    Boolean brandIDsMatch = false;
    Boolean speciesIDsMatch = false;
    if(pending_brand != null && other.getPending_brand() != null) {
        if((pending_brand.getId() != null && other.getPending_brand().getId() != null) &&
                pending_brand.getId().intValue() == other.getPending_brand().getId().intValue())
            brandIDsMatch = true;
    }
    if(species != null && other.getSpecies() != null) {
        if((species.getId() != null && other.getSpecies().getId() != null) &&
                species.getId().intValue() == other.getSpecies().getId().intValue())
            speciesIDsMatch = true;
    }
    if(brandIDsMatch && speciesIDsMatch)
        return true;
    else
        return false;
}
}

我填充相关物种集合的方法:

public void assignBrandSpecies(PendingBrandModel brandObj) {
        if(checkedSpeciesTypesStr != null) {
            String[] speciesList = checkedSpeciesTypesStr.split(ADDL_SEPARATOR);
            // Clear existing species
            if(brandObj.getSelected_species() != null)
                brandObj.getSelected_species().clear();
            // Add the roles the admin has chosen
            for(String speciesID : speciesList) {
                PendingBrandSpeciesModel newSpecies = new PendingBrandSpeciesModel();
                newSpecies.setPending_brand(brandObj);
                newSpecies.getSpecies().setId(Integer.parseInt(speciesID));
                if(brandObj.getSelected_species() == null)
                    brandObj.setSelected_species(new HashSet<PendingBrandSpeciesModel>());
                brandObj.getSelected_species().add(newSpecies);
            }
        }
    }

我已经尝试使用.clear()代码并且没有它,但行为保持不变。 然后在它全部运行后我更新了品牌对象。

pendingBrandDAO.update(brandObj);

但是,任何预先存在的记录都会保留在数据库中,无论我是否添加了已经使用过的品牌和物种ID组合,都会添加新记录。

2 个答案:

答案 0 :(得分:1)

我解决了类似的问题重写hashCode()和equals()。

答案 1 :(得分:0)

您发布的代码不完整(...&#39; s)但我的问题是它与hashCode / equals有关,以及当您的id首次出现时您的id可能为null添加到集合中。如果计算出的hashCode / equals值从您第一次插入集合后发生变化,那么它将无法获得&#34;找到&#34;在集合中。根据您发布的代码,这只是一个粗略的猜测。也许完整的代码可以进一步澄清事情。

这篇旧文章(见下面的链接)被认为是某种有争议的&#34;有争议的&#34;由hibernate社区,但无论如何它描述了一些可能的陷阱(即使一些hibernate顽固分子并不总是接受建议的解决方案):

Don't let hibernate steal your identity