Problème d'instanciation d'un bean quand appel d'un DAO dans son constructeur

Version imprimable

Voir 40 message(s) de cette discussion en une page

20/10/2015, 06h40
tchize_

Tu peux poster ton log complet. Ton persistence.xml. Tes dao. Ton connexion pool.

3 pièce(s) jointe(s)

Mon fichier persistence:

Code:

1
2
3
4
5
6
7
8
9
10
<?xml version="1.0" encoding="UTF-8"?>
<persistence version="2.1" xmlns="http://xmlns.jcp.org/xml/ns/persistence" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://xmlns.jcp.org/xml/ns/persistence http://xmlns.jcp.org/xml/ns/persistence/persistence_2_1.xsd">
    <persistence-unit name="gesthair_PU">
        <jta-data-source>jdbc/gesthairpool</jta-data-source>
        <class>com.pat.gesthair.entities.Client</class>
        <class>com.pat.gesthair.entities.Essai</class>
        <class>com.pat.gesthair.entities.Prestation</class>
        <class>com.pat.gesthair.entities.Utilisateur</class>
    </persistence-unit>
</persistence>

Mon Bean à jour ainsi que mon DAO sont en pièces jointes.
Une copie du log aussi en pièce jointe.
Un conseil pour sortir un fichier de la console d'admistration Glassfish un fichier correspondant au connection pool?

20/10/2015, 21h23
tchize_

Rien de particulier à dire, mis à part ce que j'ai déjà dit: tu ne vérifie pas la valeur de retour après ton appel

Code:

client = rendezvousDao.trouverClient(rendezvous.getIdClient());

hors dans ton cas client est null, probablement parce que tu va rechercher un id incorrect. Autrement dit, t'as dans ta DB un rdv avec un idclient "1" mais tu n'as pas de client 1.

Quand à ton erreur RAR7113: , vu qu'elle a lieu 8 secondes avant je ne pense pas qu'elle soit liée. Je penserais plutot à une connexion que t'as oublié de fermer ailleurs et que glassfish tuerais après 10 secondes.

Je maintient qu'il ne faut pas faire un new

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    @EJB
    private RendezvousDAO     rendezvousDao;
    
    public ListerRendezvousBean()
    {
        rendezvousDao             = new RendezvousDAO();
        listeVide                 = true;
        listRendezvousComplet     = new ArrayList<RendezvousComplet>();
        
        Utilisateur utilisateur;
        FacesContext context = FacesContext.getCurrentInstance();
        HttpSession session = (HttpSession) context.getExternalContext().getSession(true);
        utilisateur = (Utilisateur) session.getAttribute(ATT_SESSION_USER);
        idUtilisateur = utilisateur.getId();
    }

Peux-tu au moins faire le test ?

21/10/2015, 09h44
PatLec

Le problème est résolu:
Je faisais appel à idClient qui avait été supprimé par ailleurs et comme je ne testais pas ce cas ...
Merci tchize_ pour cette bonne leçon que je retiens. Il me reste à vérifier l'ensemble de mon code car je dois bien avoir d'autres failles du même type.
On apprend de ses erreurs.
Merci encore
21/10/2015, 10h23
tchize_

Citation:

Envoyé par OButterlin

Je maintient qu'il ne faut pas faire un new

Je suis d'accord avec toi, bien qu'à part être moche et inutile, ça n'a aucun effet sur son code.
21/10/2015, 10h47
OButterlin

Citation:

Envoyé par tchize_

Je suis d'accord avec toi, bien qu'à part être moche et inutile, ça n'a aucun effet sur son code.

C'est vrai, le constructeur est appelé avant l'injection, mais je trouve que si déjà on utilise un EJB, autant l'utiliser correctement ;)

Voir 40 message(s) de cette discussion en une page