Discussion :

[enatchia]

Bonjour tout le monde j'aimerais que vous m'aidiez à résoudre ce problème j'ai une interface qui va me permettre de changer le mot de passe de l'utilisateur et voici mon code qui me dit que la colonne loginutilisateur n'existe pas.

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
private void jButtonValiderActionPerformed(java.awt.event.ActionEvent evt) {                                               
        // TODO add your handling code here:
         boolean verif = false;
        try {
            Statement statement;
            statement = Maconnexion.seconnecter().createStatement();
            ResultSet rs = statement.executeQuery("select * from profilUtilisateur");
            while (rs.next()) {
                String loginUtilisateur = rs.getString("loginUtilisateur");
                String loginChanger = jTextFieldLoginChanger.getText();
                char[] ancienmotPasse = jPasswordFieldAncienMotPasse.getPassword();
                String motPasseUtilisateur = rs.getString("motPasseUtilisateur");
                String ancienmotpasse = String.valueOf(ancienmotPasse);
                    if (loginUtilisateur.equals(loginChanger) && motPasseUtilisateur.equals(ancienmotpasse) ) { 
                    verif = true;
 
            ResultSet rest = statement.executeQuery("UPDATE profilUtilisateur SET motPasseUtilisateur = '"+String.valueOf(jPasswordFieldNouveauMotPasse.getPassword())+"',confirmerMotPasseUtilisateur = '"+String.valueOf(jPasswordFieldConfirmerMotPasse.getPassword())+"' WHERE loginUtilisateur = "+jTextFieldLoginChanger.getText()+"");
            while (rest.next()) {
                    ConnexionJFrame connexionJF = new ConnexionJFrame();
                    connexionJF.setVisible(true);
                    this.dispose();
             }
              }
            }
               if (verif==false) {
                JOptionPane.showMessageDialog(null, "Verifier le login ou le Mot de passe", "Message d'erreur:", JOptionPane.ERROR_MESSAGE);
                jTextFieldLoginChanger.setText("");
                jPasswordFieldAncienMotPasse.setText("");
            }
        } catch (SQLException ex) {
            Logger.getLogger(ChangerMotPasseJFrame.class.getName()).log(Level.SEVERE, null, ex);
        }
 catch (Exception e) {
    e.printStackTrace();
}   
 
    }

c'est le formulais et le code est derière le bouton Valider

[joel.drigo]

Salut,

C'est quoi le message d'erreur exact ? Parce que s'il te dit que ta colonne n'existe pas, c'est qu'elle n'existe pas (il y a une différence dans le nom, peut-être juste dans la casse). Mais n'est-ce pas un autre problème, dû par exemple au fait qu'il manque les quotes autour de la valeur de loginUtilisateur dans l'update ? D'ailleurs, tu devrais utiliser plutôt un PreparedStatement, ce qui t'éviterais, entre autre (voir injection SQL par exemple), d'avoir à te soucier des quotes.

[Maine13]

Bonjour,

Je ne sais pkoi tu n'envoi pas directement la requête:

Code :

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

1
2
 
Select * from profilUtilisateur where login='"+login+"' and password='"+password+"'

Mais sinon je te conseille de récupérer le résultat d'une valeur avec :

Code :

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

1
2
 
String login=rs.getString(2);// Le 2 signifie l'indice du login dans le résultat de ta requête

[OButterlin]

Code :

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

1
2
 
Select * from profilUtilisateur where login='"+login+"' and password='"+password+"'

Tu n'as jamais entendu parler d'attaque par injection ?
Il faut prendre l'habitude d'utiliser les PreparedStatement et surtout pas construire la chaîne de requête en mettant les valeurs.

Code :

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

1
2
3
4
5
6
 
PreparedStatement pstmt = connection.prepareStatement("select * from profilUtilisateur where login = ? and password = ?");
pstmt.setString(1, login);
pstmt.setString(2, password);
ResultSet rs = pstmt.executeQuery();
...

[Maine13]

Oui bien sur je confirme