Spring Hibernate问题：自关联多对多实体无法更新子评论

我看了你的代码，问题主要出在双向关联的维护方式和冗余字段上，咱们一步步来修复：

核心问题分析 #

1. 冗余的parentId字段：你已经通过@ManyToOne的parentComment维护了父子评论的关联，再单独存parentId完全没必要，反而会导致JPA的关联映射和手动存储的ID不一致，干扰持久化逻辑。
2. 关联维护方向错误：你的@OneToMany(mappedBy = "parentComment")明确指定了子评论是关联的维护端（因为mappedBy指向子实体的关联字段）。这意味着JPA只会通过子评论的parentComment字段来维护外键，单纯更新父评论的subComments集合不会同步到数据库。
3. 缺少事务管理：如果addComment方法没有加@Transactional，两次save操作可能不在同一个事务中，父评论的修改可能无法被JPA检测到并持久化。

具体修复步骤 #

1. 清理Comment实体的冗余字段 #

删掉Comment类中的parentId字段以及对应的getParentId()、setParentId()方法，还有toString()里的parentId引用。让JPA完全通过实体关联来维护父子关系：

修改后的核心关联代码：

@Entity
public class Comment {
    // ... 其他字段不变 ...

    @OneToMany(mappedBy = "parentComment", cascade = CascadeType.ALL) // 可选：添加级联操作，方便后续删除父评论时处理子评论
    private Set<Comment> subComments = new HashSet<>();

    @ManyToOne
    @JoinColumn(name = "parent_comment_id", referencedColumnName = "id") // 明确外键列名，更清晰
    private Comment parentComment;

    // ... 其他getter/setter不变，去掉parentId相关代码 ...

    @Override
    public String toString() {
        return "id: " + id + " text: " + text + " isParent: " + isParent;
    }
}

2. 修正addComment方法，正确维护双向关联 #

添加@Transactional注解保证事务一致性，同时维护关联的两端（子评论设置父评论，父评论添加子评论）：

import org.springframework.transaction.annotation.Transactional;
import org.springframework.web.server.ResponseStatusException;
import org.springframework.http.HttpStatus;

// ...

@Transactional
public ResponseEntity<Comment> addComment(Comment comment) {
    // 如果有父评论ID（假设前端现在传的是parentComment的id字段）
    if (comment.getParentComment() != null && comment.getParentComment().getId() != null) {
        Comment parent = commentRepository.findById(comment.getParentComment().getId())
                .orElseThrow(() -> new ResponseStatusException(HttpStatus.NOT_FOUND, "Parent comment not found"));
        
        // 必须同时维护关联的两端
        comment.setParentComment(parent);
        parent.getSubComments().add(comment);
        
        // 因为在事务中，JPA会自动跟踪父评论的修改，不需要单独save(parent)
    }

    Comment newComment = commentRepository.save(comment);
    return ResponseEntity.ok(newComment);
}

如果前端是直接传parentId而不是parentComment对象，你可以临时调整方法（但最终建议用实体关联传递）：

@Transactional
public ResponseEntity<Comment> addComment(Comment comment) {
    // 假设前端传的是comment中的parentId字段（临时兼容，之后建议去掉）
    if (comment.getParentId() != null) {
        Comment parent = commentRepository.findById(comment.getParentId())
                .orElseThrow(() -> new ResponseStatusException(HttpStatus.NOT_FOUND, "Parent comment not found"));
        
        comment.setParentComment(parent);
        parent.getSubComments().add(comment);
    }

    Comment newComment = commentRepository.save(comment);
    return ResponseEntity.ok(newComment);
}

为什么之前的代码不生效？ #

• 你只更新了父评论的subComments集合，但因为mappedBy的存在，JPA不会把这个集合的变化同步到数据库——它只认子评论的parentComment字段。
• 冗余的parentId和JPA自动维护的外键列（parent_comment_id）可能冲突，导致数据不一致。
• 没有事务的话，第一次save新评论后，父评论的上下文已经脱离了JPA的持久化上下文，第二次save(parent)可能无法检测到集合的修改。

额外注意事项 #

• 可以给@OneToMany添加cascade = CascadeType.ALL，这样当你删除父评论时，子评论也会被自动删除（根据业务需求选择）。
• 双向关联一定要同时维护两端，否则会出现内存中的集合和数据库数据不一致的情况。
• 尽量避免手动维护外键ID，让JPA通过实体关联来处理，减少人为错误。

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