Discussion :

[momjunior]

Bonjour

J'ai une Class ConnexionForm qui récupère et valide les infos provenant de mon formulaire:

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package forms;
 
import java.util.HashMap;
import java.util.Map;
import javax.servlet.http.HttpServletRequest;
import beans.Candidat;
import dao.CandidatDao;
import org.jasypt.util.password.ConfigurablePasswordEncryptor;
 
public final class ConnexionForm {
 
private static final String CHAMP_EMAIL = "email";
private static final String CHAMP_PASS = "pwd";
 
private CandidatDao           candidatDao;
 
private String resultat;
private String              echec;
 
private Map<String, String> erreurs = new HashMap<String,String>();
 
 
private static final String ALGO_CHIFFREMENT = "SHA-256";
 
public ConnexionForm( CandidatDao candidatDao ) {
        this.candidatDao = candidatDao;
    }
 
public Map<String, String> getErreurs() {
return erreurs;
}
 
public String getEchec() {
        return echec;
    }
public String getResultat() {
return resultat;
}
 
public Candidat connecterCandidat( HttpServletRequest request ) {
/* Récupération des champs du formulaire */
String email = getValeurChamp( request, CHAMP_EMAIL );
String userPassword = getValeurChamp( request, CHAMP_PASS );
System.out.println("lemail est :"+email);
System.out.println("le password est :"+userPassword);
 
 
 
    Candidat candidat = new Candidat();
 
    /* Validation du champ email. */
try {
traiterEmail( email,userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_EMAIL, e.getMessage() );
}
candidat.setEmail( email );
/* Validation du champ mot de passe. */
try {
traiterPassword( userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_PASS, e.getMessage() );
}
candidat.setPassword( userPassword );
/* Initialisation du résultat global de la validation. */
if ( erreurs.isEmpty() ) {
 
 resultat = "Succès de la connexion.";
 
} else {
resultat = "<p class=\"resultat_echec\">Email ou mot de passe incorrect. Veuillez réessayer!</p>";
}
 
 
 
return candidat;
}
 
private void traiterPassword( String password, Candidat candidat ) {
        try {
            validationPassword( password );
        } catch ( FormValidationException e ) {
            setErreur( CHAMP_PASS, e.getMessage() );
 
    candidat.setPassword(password);
    }
}
private void traiterEmail( String email,String pwd, Candidat candidat ) {
        try {
            validationEmail( email,pwd );
        } catch ( FormValidationException e ) {
            setErreur( CHAMP_EMAIL, e.getMessage() );
        }
        candidat.setEmail( email );
    }
 
 
 
 
private void validationEmail( String email, String pwd ) throws FormValidationException
    {
        ConfigurablePasswordEncryptor passwordEncryptor = new    ConfigurablePasswordEncryptor();
    passwordEncryptor.setAlgorithm( ALGO_CHIFFREMENT );
    passwordEncryptor.setPlainDigest( false );
 
        if ( email != null )
        {
               if ( !email.matches("([^.@]+)(\\.[^.@]+)*@([^.@]+\\.)+([^.@]+)" ) )
               {
                throw new FormValidationException( "Merci de saisir une adresse mail valide." );
                }
               else
               {
                if(find(email)== null)
                 {
                  throw new FormValidationException( "L'adresse mail ou le mot de passe saisi est incorrect." );
                 }
                else
                 {
                  if( passwordEncryptor.checkPassword( pwd, find(email).getPassword() ))
                  {
                   if (find(email).getActivationEmail() != null)
                    {
 
                     if (find(email).getStatutActivation()== null)
                     {
                      throw new FormValidationException( "Votre compte n'a pas été activé." );
                     }
                    }
                   else
                    {
                        System.out.println("le statut du mail est: "+find(email).getActivationEmail());
                     throw new FormValidationException( "Votre adresse mail n'a pas encore été validée. Veuillez cliquer sur le lien qui vous a été envoyé par mail!" );
                    }
                  }
                  else
                  {
                   throw new FormValidationException( "L'adresse mail ou le mot de passe saisi est incorrect." );
                  }
                 }
               }
 
         }
         else
          {
                throw new FormValidationException( "Merci de saisir une adresse mail." );
          }
    }
 
private void validationPassword( String password ) throws FormValidationException {
        if ( password == null ) {
             throw new FormValidationException( "Merci d'entrer un mot de passe." );
        }
    }
 
 public Candidat find(String email) {
        Candidat u = null;    
 
        u = candidatDao.trouver(email);
 
        return u;
      }
/*
* Ajoute un message correspondant au champ spécifié à la map des
erreurs.
*/
private void setErreur( String champ, String message ) {
erreurs.put( champ, message );
}
 
/*
* Méthode utilitaire qui retourne null si un champ est vide, et son
contenu
* sinon.
*/
private static String getValeurChamp( HttpServletRequest request, String nomChamp ) {
String valeur = request.getParameter( nomChamp );
if ( valeur == null || valeur.trim().length() == 0 ) {
return null;
} else {
return valeur;
}
}
}

Et voici ma Servlet Connexion:

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package servlets;
 
import beans.Candidat;
import dao.CandidatDao;
import dao.DAOFactory;
import java.io.IOException;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;
import forms.ConnexionForm;
import javax.servlet.ServletContext;
import utility.EmailUtility;
 
 
public class Connexion extends HttpServlet {
 
public static final String CONF_DAO_FACTORY = "daofactory";   
public static final String ATT_CANDIDAT = "candidat";
public static final String ATT_MESSAGE   = "message";
public static final String ATT_FORM = "form";
public static final String SESSION_CANDIDATS ="candidats";
public static final String SUCCES = "/pages/profil.jsp";
public static final String ECHEC = "/pages/connexion.jsp";
 
 
 
 
private CandidatDao          candidatDao;
 
public void init() throws ServletException {
        /* Récupération d'une instance de notre DAO Utilisateur */
        this.candidatDao = ( (DAOFactory) getServletContext().getAttribute( CONF_DAO_FACTORY ) ).getCandidatDao();
    }
public void doGet( HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException
{
/* Affichage de la page de connexion */
this.getServletContext().getRequestDispatcher( ECHEC).forward( request, response );
}
public void doPost( HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException
{
/* Préparation de l'objet formulaire */
ConnexionForm form = new ConnexionForm(candidatDao);
/* Traitement de la requête et récupération du bean enrésultant */
Candidat candidat = form.connecterCandidat( request);
/* Récupération de la session depuis la requête */
HttpSession session = request.getSession();
 
 
if ( form.getErreurs().isEmpty() ) {
            session.setAttribute( SESSION_CANDIDATS, candidat );
            request.setAttribute( ATT_FORM, form );
        request.setAttribute( ATT_CANDIDAT, candidat );
            this.getServletContext().getRequestDispatcher( SUCCES ).forward( request, response );
        } else {
            session.setAttribute( SESSION_CANDIDATS, null );
            request.setAttribute( ATT_FORM, form );
        request.setAttribute( ATT_CANDIDAT, candidat );
            this.getServletContext().getRequestDispatcher( ECHEC ).forward( request, response );
        }
 
}
}

Voici une partie de ma page index.jsp:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<div class="indent-x">
            <span class="spana">Bienvenue,</span> <span class="spanb">${sessionScope.candidats.nom} ${sessionScope.candidats.prenom}</span>
            <table>
             <tr>
                   <td>
                                   <span class="spanc"><a href="">Modifier votre compte,</a></span>
                   </td>
                   <td>
                         <form method="post" action="<c:url value="/deconnexion"/>"><input  type="submit" value="Déconnexion" class="bouton" /></form>
                   </td> 
              </tr>
            </table>
        </div>

Le problème est que

Code :

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

sessionScope.candidats.nom

et

Code :

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

sessionScope.candidats.prenom

sont vides c'est-à-dire que le nom et le prénom ne s'affichent pas.

Que faire?

Merci

[jeffray03]

Salut,
pourquoi est ce que candidat aurait-il un nom et un prenom,
je ne vois pas ou est ce que tu les sauvegarde et aussi es-tu sur que candidatDao n´est pas null?

Eric

[tchize_]

Je ne vois nulle part dans ton code où tu attribue une valeur à candidat.nom ou candidat.prenom....

[momjunior]

Oui c'est vrai, je ne m'en suis pas rendu compte. J'ai donc modifié ma Class ConnexionForm:

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
 
public ConnexionForm( CandidatDao candidatDao ) {
...
public Candidat connecterCandidat( HttpServletRequest request ) {
/* Récupération des champs du formulaire */
String email = getValeurChamp( request, CHAMP_EMAIL );
String userPassword = getValeurChamp( request, CHAMP_PASS );
System.out.println("lemail est :"+email);
System.out.println("le password est :"+userPassword);
 
 
 
    Candidat candidat = new Candidat();
 
    /* Validation du champ email. */
try {
traiterEmail( email,userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_EMAIL, e.getMessage() );
}
candidat.setEmail( email );
/* Validation du champ mot de passe. */
try {
traiterPassword( userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_PASS, e.getMessage() );
}
candidat.setPassword( userPassword );
/* Initialisation du résultat global de la validation. */
if ( erreurs.isEmpty() ) {
 
    candidat = candidatDao.trouver(email);
 resultat = "Succès de la connexion.";
 
} else {
resultat = "<p class=\"resultat_echec\">Email ou mot de passe incorrect. Veuillez réessayer!</p>";
}
 
return candidat;
}
 
...
}

Et voici ma méthode "trouver" et la méthode "map" :

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
public class CandidatDaoImpl implements CandidatDao {
private static final String SQL_SELECT_PAR_EMAIL = "SELECT * FROM Candidat WHERE email = ? ";
...
@Override
    public Candidat trouver( String email) throws DAOException {
        Connection connexion = null;
        PreparedStatement preparedStatement = null;
        ResultSet resultSet = null;
        Candidat candidat = null;
 
        try {
            /* Récupération d'une connexion depuis la Factory */
            connexion = daoFactory.getConnection();
            /*
             * Préparation de la requête avec les objets passés en arguments
             * (ici, uniquement un id) et exécution.
             */
            preparedStatement = initialisationRequetePreparee( connexion, SQL_SELECT_PAR_EMAIL, false, email );
            resultSet = preparedStatement.executeQuery();
            /* Parcours de la ligne de données retournée dans le ResultSet */
            if ( resultSet.next() ) {
                candidat = map( resultSet );
            }
        } catch ( SQLException e ) {
            throw new DAOException( e );
        } finally {
            fermeturesSilencieuses( resultSet, preparedStatement, connexion );
        }
 
        return candidat;
    }
 
private static Candidat map( ResultSet resultSet ) throws SQLException {
        Candidat candidat = new Candidat();
        candidat.setId( resultSet.getLong( "id_candidat" ) );
        candidat.setStatutActivation(resultSet.getString("statut_activation"));        
        candidat.setCivilite( resultSet.getString( "civilite" ) );
        candidat.setNom( resultSet.getString( "nom" ) );
        candidat.setPrenom( resultSet.getString( "prenom" ) );
        candidat.setJourNaiss(resultSet.getString( "jour_naiss" ));
        candidat.setMoisNaiss(resultSet.getString( "mois_naiss" ));
        candidat.setAnneeNaiss(resultSet.getString( "annee_naiss" ));
        candidat.setPays(resultSet.getString( "pays" ));        
        candidat.setAdresse( resultSet.getString( "adresse" ) );
        candidat.setCodePostal(resultSet.getString( "code_postal" ));
        candidat.setTelephone( resultSet.getString( "telephone" ) );
        candidat.setAutreTelephone(resultSet.getString( "autre_telephone" ));
        candidat.setEmail( resultSet.getString( "email" ) );
        candidat.setActivationEmail("validation_email");
        candidat.setPassword(resultSet.getString( "password" ));
        candidat.setSituationMatr(resultSet.getString( "situation_matr" ));
        candidat.setDispo(resultSet.getString( "dispo" ));
        candidat.setDomaine(resultSet.getString( "domaine" ));
        candidat.setProfil(resultSet.getString( "profil" ));
        candidat.setAutreDomaine(resultSet.getString( "autre_domaine" ));
        candidat.setTypeContrat(resultSet.getString("type_contrat"));
        candidat.setCv( resultSet.getString( "cv" ) );
        candidat.setDateInscription(resultSet.getTimestamp("date_inscription"));
        return candidat;
    }
 
...
}

Normallement quand je fais ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
if ( erreurs.isEmpty() ) {
 
    candidat = candidatDao.trouver(email);
 resultat = "Succès de la connexion.";
 
} else {
resultat = "<p class=\"resultat_echec\">Email ou mot de passe incorrect. Veuillez réessayer!</p>";
}
 
return candidat;
}

candidat ne devrait pas être vide puisque dans la méthode "trouver", la méthode "map" récupère la valeur des champs de la base de données.

Qu'est ce qui cloche?

[tchize_]

ben commence à afficher dans ta méthode ce que tu met dans ton candidat, ça permettra de savoir si ton erreur est dans le DAO ou ailleurs

[momjunior]

En faisant ceci par exemple?

Code :

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

1
2
3
4
5
if ( resultSet.next() ) {
                candidat = map( resultSet );
 
            }
            System.out.println("la civilité est :"+candidat.getCivilite());

Ceci m'affiche:

la civilité est :null
la civilité est :null
la civilité est :null
la civilité est :null
la civilité est :null

[tchize_]

Bon ben voilà, c'est ta méthode map qui est erronée. En l'occurence, je suppose qu'elle demande des colonnes que le select * n'a pas retournées. Pour connaitre les noms des colonnes retournées, regarde dans resultSet.getMetadata()

[momjunior]

J'ai inséré le code suivant dans le but d'afficher les noms de colonne:

Code :

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

1
2
3
4
5
6
7
ResultSetMetaData metadata = resultSet.getMetaData();
   String[] noms = new String[metadata.getColumnCount()];
   for(int i = 0; i < noms.length; i++){
      String nomColonne = metadata.getColumnName(i+1);
      noms[i] = nomColonne;
   }
            System.out.println("les noms de colonne sont :"+noms);

J'ai le résultat suivant:

Code :

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

1
2
3
4
5
les noms de colonne sont :[Ljava.lang.String;@1e79bd1
les noms de colonne sont :[Ljava.lang.String;@1b84acc
les noms de colonne sont :[Ljava.lang.String;@f97c90
les noms de colonne sont :[Ljava.lang.String;@1e31c2c
les noms de colonne sont :[Ljava.lang.String;@136a302

Je me pose aussi une quetion: pourquoi m'affiche t-il 5 lignes alors que ma requete est la suivante:

Code :

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

private static final String SQL_SELECT_PAR_EMAIL = "SELECT * FROM Candidat WHERE email = ? ";

Merci

[tchize_]

Afficher le contenu du tableau de String serait peut être un chouilla plus utile qu'afficher juste que c'est un tableau de string. Ca on le sait en lisant le code


Et vu que je ne sais pas où tu as mis ce bout de code, impossible de te dire pourquoi il s'affiche 5 fois

[momjunior]

Oui t'as raison, je ne sais pas où j'avais la tête. Voici donc le code:

Code :

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

1
2
3
4
5
for(int i = 0; i < noms.length; i++){
      String nomColonne = metadata.getColumnName(i+1);
      noms[i] = nomColonne;
      System.out.println("les noms de colonne sont :"+noms[i]);
   }

J'ai donc ce résultat:

les noms de colonne sont :id_candidat
les noms de colonne sont :statut_activation
les noms de colonne sont :civilite
les noms de colonne sont :nom
les noms de colonne sont :prenom
les noms de colonne sont :id_candidat
les noms de colonne sont :statut_activation
les noms de colonne sont :civilite
les noms de colonne sont :nom
les noms de colonne sont :prenom
les noms de colonne sont :id_candidat
les noms de colonne sont :statut_activation
les noms de colonne sont :civilite
les noms de colonne sont :nom
les noms de colonne sont :prenom
les noms de colonne sont :id_candidat
les noms de colonne sont :statut_activation
les noms de colonne sont :civilite
les noms de colonne sont :nom
les noms de colonne sont :prenom
les noms de colonne sont :id_candidat
les noms de colonne sont :statut_activation
les noms de colonne sont :civilite
les noms de colonne sont :nom
les noms de colonne sont :prenom

Donc les noms de colonne s'affichent bien. C'est quoi la suite?

Je tiens aussi à signaler que cette liste s'affiche toujours 5 fois, je ne sais toujours pas pourquoi

[tchize_]

Ok, donc les id de colonne sont bons. Donc la seule raison que je voie pour laquelle les données sont vides dans ton candidat, c'est que les colonne en question sont vides dans la base de données.

[momjunior]

Les colonnes ne sont pas vides dans la base de données.

Le problème est surement à ce niveau:

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
public Candidat connecterCandidat( HttpServletRequest request ) {
/* Récupération des champs du formulaire */
String email = getValeurChamp( request, CHAMP_EMAIL );
String userPassword = getValeurChamp( request, CHAMP_PASS );
System.out.println("lemail est :"+email);
System.out.println("le password est :"+userPassword);
 
 
 
    Candidat candidat = new Candidat();
 
    /* Validation du champ email. */
try {
traiterEmail( email,userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_EMAIL, e.getMessage() );
}
candidat.setEmail( email );
/* Validation du champ mot de passe. */
try {
traiterPassword( userPassword, candidat );
} catch ( Exception e ) {
setErreur( CHAMP_PASS, e.getMessage() );
}
candidat.setPassword( userPassword );
/* Initialisation du résultat global de la validation. */
if ( erreurs.isEmpty() ) {
 
    candidat = candidatDao.trouver(email);
 resultat = "Succès de la connexion.";
 
} else {
resultat = "<p class=\"resultat_echec\">Email ou mot de passe incorrect. Veuillez réessayer!</p>";
}
 
return candidat;
}

Qu'en penses-tu?

[tchize_]

Pour ce que je vois du code, si les valeurs sont nulles, c'est que ta méthode map met du null dedans, donc affiche dans map ce que tu as mis dans Candidat, si tu pense que ce n'est pas le cas. A la fin de

Code :

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

private static Candidat map( ResultSet resultSet ) throws SQLException

avant le return, affiche tous les champs de Candidat avec leur valeur.

[momjunior]

Finalement je ne sais pas ce qui s'est passé mais ça marche. J'arrive à récupérer les infos de l'utilisateur dans la session. Mais je ne sais toujours pas ce qui clochait, et c'est bien dommage car si le problème se pointe à nouveau je serais bien embeté...

Merci beaucoup pour votre aide.