为什么Hibernate新增角色而非在用户外键列存储已有角色ID？

问题根因 #

• 核心配置错误：UserAccount实体类中@OneToOne注解配置了cascade = CascadeType.ALL，该配置包含级联新增（CascadeType.PERSIST）逻辑，保存用户时会自动触发关联角色对象的持久化操作。
• 上下文状态问题：接口传入的角色对象是JSON反序列化生成的临时对象，未被当前JPA持久化上下文托管，Hibernate会将其判定为未存储的新对象，触发级联插入，生成新的角色记录。

修复步骤 #

1. 移除冗余级联配置 #

删除UserAccount类中@OneToOne的cascade = CascadeType.ALL配置，不需要级联修改角色表数据的场景下无需配置级联策略：

@NotNull
@OneToOne
@JoinColumn(name = "role", referencedColumnName = "id")
private UserRole role;

2. 关联已持久化的角色实例 #

新增UserRoleRepository用于查询角色数据，在业务层先查询已存在的角色再和用户关联，避免使用未托管的临时对象：
首先定义角色持久层接口：

@Repository
public interface UserRoleRepository extends JpaRepository<UserRole, Integer> {
}

修改UserService的create方法逻辑：

@Autowired
private UserRoleRepository userRoleRepository;

@Override
public UserAccountDTO create(UserAccount user) {
    // 校验并查询已存在的角色实例
    UserRole existRole = userRoleRepository.findById(user.getRole().getId())
            .orElseThrow(() -> new IllegalArgumentException("指定角色不存在"));
    user.setRole(existRole);
    UserAccount recordedUser = userRepository.save(user);
    return modelMapper.map(recordedUser, UserAccountDTO.class);
}

若无需校验角色是否存在，可使用userRoleRepository.getReferenceById(user.getRole().getId())获取JPA代理对象，减少一次数据库查询。

（可选）优化接口入参设计 #

不要直接将JPA实体作为接口入参，建议单独定义创建用户的请求DTO，仅传递角色ID避免非法修改角色属性：

public class UserCreateRequest {
    private String firstName;
    private String lastName;
    private String password;
    private String username;
    private Integer roleId;
}

控制器层将请求参数转换为UserAccount实体后再调用业务层即可。

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