Java JPA - 如何使用列表放置对象并使用 CRUD 更新列表对象？

Question

我有一个对象，其中包含我似乎无法正确更新的其他对象列表。我可以用对象列表（ProductItemQuantity）创建对象（产品）没问题。我也可以对对象列表进行 PUT，但它会创建一个新的对象列表，我所做的一切都是 PUT。我希望更新我提供的对象列表，而不是在每次放置父对象时都创建一个新列表。

如果我向 ProductItemQuantity 添加一个 ID，我会得到一个异常：

detached entity passed to persist

这是我的课程：

Product.java

    @Entity 
public class Product {

    @Id
    @GeneratedValue(strategy=GenerationType.IDENTITY) 
    private Long id;
    @ManyToOne
    private Organization org;
    private String barCode;
    private String name;
    @ManyToOne(cascade=CascadeType.MERGE)
    private Status status;
    @ManyToMany(cascade=CascadeType.MERGE)
    private List<Fee> fees;
    @ManyToMany(cascade=CascadeType.ALL)
    private List<Note> notes;
    @ManyToMany(cascade=CascadeType.ALL)
    private List<ProductItemQuantity> productItems;
    private Integer stock;
    private BigDecimal msrp;
    @CreationTimestamp
    private LocalDateTime createdOn;
    @UpdateTimestamp
    private LocalDateTime updatedOn;

    // Getter & Setters

ProductItemQuantity.java

@Entity 
public class ProductItemQuantity {
    @Id
    @GeneratedValue(strategy=GenerationType.IDENTITY) 
    private Long id;
    private Integer count;
    @ManyToOne
    private ProductItem productItem;
    @CreationTimestamp
    private LocalDateTime createdOn;
    @UpdateTimestamp
    private LocalDateTime updatedOn;

// Getters / setters

ProductItem.java

@Entity 
public class ProductItem {

    @Id
    @GeneratedValue(strategy=GenerationType.IDENTITY) 
    private Long id;
    @ManyToOne
    private Organization org;
    @ManyToOne
    private Supplier supplier;
    private String barCode;
    private String description;
    private String name;
    private Integer stock;
    private Integer caseQty;
    private BigDecimal caseCost;
    @ManyToMany(cascade=CascadeType.ALL)
    private List<Note> notes;
    @CreationTimestamp
    private LocalDateTime createdOn;
    @UpdateTimestamp
    private LocalDateTime updatedOn;

ProductController.java

@PutMapping("/{id}")
public Product update(@RequestBody Product product, @PathVariable long id) {


    Product savedProduct = productService.save(product);


    return savedProduct;
}

工作 CRUD PUT 请求：http://localhost:8080/product/1

{
    "barcode":"12347163",
    "name":"Product 1",
    "stock": 12,
    "msrp": 29.99,
    "org": {
        "id":1
    },
    "status":{
        "id":1
    },
    "productItems":[{
        "count":30
    },{
        "count":30
    }
        ],
        "fees":[{
            "id":1

        },{
            "id":2

        }],
    "notes":[{
        "title":"Product Created",
        "description":"Note created by user X on 12/16/2019 11:00PM"
    },{
        "title":"Product Updated",
        "description":"Product updated stock by user X on 12/16/2019 11:00PM"
    }]
}

CRUD PUT 请求中断：http://localhost:8080/product/1

{
    "barcode":"12347163",
    "name":"Product 1",
    "stock": 12,
    "msrp": 29.99,
    "org": {
        "id":1
    },
    "status":{
        "id":1
    },
    "productItems":[{
        "id":1,
        "count":30
    },{
        "id":2,
        "count":30
    }
        ],
        "fees":[{
            "id":1

        },{
            "id":2

        }],
    "notes":[{
        "title":"Product Created",
        "description":"Note created by user X on 12/16/2019 11:00PM"
    },{
        "title":"Product Updated",
        "description":"Product updated stock by user X on 12/16/2019 11:00PM"
    }]
}

Answer 1

您的对象已分离，因为关系（OneToMany、ManyToMany）仅从一个方向设置。为了持久化它们，您必须将关系设置为双向。您的关系是单向的，因为解析器 (jackson) 将生成以下对象：

Product product = new Product();
Fee fee = new Fee();
fee.setId(1);
product.setFees(Arrays.asList(fee));

在双向关系中，双方都必须设置：

product.getFees().forEach(fee-> fee.getProducts().add(product));

最好将持久性对象与控制器对象分开，因为持久性对象也处理关系。

根据我的经验，如果你想使用 GeneratedValue，你必须先从数据库中选择实体，然后修改它。如果您希望 hibernate 生成它，那么创建一个新对象并在其上设置 id 没有多大意义。

因此，您可能需要先进行选择：

List fee = // 选择产品费用列表中带有id的所有费用

之后：

product.setFees(fees);
fees.forEach(fee -> fee.getProducts().add(product));

您的方法是 PUT，因此您不应直接保存产品对象（它将在数据库中创建一个新条目）。

@PutMapping("/{id}")
public Product update(@RequestBody Product product, @PathVariable long id) {
    Optional<Product> originalProductOptional = productRepository.findById(id);
    // you should add a check here : if originalProduct is not found, return 404
    Product originalProduct = originalProductOptional.get();
    originalProduct.setName(product.getName());
    // here update all fields and relations

    productRepository.save(originalProduct);
    return originalProduct;
 }

Answer 2

答案是将属性更新为：

@JsonIgnoreProperties({"hibernateLazyInitializer", "handler"}) 
private List<ProductItemQuantity> productItemQuantities;

然后在put里面手动设置

    List<ProductItemQuantity> piqs = product.getProductItemQuantities();

    if (piqs != null) {
        List<ProductItemQuantity> piiList = new ArrayList<ProductItemQuantity>();
        for (ProductItemQuantity pii : piqs) {
            // Check for ID in here?
            ProductItemQuantity curPII = piqService.getOne(pii.getId());
            curPII.setCount(pii.getCount());
            piiList.add(curPII);
        }
        originalProduct.setProductItemQuantities(piiList);
    }