Spring实体双向关联：仅保存单个实体能否同步关联数据？

首先，咱们先拆解你当前实现存在的问题，再一步步给出优化方案：

1. 当前操作的正确性问题 #

你现在的代码有两个核心问题：

• 实体关联设计不符合JPA规范：你的Tournament用@ElementCollection存储用户ID列表，但没有和ApplicationUser建立实体级关联；同时ApplicationUser里也没有定义和Tournament的关联字段，控制器里的user1.tournament.add(tournament)会直接编译报错——因为ApplicationUser根本没有这个属性。
• 冗余的保存操作：退一步说，即便忽略编译错误，保存user1的操作完全没用，因为ApplicationUser里没有关联赛事的字段，保存它不会对数据库产生任何关联相关的变化；而保存tournament只是把用户ID加到ids列表里，这种手动存ID的方式也不是JPA推荐的关联映射方式。

2. 正确的实体关联设计：用双向@ManyToMany实现用户与赛事的关联 #

用户和赛事是典型的多对多关系（一个用户可报名多个赛事，一个赛事可容纳多个用户），我们需要重新设计实体间的关联：

修改Tournament实体（维护端） #

@Entity
public class Tournament {
    // 原有字段保持不变...

    // 替换原来的List<Long> ids，改为关联ApplicationUser实体
    @ManyToMany
    @JoinTable(
        name = "tournament_users", // 关联表的表名
        joinColumns = @JoinColumn(name = "tournament_id"), // 当前实体在关联表的外键
        inverseJoinColumns = @JoinColumn(name = "user_id") // 关联实体在关联表的外键
    )
    private List<ApplicationUser> users;

    // 添加便捷的关联方法
    public void addUser(ApplicationUser user) {
        if (this.users == null) {
            this.users = new ArrayList<>();
        }
        if (!this.users.contains(user)) {
            this.users.add(user);
        }
    }

    // 原有方法保持不变...
}

修改ApplicationUser实体（被维护端） #

@Entity
public class ApplicationUser {
    // 原有字段保持不变...

    // 定义双向关联，mappedBy指定维护端是Tournament的users字段
    @ManyToMany(mappedBy = "users")
    private List<Tournament> tournaments;

    // 添加getter、setter方法
    // 原有方法保持不变...
}

3. 仅保存一个实体同步关联数据的方法 #

在JPA双向关联中，分为维护端（Owning Side）和被维护端（Inverse Side）：

• 维护端是拥有关联表外键控制权的一方（也就是上面Tournament里的users字段，带有@JoinTable注解）
• 被维护端是通过mappedBy指向维护端的一方（ApplicationUser里的tournaments字段）

更新关联关系时，只需要修改维护端的关联集合，然后保存维护端实体即可，JPA会自动同步关联表的数据，无需再保存被维护端。

修改后的控制器代码如下：

@RequestMapping(method = POST, path = "/tournaments/{id}")
@ResponseBody
public ResponseEntity<?> registerInTournament(@PathVariable long id) {
    Authentication auth = SecurityContextHolder.getContext().getAuthentication();
    
    // 用Optional接收查询结果，更符合Spring Data JPA规范
    Optional<ApplicationUser> userOpt = applicationUserRepository.findByUsername(auth.getName());
    if (userOpt.isEmpty()) {
        return ResponseRestBuilder.createErrorResponse("User with email " + auth.getName() + " not found");
    }
    ApplicationUser user = userOpt.get();

    Optional<Tournament> tournamentOpt = tournamentRepository.findById(id);
    if (tournamentOpt.isEmpty()) {
        return ResponseRestBuilder.createErrorResponse("Tournament id " + id+ " not found");
    }
    Tournament tournament = tournamentOpt.get();

    // 检查用户是否已报名
    if (tournament.getUsers().contains(user)) {
        return ResponseRestBuilder.createErrorResponse("User id " + user.id + " is already registered in tournament with id " + id);
    }
    // 检查赛事容量
    if (tournament.getUsers().size() >= tournament.getMaxPlayers()) {
        return ResponseRestBuilder.createErrorResponse("Tournament has reached max capacity");
    }

    // 仅修改维护端的关联集合
    tournament.addUser(user);
    // 仅保存维护端实体，JPA自动同步关联表
    tournamentRepository.save(tournament);

    return ResponseRestBuilder.createSuccessResponse("user successfully registered to tournament");
}

额外小建议 #

• Spring Data JPA的findById默认返回Optional<T>，你的原代码用List接收是不符合规范的，建议用Optional处理空值更优雅。
• 可以把重复判断、空值处理逻辑放到实体的addUser方法里，让实体自己维护关联的完整性，减少控制器里的冗余代码。

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