Discussion :

[sophiaa]

saluut!

Je travaille avec JPA,JSf, et j'ai un petit problème: quand je lance ce petit programme, l'exception NullPointerException se lance: voici le prog:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
package controller;
 
import java.util.ArrayList;
import java.util.List;
import javax.annotation.PostConstruct;
import javax.faces.bean.ApplicationScoped;
import javax.faces.bean.ManagedBean;
 
import entities.Membre;
import javax.ejb.EJB;
import model.MembreFacade;
 
@ManagedBean(name = "themeService", eager = true)
@ApplicationScoped
public class ThemeService {
 
    @EJB
    private MembreFacade membreFacade;
 
    private List<Membre> membres;
 
    @PostConstruct
    public void init() {
        membres = new ArrayList<>();
        membres = null;
        try {
            membres = membreFacade.findAll();
 
        } catch (NullPointerException e) {
        }
    }
 
    public List<Membre> getMembres() {
        return membres;
    }
 
    public static void main(String args[]) throws NullPointerException{
        ThemeService t = new ThemeService();
        System.out.println(t.membres.size());
 
    }
 
}

Que faire svp, j'ai vraiment besoin d'aide, j'ai cherché partt mais j'arrive a résoudre mon prob
Merci bcp d'avance

[habibouss]

Salut,

Pouvez-vous être plus explicite dans le post. Sinon ça va être un peut compliqué de vous donner un coup de main.

Je veux dire le NullPointerException est généré quand? Par quelle fonction (ou méthode)?


Best regards!

[eulbobo]

Code :

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

1
2
3
4
5
 
    public static void main(String args[]) throws NullPointerException{
        ThemeService t = new ThemeService();
        System.out.println(t.membres.size());
    }

Dans ton main, tu appelles membres juste après la création de ton objet ThemeService en faisant un new.
Du coup, tu ne profites pas de l'injection ni du fonctionnement de base des EJB avec les annotations (c'est magique, mais faut les faire passer par la boîte magique avant)
Du coup "membres" est null
Du coup, paf quand tu appelles .size() dessus


Ha, et vire ce gros moche catch(NullPointerException e) : un NullPointer est quelque chose qu'on évite en testant la valeur de l'objet avant de l'utiliser

[facilus68]

Bonjour,

Le probleme n'est pas liée au framework jsf.

L'exception est levéé en raison de la liste membres qui est null

dans ta méthode init(); tu crée une nouvelle liste de membres ensuite tu la remet à null, et quand tu esseye de recuperer la taille de ta liste sa te fera cette exception

supprime la ligne dans init :

Code :

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

membres = null;

peut etre tu peux faire :

Code :

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

1
2
3
4
if(membre == null){
   membres = new ArrayList<>();
   membres = membreFacade.findAll();
}

ta liste est chargé une seule fois

[eulbobo]

Non...

C'est juste qu'elle fait un new sans passer par l'injection de dépendance

[kolodz]

Si membreFacade n'est pas injecté, il aura effectifivement une erreur lancé. Mais celle-ci est catché par le try/catch mise en place.
Comme le dit facilus68, si il supprime son affectation membres = null;. Il aura dans membres une liste vide et non un null.

Cependant, il est aussi possible que membreFacade.findAll() retourne lui-même la valeur null. Peu probable, mais pas impossible.

Il y a deux codes qui me semblerai bon en fonction du comportement voulu.

On considère qu'on peut fonctionner sans la liste des membres fournit par la facade :

Code :

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

1
2
3
4
5
6
7
8
9
10
@PostConstruct
    public void init() {
        if(membreFacade!=null) {
            membres = membreFacade.findAll();
        }
        // Quoi qu'il arrive, on s'assure que members n'est pas null.
        if(membres !=null) {
            membres = new ArrayList<>();
        }
    }

On considère qu'on ne peut pas fonctionner sans la liste des membres fournit par la facade :

Code :

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

1
2
3
4
5
6
7
8
@PostConstruct
    public void init() {
        if(membreFacade!=null) {
             membres = membreFacade.findAll();// On suppose dans ce cas que findAll() retourne toujours quelque chose de non null.
        } else {
             throw new ConfigurationException("Initialisation impossible : Le membre membreFacade n'as pas été injecté.")
        }
    }

Cordialement,
Patrick kolodziejczyk.

[OButterlin]

eulbobo a raison, le problème vient du fait que ce n'est pas le conteneur qui va créer l'instance de la classe ThemeService, du coup, inutile de croire que l'injection de dépendance va se faire ou que l'appel de la méthode init() liée à @PostConstruct va être appelée

Donc, pour tester cette classe, ce n'est pas via une méthode main(...) lancée en mode ligne de commande qu'on procède, le plus simple serait de la tester via un contrôleur JSF tous simplement.
Sinon, tu as la possibilité de définir ton bean comme étant chargé dès le démarrage de l'application par eager, comme ceci

Code :

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

@ManagedBean(eager=true)