Discussion :

[jean.dufour]

Salut

J'ai ces deux classes

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
@Entity
public class AccountOperation {
 
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long accountOperationId;
 
    @ManyToOne
    @JoinColumn(name = "lodger_id")
    private Lodger lodger;
    private BigDecimal balanceAccountValue;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
@Entity
public class Lodger implements Serializable {
 
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long lodgerId;
 
    @OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "lodger", orphanRemoval = true)
    private List<AccountOperation> accountOperationList;
 
}

Je voudrais aller chercher la dernière opération pour chaque lodger

en Sql je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
select * from account_operation ao1
where ao1.account_operation_id in (
select max(ao2.account_operation_id) as maxAoId 
from account_operation ao2 
)

In a table account_operation, I have columns: balance, operation_value, transaction_type, transaction_date, lodger_id

AVec les données suivant

balance, lodger_id
20 , 20, 1
10 , 30, 1
50 , 50, 2
70 , 20, 2
60 , 10, 2

j'aurais

balance, operation_value, lodger_id
10 , 30, 1
60 , 10, 2

En jpa,

j'ai réussi avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

select ao1 from AccountOperation ao1 where ao1.accountOperationId  in (select max(ao2.accountOperationId) from AccountOperation ao2)

Pourquoi ça fonctionne alors que je n'ai pas fait de jointure?

je me demande si on peut faire mieux?

[OButterlin]

Ben si, JPA l'a faite grâce au mapping, c'est un principe de base.

Pour ce qui est de faire mieux, c'est discutable, ça dépend un peu de ce que tu fais après.
Ce que tu as codé a le mérite de limiter les lectures... (perso, c'est ce que j'aurais fait pour récupérer uniquement la dernière opération)

[jean.dufour]

Ben si, JPA l'a faite grâce au mapping, c'est un principe de base.

Pour ce qui est de faire mieux, c'est discutable, ça dépend un peu de ce que tu fais après.
Ce que tu as codé a le mérite de limiter les lectures... (perso, c'est ce que j'aurais fait pour récupérer uniquement la dernière opération)

ok mais pourquoi je n'ai pas besoin de mettre de jointure dans la requête?

[tchize_]

Ce code te retourne la dernière opération de ta base. Pas la dernière opération pour chaque lodge. Pas besoin de jointure pour retourner 1 élément. D'ailleurs il n'y a pas de jointure dans ton sql non plus....

[OButterlin]

ok mais pourquoi je n'ai pas besoin de mettre de jointure dans la requête?

Le JPAQL n'est pas SQL, il fait des choses pour toi en fonction du mapping de tes entities.
Dans ton exemple, comme le dit tchize_, il n'y a pas de jointure des 2 tables et tant que tu n'accèdes pas à une propriété du "sous-objet", elle ne se fera pas (en lazy loading).
Pour faire une jointure explicite dans une requête, il faudrait passer par un "join fetch" sur la propriété

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
select a from AccountOperation a join fetch a.lodger

Là, l'objet "lodger" sera initialisé

[jean.dufour]

Le JPAQL n'est pas SQL, il fait des choses pour toi en fonction du mapping de tes entities.
Dans ton exemple, comme le dit tchize_, il n'y a pas de jointure des 2 tables et tant que tu n'accèdes pas à une propriété du "sous-objet", elle ne se fera pas (en lazy loading).
Pour faire une jointure explicite dans une requête, il faudrait passer par un "join fetch" sur la propriété

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
select a from AccountOperation a join fetch a.lodger

Là, l'objet "lodger" sera initialisé

@tchize_ a raison, la requête n'est pas bonne, elle ne retourne pas la dernière transaction pour chaque lodger

en sql je ferais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
select * from account_operation ao1
where ao1.account_operation_id in (
select max(ao2.account_operation_id) as maxAoId 
from account_operation ao2 group by ao2.lodger_id
)

pour le jpa

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

select ao1 from AccountOperation ao1 join fetch ao1.lodger where ao1.accountOperationId  in (select max(ao2.accountOperationId) from AccountOperation ao2 group by ao2.lodger.id)

ça fonctionne