如何用JPA/Hibernate正确实现实体修改其他实体的逻辑？

针对你的需求，推荐采用分层架构+领域实体封装业务行为的方式实现，既符合JPA的设计原则，又能保持代码的清晰性和可维护性。

1. 实体设计（保持纯净，封装状态与基础行为） #

首先定义User和Comment实体，确保关联关系正确，同时让实体封装自身的业务逻辑（比如downvote行为）：

User实体 #

@Entity
class User(
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    val id: Long? = null,
    var karma: Int = 0,
    // 其他字段如用户名、邮箱等
)

Comment实体 #

@Entity
class Comment(
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    val id: Long? = null,
    var rating: Int = 0,
    @ManyToOne(fetch = FetchType.LAZY) // 懒加载优化性能，事务内会自动加载关联对象
    @JoinColumn(name = "owner_id")
    val owner: User,
    // 其他字段如内容、发布时间等
) {
    // 封装downvote的业务逻辑：修改自身评分 + 修改作者karma
    fun downvote() {
        this.rating -= 1
        this.owner.karma -= 1
    }
}

2. 持久层Repository #

使用Spring Data JPA提供的Repository接口，无需手动实现持久化逻辑：

interface CommentRepository : JpaRepository<Comment, Long>
interface UserRepository : JpaRepository<User, Long>

3. 业务层Service（协调事务与持久化） #

把事务控制和持久化操作放在Service层，实体只专注于业务规则：

@Service
@Transactional // 事务保证评论修改和用户karma修改的原子性
class CommentService(
    private val commentRepository: CommentRepository
) {
    fun downvoteComment(commentId: Long) {
        // 根据ID获取评论（事务内获取的实体为托管状态，修改后自动同步到数据库）
        val comment = commentRepository.findById(commentId)
            .orElseThrow { IllegalArgumentException("评论不存在") }
        
        // 调用实体的downvote方法修改状态
        comment.downvote()
        
        // 无需手动调用save！因为事务内的托管实体，JPA会自动将修改同步到数据库
    }
}

为什么这是JPA风格的最佳实践？ #

• 实体职责单一：实体只封装自身的状态和业务行为，不依赖持久化上下文（比如Session），避免耦合。
• 事务保证一致性：@Transactional注解确保评论评分修改和用户karma修改要么同时成功，要么同时失败，避免数据不一致。
• 分层清晰：控制器只负责接收请求、调用Service并返回响应；Service负责业务逻辑和事务控制；Repository负责持久化，符合Spring的分层架构思想。

为什么你之前的方案感觉违和？ #

• 实体中传入Session会让实体与持久化层强耦合，违反单一职责原则，也不利于单元测试。
• 把业务逻辑放在控制器中会导致控制器臃肿，难以维护，不符合MVC的设计思想。

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