9

接下来的两个实体之间存在 OneToOne 关系：

@Entity
@Table(name = "tasks")
public class Task {
    @OneToOne(mappedBy = "task", cascade = CascadeType.PERSIST)
    private Tracker tracker;

    /* More code */
}

@Entity
@Table(name = "trackers")
public class Tracker {
    @OneToOne
    @JoinColumn(name = "trk_task", unique = true)
    private Task task;

    /* More code */
}

我正在尝试运行此代码：

Task task = taskService.findDispatchableTask();
if (task != null) {
    Tracker tracker = trackerService.findIdleTracker();
    if (tracker != null) {
        task.setTracker(tracker);
        task.setStatus(TaskStatus.DISPATCHED);
        taskService.save(task);
    }
}

但我得到这个错误：

ERROR org.hibernate.AssertionFailure  - an assertion failure occured (this may indicate a bug in Hibernate, but is more likely due to unsafe use of the session) 
org.hibernate.AssertionFailure: non-transient entity has a null id

我可以“解决”它，将我的代码更改为：

Task task = taskService.findDispatchableTask();
if (task != null) {
    Tracker tracker = trackerService.findIdleTracker();
    if (tracker != null) {
        tracker.setTask(task);
        trackerService.save(tracker);
        task.setTracker(tracker);
        task.setStatus(TaskStatus.DISPATCHED);
        taskService.save(task);
    }
}

我的问题是，保持 OneToOne 关系的正确方法是什么？在我的代码中，为什么我要保存关系的两个部分以使其工作？

4

1 回答 1

22

我们重新来过吧。

每个双向关联都有两个方面：所有者端和反向端。反面是具有mappedBy属性的一面。所有者一方是另一方。JPA/Hibernate 只关心所有者方面。因此，如果您只是初始化反面，则不会保持关联。

通常，初始化关联的双方是一个很好的做法。首先是因为它确保所有者端已初始化，其次是因为它使实体图连贯，为了您自己的利益。

另请注意，如果您在事务中工作（并且应该），则查询返回的所有实体都是附加实体。当事务提交时（或之前），应用于实体的更改会自动持久化。无需像您正在做的那样明确地保存实体。

于 2013-05-05T06:59:05.910 回答