java - Hibernate/JPA 使用提供的 Id 进行保存

Question

将实体保存到空数据库时，是否有一种简单的方法可以使用为实体提供的 ID？

当使用EntityManager.persist(...)时调用它会失败，因为它认为实体已分离(因为它有一个 Id)。我不想使用EntityManager.merge(...)出于性能原因(它为每个实体触发额外的选择)。

我尝试了不同的方法，例如提供ForeignGenerator ，但在保存之前 Id 会被清空，并且 NPE 会被触发。

我还尝试提供 Hibernate Interceptor并覆盖其 isTransient()始终返回true ，但是我无法正确级联保存(或者至少我必须自己维护已持久化实体的缓存)。

我认为应该有一个简单的方法来做到这一点，但找不到任何方法。

更新:添加实体映射

@Entity
@Data
@NoArgsConstructor
@EqualsAndHashCode(of = "admNr")
public class Adm implements Serializable {

    @Id
    @Column(name = "ADM_NR", nullable = false)
    private String admNr;

    @OneToMany(mappedBy = "adm", cascade = CascadeType.ALL)
    private Set<Kunde> kundeList = new HashSet<>();

}

@Entity
@Data
@NoArgsConstructor
@EqualsAndHashCode(of = "kundeNr")
public class Kunde implements Serializable {

    @Id
    @Column(name = "Kunde_Nr", nullable = false)
    private String kundeNr;

    @ManyToOne(optional = false)
    @JoinColumn(name = "ADM_Nr", referencedColumnName = "ADM_NR")
    private Adm adm;

}

更新:提供了更多信息

我不手动创建实体，它们是作为数据迁移例程的一部分创建的，并且在此过程中正确设置了双向关系，因此 kunde.getAdm() 是不可能的将永远为空。不管怎样，我尝试删除 nullable = false来自Kunde.adm但结果是一样的。

我确实设置了级联，以便能够仅使用 em.persist(adm) 保存整个图表。因为我不想访问每个 child 并明确地保留它(我提供的模型是一个简化的模型，实际上其中有更多的嵌套级别)。

String不幸的是，PK 是一些我无法摆脱的遗留东西。

持久化图表(调用 em.persist(adm) )时遇到的错误是:

java.lang.IllegalStateException: org.hibernate.TransientPropertyValueException: Not-null property references a transient value - transient instance must be saved before current operation : de.apollon.dmx.workflow.processing.sqlite.adm.domain.Kunde.adm -> de.apollon.dmx.workflow.processing.sqlite.adm.domain.Adm
    at org.hibernate.internal.ExceptionConverterImpl.convert(ExceptionConverterImpl.java:146) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.ExceptionConverterImpl.convert(ExceptionConverterImpl.java:157) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.ExceptionConverterImpl.convert(ExceptionConverterImpl.java:164) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.SessionImpl.firePersist(SessionImpl.java:797) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.SessionImpl.persist(SessionImpl.java:768) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:na]
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[na:na]
    at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
    at java.base/java.lang.reflect.Method.invoke(Method.java:564) ~[na:na]
    at org.springframework.orm.jpa.SharedEntityManagerCreator$SharedEntityManagerInvocationHandler.invoke(SharedEntityManagerCreator.java:305) ~[spring-orm-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at com.sun.proxy.$Proxy129.persist(Unknown Source) ~[na:na]
    at de.apollon.dmx.workflow.processing.sqlite.adm.service.SqliteAdmPackageService.storeAdmPackage(SqliteAdmPackageService.java:60) ~[classes/:na]
    at de.apollon.dmx.workflow.processing.sqlite.adm.service.SqliteAdmPackageService.createAdmPackageResource(SqliteAdmPackageService.java:48) ~[classes/:na]
    at de.apollon.dmx.workflow.processing.sqlite.adm.service.SqliteAdmPackageService$$FastClassBySpringCGLIB$$6a54d95.invoke(<generated>) ~[classes/:na]
    at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204) ~[spring-core-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:746) ~[spring-aop-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163) ~[spring-aop-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:294) ~[spring-tx-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:98) ~[spring-tx-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:185) ~[spring-aop-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:688) ~[spring-aop-5.0.8.RELEASE.jar:5.0.8.RELEASE]
    at de.apollon.dmx.workflow.processing.sqlite.adm.service.SqliteAdmPackageService$$EnhancerBySpringCGLIB$$5ba50508.createAdmPackageResource(<generated>) ~[classes/:na]
    at de.apollon.dmx.workflow.processing.service.AdmPackageService.createAdmPackage(AdmPackageService.java:59) ~[classes/:na]
    at de.apollon.dmx.workflow.processing.service.ProcessingService.processAdmPackage(ProcessingService.java:111) [classes/:na]
    at de.apollon.dmx.workflow.processing.service.ProcessingService.runTest(ProcessingService.java:58) [classes/:na]
    at de.apollon.dmx.workflow.Application.lambda$applicationRunner$0(Application.java:31) [classes/:na]
    at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:514) ~[na:na]
    at java.base/java.util.concurrent.FutureTask.runAndReset$$$capture(FutureTask.java:305) ~[na:na]
    at java.base/java.util.concurrent.FutureTask.runAndReset(FutureTask.java) ~[na:na]
    at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:305) ~[na:na]
    at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1135) ~[na:na]
    at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[na:na]
    at java.base/java.lang.Thread.run(Thread.java:844) ~[na:na]
Caused by: org.hibernate.TransientPropertyValueException: Not-null property references a transient value - transient instance must be saved before current operation : de.apollon.dmx.workflow.processing.sqlite.adm.domain.Kunde.adm -> de.apollon.dmx.workflow.processing.sqlite.adm.domain.Adm
    at org.hibernate.action.internal.UnresolvedEntityInsertActions.checkNoUnresolvedActionsAfterOperation(UnresolvedEntityInsertActions.java:122) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.engine.spi.ActionQueue.checkNoUnresolvedActionsAfterOperation(ActionQueue.java:432) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.SessionImpl.checkNoUnresolvedActionsAfterOperation(SessionImpl.java:631) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    at org.hibernate.internal.SessionImpl.firePersist(SessionImpl.java:794) ~[hibernate-core-5.2.17.Final.jar:5.2.17.Final]
    ... 29 common frames omitted

Answer 1

你遇到了先有鸡还是先有蛋的问题。一方面，您拥有带有 Kunde 集和 cascade = CascadeType.ALL 的 Adm。通过级联，您可以要求 Adm 实体为您保存 Kunde 关系，而无需设置 Kunde< 的 Adm 字段。所以，我假设你试图拯救他们两个而不这样做。

    Adm adm = new Adm();
    adm.setAdmNr("NO1");
    Kunde kunde = new Kunde();
    kunde.setKundeNr("NO1");
    adm.getKundeList().add(kunde);
    em.persist(adm);

另一方面，您已使用 optional = false 注释了 Kunde 字段，表示不允许不设置 Adm 字段Kunde，上面的代码没有做到这一点。因此它会给你一个非空错误。您应该在问题中提供(部分)堆栈跟踪。

Caused by: org.hibernate.PropertyValueException: not-null property references a null or transient value : model.Kunde.adm

因此，要修复此问题，请删除非空条件或设置 Kunde 的 Adm 属性

    Adm adm = new Adm();
    adm.setAdmNr("NO1");
    Kunde kunde = new Kunde();
    kunde.setKundeNr("NO1");
    kunde.setAdm(adm);
    adm.getKundeList().add(kunde);
    em.persist(adm);

此外，正如我上面所说，您不需要在主键上使用 nullable = false，因为每个主键字段都是使用禁止空值的唯一索引创建的。

顺便说一句，为每个新的 Adm 实例为 kundeList 创建一个 HashSet 并不是一个好主意。您有一个由 Kunde 实体拥有的双向映射，但该所有权被操作子集的 cascade 设置覆盖。假设您将执行的查询多于更新，那么当您查询时，新的 HashSet 将始终被 JPA 托管列表丢弃。 cascade 设置不适用于查询，因此您不会获得填写的 kundeList，除非您的查询特别要求它并且哈希集将被替换为空 JPA无论如何，管理列表。如果您只需向其中添加 em.persist(Kunde) 并删除 cascade 设置，上面的工作代码就可以正常工作。出于所有这些原因，可能还有更多原因，我认为最好自己专门管理关系，并将 kundeList 用作最适合的仅查询字段。

最后，使用字符串作为主键是一个非常糟糕的主意。