Discussion :

[®om]

Salut,

Je souhaiterais avoir un objet qui n'est pas un EJB, de type Cart (caddie), qui contient une liste de Product (ejb) et qui est contenu dans un User.

Pour cela, j'ai déclaré Cart comme @Embeddable, et dans l'attribut Cart de User j'ai mis @Embedded. Jusque là, tout va bien.
Le problème, c'est que quand je récupère le Cart à partir du User, la liste des Product n'est pas chargée, et j'obtiens l'erreur suivante:

Code :

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

LazyInitializationException: failed to lazily initialize a collection of role: ejb.User.cart.products, no session or session was closed

Je ne sais pas comment résoudre le problème, apparemment un problème de session, mais bon je récupère mon User dans la servlet, et je fais un getProductsInCart() directement (ou alors un getCart().getProducts()), et ça me lance cette exception.

Voici le début de la classe Cart:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
@Embeddable
public class Cart implements Serializable {
 
    private static final long serialVersionUID = 1L;
 
    @OneToMany
    private List<Product> products;
 
    public Cart() {
        products = new ArrayList<Product>();
    }
    ...
}

Une idée?

[®om]

Je pense que ça devrait résoudre le problème, mais ayant une autre erreur actuellement, je ne suis pas sûr que ça fonctionne, je mettrai résolu si cela marche

Code :

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

@OneToMany(fetch=FetchType.EAGER)

Si qqn a mieux à proposer, je suis preneur

[xv-mnt]

Le fetch=eager va effectivement marcher, mais il faut savoir que tu vas systématiquement récupérer la liste, même si tu n'en n'as pas besoin.
Après, tout est une question d'optimisation.

Perso, je ne l'utilise que rarement. Lorsque j'ai besoin de la liste je fais un

Code :

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

myListe.size();

dans ma transaction, ce qui permet de retriever le contenu de ma liste.

[®om]

Le fetch=eager va effectivement marcher, mais il faut savoir que tu vas systématiquement récupérer la liste, même si tu n'en n'as pas besoin.
Après, tout est une question d'optimisation.

Perso, je ne l'utilise que rarement. Lorsque j'ai besoin de la liste je fais un

Code :

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

myListe.size();

dans ma transaction, ce qui permet de retriever le contenu de ma liste.

Pourquoi le .size() permet de retriever le contenu de la liste?

[xv-mnt]

oui, mais il faut le faire dans ta transaction, et non hors transaction car alors ton objet serait détaché et tu aurais une LazyLoadingException.

[martel]

@ rom : le .size() permet non pas de faire le retrieve mais bien de charger ta liste, pour le retrive c'est par ce qu'il le place dans une transaction.

[®om]

Je le fais dans une méthode d'une session facade... il faut en plus gérer une transaction?

[Kikito]

Sur ton serveur d'application tu es dans une transaction.
Ce qu'ils veulent dire c'est que le myListe.size(); doit être fait coté serveur d'application pas coté client sinon tu vas avoir une erreur.

Si tu mets ton FetchType à Eager tu vas toujours récupérer la liste, alors que tu peux laisser en lazy et charger quand tu en as besoin, par exemple dans ton facade :

Code :

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

1
2
3
4
5
6
7
8
getCartWithoutProducts(){
    return cart;
}
 
getCartWithProducts(){
    cart.products.size();
    return cart;
}