Spring JPA/Hibernate双向OneToMany新建租户绑定已有房产问题

• JPA双向关联中，加了mappedBy的一侧是关系被拥有方，你的场景下Tenant类上的@OneToMany(mappedBy = "tenant")意味着Hibernate持久化时完全忽略这一侧的关联关系，只会根据Property类中tenant字段（关系拥有方）的值更新外键。
• 你当前流程中MapStruct直接映射得到的Property是仅携带id的游离态对象，且这些对象的tenant字段全为null：
  • 游离态对象不在持久化上下文管理范围内，直接级联操作不会触发预期的关联更新
  • 就算触发CascadeType.MERGE，也会把数据库中对应Property的tenant字段覆盖为null，反而会清空原有绑定，不可能实现关联效果
• 新建的Tenant是瞬时态对象，调用JpaRepository.save()时实际触发的是persist流程，你配置的CascadeType.MERGE在这个场景下根本不会被触发。

不要依赖级联完成已存在实体的关联绑定，级联设计的目标是处理关联实体的新建/更新生命周期，绑定已存在的托管实体必须手动维护双向关联状态，步骤如下：

• 第一步：（可选）优化实体关联维护逻辑
  Tenant侧的@OneToMany不需要额外配置级联类型即可实现需求，如果后续有其他操作需要可以保留原有配置，但不要指望靠它完成本次的绑定逻辑。建议在实体中添加关联维护的工具方法，避免漏设某一侧的关联：

// Tenant类内添加
public void addProperty(Property property) {
    this.properties.add(property);
    property.setTenant(this);
}

public void removeProperty(Property property) {
    this.properties.remove(property);
    property.setTenant(null);
}

• 第二步：修正Service层持久化逻辑
  不要直接保存MapStruct映射出的带游离Property的Tenant实例，先查询出待绑定的托管态Property，手动完成双向关联后再持久化：

@Override
public Long saveTenant(TenantDto tenantDto) {
    Tenant tenant = tenantMapper.toEntity(tenantDto);
    // 提取所有待绑定的Property ID，过滤无效id
    Set<Long> bindPropertyIds = tenant.getProperties().stream()
            .map(Property::getId)
            .filter(Objects::nonNull)
            .collect(Collectors.toSet());
    // 清空映射得到的游离态集合，避免非托管实体引发异常
    tenant.setProperties(new HashSet<>());
    // 查询数据库中已存在的托管态Property
    List<Property> managedProperties = propertyRepository.findAllById(bindPropertyIds);
    // 手动维护双向关联
    for (Property property : managedProperties) {
        tenant.addProperty(property);
    }
    return tenantRepository.save(tenant).getId();
}

• 第三步：（可选）补充参数校验
  如果业务要求传入的Property必须全部存在，可以在查询后校验managedProperties的size和传入的id数量是否一致，不一致则抛出异常提示参数错误。

• 双向关联的状态同步必须由业务代码手动完成，Hibernate、MapStruct都不会自动帮你设置反向引用，只要漏设关系拥有方（Property侧的tenant字段）的值，数据库外键就不会更新。
• 不要给瞬时态的新Tenant直接关联带id的游离Property，不管配什么级联策略，要么触发持久化异常，要么把关联字段覆盖成null，不可能得到预期结果。

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