Блог

У меня следующая конфигурация:

ProductPriceEntity

 @Entity
@Table(name = "PRODUCTPRICE")
public class ProductPriceEntity {

    ...

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "SUPERMARKET_STORE_ID", nullable = false)
    private SupermarketStoreEntity store;

    ...
}
  

ProductPriceNewRequest

 @Setter @Getter
public class ProductPriceNewRequest {

    ...

    private Long storeId;

    ...

}
  

ProductPriceControllerImpl

 @PostMapping
@ResponseStatus(HttpStatus.CREATED)
public ResponseEntity<ProductPriceResponse> save(@PathVariable(value = "product_id", required = true) Long productId, @RequestBody @Valid ProductPriceNewRequest productPriceNewRequest) {
    ProductEntity productEntity = productService.findById(productId);

    ProductPriceEntity productPriceEntity = modelMapper.map(productPriceNewRequest, ProductPriceEntity.class);

    productPriceEntity.setProduct(productEntity);
    productPriceEntity = service.insert(productPriceEntity);

    URI location = ServletUriComponentsBuilder.fromCurrentRequest()
            .path("/{id}")
            .buildAndExpand(productPriceEntity.getId())
            .toUri();


    ProductPriceResponse productPriceResponse = modelMapper.map(productPriceEntity, ProductPriceResponse.class);

    return ResponseEntity.created(location).body(productPriceResponse);

}
  

ProductPriceResponse

 @Getter @Setter
public class ProductPriceResponse {

    ...

    private String supermarket;
    private String store;

    ...
}
  

Это работает, но я не могу вернуть DTO. Супермаркет и магазин равны нулю для ProductPriceResponse.

Ну, тогда я изменил каскад отношений атрибутов хранилища.

 @ManyToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL)
@JoinColumn(name = "SUPERMARKET_STORE_ID", nullable = false)
private SupermarketStoreEntity store;
  

И я получил эту ошибку:

«отдельная сущность передана в persist: model.SupermarketStoreEntity; вложенным исключением является org.hibernate.PersistentObjectException: отдельная сущность передается в persist: model.SupermarketStoreEntity»

Это имеет смысл. Modelmapper преобразует длинный StoreID в SupermarketStoreEntity с единственным идентификатором и отсоединяется…

наконец, мой вопрос: какова наилучшая практика?

Должен ли я получать StoreID и не преобразовываться в отделенный от SupermarketStoreEntity и находить SupermarketStoreEntity с вашим StoreID в ProductPriceControllerImpl?

Или нет. Должен ли я удалить каскад и после сохранения ProductPriceEntity я должен получить сохраненный ProductPriceEntity? Я подозреваю, что магазин и супермаркет все равно получат null, потому что каскада там нет.

спасибо всем

Думаю, лучше всего ввести класс @Service, где у вас есть метод с аннотацией @Transactional . Служба внедряется в контроллер, и там происходит вся загрузка и сохранение объектов. Это должно помочь с ошибкой отсоединения.

Я не большой поклонник атрибута cascade и предпочел бы делать это в коде, но это скорее мнение.

Может быть, вы хотите взглянуть на https://bootify.io — вы можете определить свою базу данных с помощью REST api там и получить представление о лучших практиках для контроллеров и служб.:)