Java JPA @OneToMany关联列表新增条目时旧属性被覆盖问题求助

问题根因 #

以下是代码存在的几个核心错误，直接导致旧条目属性被覆盖、关联异常：

1. 最终持久化的是前端传入的脱管myList对象，而非从数据库查询出来的持久态oldList。传入的myList只携带了id和items字段，其余字段（如created、其他未传的自定义字段）全为空，执行saveAndFlush时会直接覆盖数据库中已有记录的对应字段为null。
2. 新添加的myItem没有设置父实体关联（setMyList），由于配置了mappedBy = "myList"，关联关系由子实体维护，未设置关联的子条目外键list_id会为空，无法正确关联到父列表。
3. 条目列表逻辑重复：先把新条目、旧条目合并到临时items列表，又把整个items列表addAll到传入的myList.getItems()里，导致新、旧条目都被重复添加了一遍，存在主键冲突风险。
4. 没有判断新传入的条目是否携带合法主键：如果新条目传入的itemId和已有条目的主键重复，会直接覆盖已有旧条目的全部数据。

修复方案 #

修正后的业务逻辑代码 #

public ResponseEntity<myList> addItem(myList inputList) {
    // 先校验入参合法性
    if (inputList == null || inputList.getId() == null || inputList.getItems() == null || inputList.getItems().isEmpty()) {
        return new ResponseEntity<>(null, HttpStatus.BAD_REQUEST);
    }
    myList oldList = myListRepository.findById(inputList.getId()).orElse(null);
    if (oldList == null) {
        return new ResponseEntity<>(null, HttpStatus.BAD_REQUEST);
    }

    // 处理新条目
    List<myItem> newItems = inputList.getItems();
    for (myItem item : newItems) {
        // 建议此处添加主键生成逻辑，避免传入重复主键覆盖旧数据，比如：
        // if (item.getItemId() == null) item.setItemId(UUID.randomUUID().toString());
        item.setCreated(new Date());
        item.setUpdated(new Date());
        // 绑定父实体关联，维护外键关系
        item.setMyList(oldList);
    }
    syncStuff(newItems);

    // 批量保存新条目
    myItemRepository.saveAllAndFlush(newItems);

    // 更新旧列表的更新时间、合并条目列表
    oldList.setUpdated(new Date());
    oldList.getItems().addAll(newItems);
    
    // 持久化的是数据库查出来的oldList，不会覆盖原有字段
    myList saved = myListRepository.saveAndFlush(oldList);
    return new ResponseEntity<>(saved, HttpStatus.OK);
}

额外优化建议 #

• 建议给myItem的主键itemId添加自动生成策略，避免前端传入重复主键导致旧数据被覆盖。
• 如果允许更新已有条目，可新增判断：如果传入的itemId已存在则走更新逻辑，不存在才走新增逻辑。
• 可开启JPA的show-sql配置，打印执行的SQL语句，快速定位字段被覆盖的具体原因。

内容的提问来源于stack exchange，提问作者CompileNow