不能在Hibernate中持久化Hashset

时间:2017-02-03 15:49:39

标签: java mysql hibernate spring-mvc

我在hibernate中保存Hashset时遇到了麻烦。我试图保存多个条目,但它只保存它获得的第一个项目。我循环并添加新元素以在每次迭代中设置但最终其大小始终为1.执行后,在数据库表中,创建一个新的priceList,但在价格表中只创建一个条目(它应该创建5) 。更多信息在图片中。

PriceList.java

@Id @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;
    public void setId(int id){
        this.id = id;
    }
    public Integer getId(){
        return this.id;
    }

    @Column(name = "name")
    @NotBlank( message = "Name is required" )
    private String name;
    public void setName(String name){
        this.name = name;
    }
    public String getName(){
        return this.name;
    }

    @OneToMany(mappedBy = "priceList", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
    @JsonIgnore
    private Set<Price> prices = new HashSet<Price>();
    public void setPrices(Set<Price> prices)
    {
        this.prices.addAll(prices);
    }
    public Set<Price> getPrices()
    {   return this.prices; }

Price.java

@Id @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;
    public void setId(int id){
        this.id = id;
    }
    public Integer getId(){
        return this.id;
    }

    @ManyToOne
    @JoinColumn(name = "product_id",referencedColumnName = "id")
    @JsonSerialize(using = ProductSerializer.class)
    private Product product;
    public void setProduct(Product product){
        this.product = product;
    }
    public Product getProduct(){
        return this.product;
    }

    @ManyToOne
    @JoinColumn(name = "price_list_id",referencedColumnName = "id",updatable = true,insertable = true)
    private PriceList priceList;
    public void setPriceList(PriceList priceList){
        this.priceList = priceList;
    }
    public PriceList getPriceList(){
        return this.priceList;
    }

    @Column(name = "price")
    private float price;
    public void setPrice(float price){
        this.price = price;
    }
    public float getPrice(){
        return this.price;
    }

    @Column(name = "discount")
    private float discount;
    public void setDiscount(float discount){
        this.discount = discount;
    }
    public float getDiscount(){
        return this.discount;
    }

在我的控制器中我正在做这个

控制器

PriceList defaultPriceList = priceListService.findDefaultPriceList();
                    List<Price> defaultPriceListPrices = pricesService.findPricesByPriceList(defaultPriceList);
                    Set<Price> newPrices = new HashSet<Price>();

                    System.out.println(defaultPriceListPrices.size());
                    for (Price p : defaultPriceListPrices)
                    {
                        Price myPrice = new Price();
                        myPrice.setPrice(p.getPrice());
                        myPrice.setDiscount(p.getDiscount());
                        myPrice.setProduct(p.getProduct());
                        myPrice.setPriceList(p.getPriceList());

                        newPrices.add(myPrice);

                        System.out.println("product id" + myPrice.getProduct().getId());
                        System.out.println("price list id" + myPrice.getPriceList().getId());
                        System.out.println("price" + myPrice.getPrice());
                        System.out.println("discount" + myPrice.getDiscount());
                        System.out.println();

                    }
                    System.out.println("new prices size" + newPrices.size());

                    priceList.setPrices(newPrices);
                  priceListService.save(priceList);

我目前在控制台上获得此输出

enter image description here

执行前

enter image description here

enter image description here

执行后

enter image description here

enter image description here

更新

Price.java

@Override
public boolean equals(Object object) {
    if (object instanceof Price) {
        Price price = (Price) object;
        return
                price.getId() == this.getId() && price.getPriceList().getId() == this.getPriceList().getId();
    }
    return true;
}

@Override
public int hashCode() {
    return new HashCodeBuilder(17,37)
            .append(id)
            .toHashCode();
}

2 个答案:

答案 0 :(得分:2)

问题在于hashCode上的Price实施。

equalshashCode的实现通常都是错误的,因为它们仅仅根据实体ID的价值来确定其等式和哈希计算的基础。如果新创建的实例中ID@GeneratedValue结果,则无法正常工作。

在您的情况下,每次向Price添加新的Set<>实例时,都会计算相同的hashCode值,因为每个新实例都有一个空ID,所以他们不断被取代。

调整您的equalshashCode实施:

@Override
public boolean equals(Object object) {
  if ( object == this ) {
    return true; // instance equality
  }
  if ( object == null || object.getClass() != getClass() ) {
    return false; 
  }
  final Price other = Price.class.cast( object );
  if ( getId() == null && other.getId() == null ) {
    // perform equality check against all non-id attributes
  }
  else {
    // perform equality check only on id
  }
}

@Override
public int hashCode() {
  final HashCodeBuilder hcb = new HashCodeBuilder( 17, 37 );
  if ( id == null ) {
     hcb.append( price );
     hcb.append( discount );
     // other fields
  }
  else {
    // only identity basis
    hcb.append( id );
  }
  return hcb.toHashCode();
}

这确保在比较Price的两个非持久对象时,它们的比较/散列基于非标识属性。一旦持久化,这些方法将比较/哈希仅基于身份值,允许两个实例,其中一个已被修改,另一个没有等同于。

答案 1 :(得分:1)

您的代码似乎以某种方式复制了新Price实例中的一组现有PriceList实例,并且最后一步它会保留这些新数据(新PriceList和新的Price个实例)。

如果我是对的,请在插入后仔细查看price_list_id表中记录的Price列值。
您不能将新price_list_id(36)用作fk,但仍使用上一组价格使用的price_list_id(25)。

问题出在你写的循环中   myPrice.setPriceList(p.getPriceList());而不是引用priceList,引用您正在创建的新PriceList实例的变量,用于在保留prices实例之前设置PriceList Set :

priceList.setPrices(newPrices);
priceListService.save(priceList);

我认为将myPrice.setPriceList(p.getPriceList());替换为myPrice.setPriceList(priceList);可以解决您的问题。