JPA错误：InvalidDataAccessApiUsage异常：传递到持久化的分离实体

Question

我使用oneToMany注释将两个实体相互映射。一个实体是bookedBus，第二个实体是drivers。drivers实体已经插入了一行，该行后来会成为bookedBus实体（PK）的外部引用（FK）。下面是两个实体，setter和getter为简洁起见被跳过。

第一实体

@Entity
@Table(name = "bookedBuses")
public class BookedBuses implements Serializable {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    @ManyToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL)
    @JoinColumn(name = "driver_id")
    private Drivers driver;
}

第二实体

@Entity
public class Drivers implements Serializable {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    @OneToMany(fetch = FetchType.LAZY, cascade = CascadeType.ALL, orphanRemoval = true, mappedBy = "driver")
    private List<BookedBuses> bookedBus;
}

现在当我尝试保存到已预订的总线实体时，它会抛出以下异常

组织。springframework。刀。InvalidDataAccessApiUsageException:传递给persist:com的分离实体。公共汽车应用程序编程接口。实体司机；嵌套的异常是org。冬眠PersistentObjectException:传递给persist:com的分离实体。公共汽车应用程序编程接口。实体司机

下面是我如何尝试保存到bookedBus实体的

BookedBuses bookedRecord = new BookedBuses();
bookedRecord.setBookedSeats(1);
bookedRecord.setBookedBusState(BookedBusState.LOADING);
      bookedRecord.setBus(busService.getBusByPlateNumber(booking.getPlateNumber()));
bookedRecord.setRoute(booking.getRoute());
 infoLogger.info("GETTING DRIVER ID ======= " +    booking.getDriver().getId());
 Drivers drivers = new Drivers(booking.getDriver().getId());
 List<BookedBuses> d_bu = new ArrayList<>();
 drivers.setBooked(d_bu);
 drivers.addBooked(bookedRecord);
 bookedRecord.setDriver(drivers);
 bookedBusService.save(bookedRecord);

我的BookBusService按要求保存方法

@Autowired
private BookedBusRepository bookedBusRepo;
public boolean save(BookedBuses bookedRecord) {
    try {
        bookedBusRepo.save(bookedRecord);
        return true;
    } catch (DataIntegrityViolationException ex) {
        System.out.println(ex);
        AppConfig.LOGGER.error(ex);
        return false;
        // Log error message
    }
}

Answer 1

这条线

驱动程序=新的驱动程序（booking.get驱动程序（）. getId（））；

如果您已经拥有可用的驱动程序ID，则无需再次从数据库中提取驱动程序ID。

删除@OneToMore中的Cascade属性后

@Entity
@Table(name = "bookedBuses")
public class BookedBuses implements Serializable {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;
`
    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "driver_id")
    private Drivers driver;
}

@Entity
public class Drivers implements Serializable {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    @OneToMany(fetch = FetchType.LAZY)
    @JoinColumn(name = "driver_id")
    private List<BookedBuses> bookedBus;
}

Answer 2

首先，你在命名上有些混乱：你有驱动程序

private Drivers driver;

还可以选择如下变量名：

BookedBuses bookedRecord = new BookedBuses();

会造成很多混乱。不要混合复数

private List<BookedBuses> bookedBus;

应该是这样的：

private List<BookedBus> bookedBuses;

（我亦会否要求更改你的类别名称BookedBuse-

不管怎样，实际问题似乎就在这里：

Drivers drivers = new Drivers(booking.getDriver().getId());

您需要在repository的帮助下按id获取现有实体，而不是创建id为existing的新实体。比如：

Drivers drivers = driverRepo.findOne(booking.getDriver().getId()); // or findById(..)

似乎您有一个构造函数（您没有显示），可以创建一个id为的驱动程序。如果没有管理，则认为它是分离的。（您还有drivers.addBooked（bookedRecord） 你没有分享，但可能很琐碎）

请注意，一些帖子建议更改cascade类型。所有到级联类型。合并这是否有效取决于您的需要。Spring数据可以在save（…）上进行一些合并 基于实体id，但在本例中不一定如此。