DTO-запрос с JPA CriteriaQuery и Hibernate с использованием сущности в качестве конструктора DTO создает много вариантов выбора

Question

hibernate 5.2.10.Final

jpa 2.1

Я хочу сопоставить запрос проекции с DTO (объект передачи данных)с JPA Criteria Query и Hibernate.Я указываю конструктор, который будет применяться к результатам выполнения запроса.

Если конструктор предназначен для всего класса сущностей, у меня есть несколько вариантов выбора вместо одного (это длительный процесс для тысяч записей),Если конструктор предназначен для набора параметров сущности, то я вижу только один выбор в консоли.Я не могу понять, где я ошибся или это ошибка?

public class ServiceDAO {

      public List<ServicesDTO> getAllServicesByFilter(ServicesFilter filter) {

          CriteriaBuilder cb = entityManager.getCriteriaBuilder();
          CriteriaQuery<ServicesDTO> criteria = cb.createQuery(ServicesDTO.class);
          Root<ServicesEntity> serviceEntity = criteria.from(ServicesEntity.class);

          // here is only one select to get list of services    
          criteria.select(cb.construct(ServicesDTO.class, serviceEntity.get("active"), serviceEntity.get("providerId"), serviceEntity.get("serviceId")));

          // in this case I have multiple selects
          //criteria.select(cb.construct(ServicesDTO.class, serviceEntity)); 
          if(filter != null) {
             List<Predicate> pcl = new ArrayList<Predicate>();

             if(filter.getActive() != null)
                pcl.add(cb.equal(serviceEntity.get("active"), filter.getActive()));
             if(filter.getProviderId() != null)
                pcl.add(cb.equal(serviceEntity.get("providerId"), filter.getProviderId()));
             if(filter.getServiceId() != null)
                pcl.add(cb.equal(serviceEntity.get("serviceId"), filter.getServiceId()));

             criteria.where(pcl.toArray(new Predicate[pcl.size()]));
          }

          return entityManager.createQuery(criteria).getResultList();
      }

}

-

public class ServicesDTO implements Serializable {

    private static final long serialVersionUID = 1L;

    private Boolean active;
    private Integer providerId;
    private Integer serviceId;

    public ServicesDTO() {}

    public ServicesDTO(Boolean active, String providerId, Integer serviceId) {

         this.active = active;
         this.providerId = Integer.parseInt(providerId);
         this.serviceId = serviceId;
    }


    public ServicesDTO(ServicesEntity service) {

       if(service != null) {
           this.active = service.isActive();
           this.providerId = Integer.parseInt(service.getProviderId());
           this.serviceId = service.getServiceId();
       }

    // getters & setters

}

-

@Entity
@Table
public class ServicesEntity {

   @Id
   @Column(name = "id", unique = true)
   @GeneratedValue(strategy=GenerationType.IDENTITY)
   private Long id;

   @Column(name = "serviceId", nullable = false)
   private int serviceId;

   @Column(nullable = false)
   private String providerId;

   @ManyToOne(fetch=FetchType.LAZY)
   @JoinColumn(name="categoryId")
   private Categories categoryId;

   private boolean active;

   @OneToMany(fetch = FetchType.LAZY, mappedBy = "service", cascade = CascadeType.ALL)
   private List<Service_Area_Ref> areas = new ArrayList<Service_Area_Ref>();

   @ManyToOne(fetch=FetchType.LAZY, optional = true)
   @JoinColumn(name="parentCatId")
   private Categories parentCatId;

   public ServicesEntity() {}

   public ServicesEntity(int serviceId) {
       this.serviceId = serviceId;
   }

   // getters & setters

   // equals & hashcode

}

Answer 1

Да, так и есть.Вероятно, не так много вариантов использования для этого.Учитывая

@Entity
public class A {
    @Id @GeneratedValue(strategy=GenerationType.IDENTITY)
    private Integer id;
    private Integer value;

public class ADto {
    private Integer va;

    public ADto(A a) {
        this.va = a.getValue();
    }
    public ADto(Integer va) {
        this.va = va;
    }

Тогда

tx.begin();

A a1 = new A();
a1.setValue(1);
A a2 = new A();
a1.setValue(2);
em.persist(a1);
em.persist(a2);

tx.commit();

em.clear();

System.out.println("As usual");
em.createQuery("select new dto.ADto(a.value) from A a where a.value <= 2", ADto.class).getResultList();

System.out.println("As A");
em.createQuery("select new dto.ADto(a) from A a where a.value <= 2", ADto.class).getResultList();

дает вам

create table A (id integer generated by default as identity (start with 1), value integer, primary key (id))
create table B (id integer generated by default as identity (start with 1), value integer, primary key (id))
insert into A (id, value) values (default, ?)
insert into A (id, value) values (default, ?)
As usual
select a0_.value as col_0_0_ from A a0_ where a0_.value<=2
As A
select a0_.id as col_0_0_ from A a0_ where a0_.value<=2
select a0_.id as id1_0_0_, a0_.value as value2_0_0_ from A a0_ where a0_.id=?
select a0_.id as id1_0_0_, a0_.value as value2_0_0_ from A a0_ where a0_.id=?

И вам не нравится тот факт, что объект A выбирается каждый раз для нового ADtoпример.Вероятно, это так, потому что вы могли бы создать DTO с несколькими объектами, а не только с A, такими как A, B и C, и как бы JPA / Hibernate сделал это удобно в одном операторе select?Хотя он может выбрать все атрибуты, а затем отслеживать, какие атрибуты принадлежат каким объектам, а затем создать их и передать их в DTO, чтобы вы могли деконструировать их, что кажется редкой работой.Вероятно, это будет более эффективно и лучше, если вы выберете нужные атрибуты и сделаете из них конструктор, как в первом случае.