防止删除级联到实体实例

时间:2015-10-29 11:20:25

标签: hibernate spring-data-jpa jpa-2.1 hibernate-cascade

考虑以下JPA实体:

@Entity @Table(name = "product") class Product { ... }

@Entity @Table(name = "stock") class Stock {
  @JoinColumn(name = "product_id", updatable = false)
  @ManyToOne(fetch = FetchType.LAZY)
  private Product product;

  @Column(name = "quantity")
  private Long quantity;
}

@Entity @Table(name = "cart") class Cart {
  @OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "cart", orphanRemoval = true)
  private List<CartItem> items = new ArrayList<>();

  public void addItem(CartItem item) { items.add(item); }
  public void removeItem(CartItem item) { items.remove(item); }
}

@Entity @Table(name = "cart_item") class CartItem {
  @JoinColumn(name = "cart_id", updatable = false)
  @ManyToOne(fetch = FetchType.LAZY)
  private Cart cart;

  @JoinColumn(name = "product_id", updatable = false)
  @ManyToOne(fetch = FetchType.LAZY)
  private Product product;

  @Column(name = "quantity")
  private Long quantity;

  @JoinColumn(name = "stock_id", updatable = false)
  @ManyToOne(cascade = CascadeType.ALL, fetch = FetchType.LAZY)
  private Stock stock;

  public void setQuantity(Long quantity) {
    final Long delta = this.quantity - Math.max(0L, quantity);

    this.quantity += delta;
    this.stock.setQuantity(this.stock.getQuantity() - delta);

    if(quantity < 1) { cart.removeItem(this); }
  }
}

请注意CartItemStock之间的关联。此关联已注释,以便更改购物车项目数量会影响其在另一个方向上的可用库存,也就是说,如果购物车项目数量增加,则产品的可用库存数量会减少,反之亦然。

这允许我触发cartRepository.save(cart),同时保存所有购物车商品并更新其库存(由于Cascade.ALLCartCartItem)。只要购物车商品的数量非零,这就可以正常工作。

但是,在调用cartRepository.save(cart)后调用cart.removeItem(item)时,级联也会尝试删除购物车商品的库存,这不是意图。应删除购物车项目,但应仅更新其关联库存。有没有办法将更新从CartItem级联到Stock,但在CartItem上级联删除作为Stock上的更新?

2 个答案:

答案 0 :(得分:2)

更改

class CartItem {
  @JoinColumn(name = "stock_id", updatable = false)
  @ManyToOne(cascade = CascadeType.ALL, fetch = FetchType.LAZY)
  private Stock stock;
}

class CartItem {
  @JoinColumn(name = "stock_id", updatable = false)
  @ManyToOne(cascade = { CascadeType.MERGE, CascadeType.PERSIST }, fetch = FetchType.LAZY)
  private Stock stock;
}

CascadeType.ALL{ CascadeType.MERGE, CascadeType.PERSIST })解决了这个问题。以前,在调用cartItem.setQuantity(0)时执行了以下SQL查询:

delete from cart_item where id=?
delete from stock where id=?

根据需要,执行以下SQL查询:

update stock set quantity=? where id=?
delete from cart_item where id=?

示例应用程序为available on Github,用于检查解决方案的正确性。

答案 1 :(得分:0)

至少可以说,你的方法很成问题。如果多个线程修改同一产品的数量,会发生什么? JPA无法防范并发访问!即使您在技术上不会遇到同步问题,您也会逻辑,因为(持久性)数量将被错误的值覆盖(或者您得到OptimisticLockException )。

您应该避免这种头痛并将数量视为动态值,并使用select语句计算产品的当前数量。

如果您坚持缓存数量,则应为此任务使用单独的服务,该服务使用正确的同步,并发地图等。