JPA+Hibernate中含@EmbeddedID的父子双向关联映射问题

Alright, let's break down why your child entities (ZonePointDTO) are getting null foreign keys when saving the parent (ZoneDTO), and fix it step by step.

核心问题：双向关联的维护逻辑缺失 #

In JPA bidirectional relationships, the side that owns the foreign key (your ZonePointDTO, since it has the @ManyToOne annotation) is responsible for maintaining the association. Right now, you've set up cascading persist on the parent, but you're not setting the parent reference on each child entity—so Hibernate has no way to link them, hence the null foreign keys.

解决方案步骤 #

1. 手动维护子实体的父引用

You need to ensure every ZonePointDTO in the parent's zonePointsList has its zoneDTO field set to the parent ZoneDTO. You can do this in two ways:

Option A: Add a helper method in ZoneDTO

Add a method to ZoneDTO that automatically sets the parent reference when adding a child:

@Entity @Getter @Setter 
public class ZoneDTO implements GenericPJM<ZoneDTO_Id>, Persistable<ZoneDTO_Id> {
    // ... existing fields ...

    // Initialize the list in the constructor to avoid NPEs
    public ZoneDTO() {
        this.zonePointsList = new ArrayList<>();
    }

    public void addZonePoint(ZonePointDTO zonePoint) {
        zonePointsList.add(zonePoint);
        zonePoint.setZoneDTO(this); // 关键步骤：关联子实体到父实体
    }
}

之后构建ZoneDTO对象时（手动创建或反序列化后），用这个方法替代直接往列表中添加元素。

Option B: 在服务层设置关联关系

如果是通过JSON反序列化得到ZoneDTO（比如你的示例请求），框架不会自动帮你关联父与子。在保存前的服务层添加逻辑：

@Service
public class ZoneService {
    private final ZoneRepository zoneRepository;

    public ZoneService(ZoneRepository zoneRepository) {
        this.zoneRepository = zoneRepository;
    }

    public ZoneDTO saveZone(ZoneDTO zoneDTO) {
        if (zoneDTO.getZonePointsList() != null && !zoneDTO.getZonePointsList().isEmpty()) {
            // 遍历所有子实体，设置父引用
            zoneDTO.getZonePointsList().forEach(point -> point.setZoneDTO(zoneDTO));
        }
        return zoneRepository.save(zoneDTO);
    }
}

2. 使用JPA标准级联注解（可选但推荐）

你当前用的是Hibernate专属的@Cascade注解，建议切换为JPA标准的cascade属性，提升代码可移植性：

@OneToMany(mappedBy = "zoneDTO", fetch = FetchType.EAGER, cascade = CascadeType.PERSIST)
private List<ZonePointDTO> zonePointsList;

这和原来的级联持久化效果一致，但遵循JPA规范而非Hibernate特定API。

3. 修复isNew()方法（潜在隐患）

你当前的isNew()实现依赖静态的zoneNamesList，这不是线程安全的，在多用户环境下会导致错误的持久化判断。建议改用更可靠的方式：

• 通过仓库查询判断父实体的复合主键是否已存在于数据库
• 给ZoneDTO添加@Version字段，通过字段是否为null判断是否为新实体

示例（用@Version实现）：

@Entity @Getter @Setter 
public class ZoneDTO implements GenericPJM<ZoneDTO_Id>, Persistable<ZoneDTO_Id> {
    // ... existing fields ...

    @Version
    private Integer version;

    @Override
    public boolean isNew() {
        // 新实体的version字段为null
        return version == null;
    }
}

为什么这样能解决问题 #

当你给每个ZonePointDTO设置zoneDTO引用后，Hibernate会识别到这个关联关系，在级联持久化时自动将父实体的复合主键值填充到子表的外键字段中。同时这样也确保你只需要调用一次zoneRepository.save()就能完成父子实体的持久化，减少数据库交互次数。

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