Discussion :

[tyger]

bonsoir,
s'il vous plait je besoin d'aide
je crée une petite application avec netbeans, dans cette application l'utilisateur devra saisir son login et son password pour accéder à la suite.
mon problème est le suivant :
j'ai créer l'interface de connexion avec netbeans, j'ai aussi créer la base de données mais quand saisie le login et le mot de passe il me renvoi "mot de passe incorrect !" je ne sais pas ce qui ce passe.
svp quelqu'un peut m'aider help

voici mon code

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
String login = JTLogin.getText();
        String mdp = JPMdp.getPassword().toString();
 
        maConnexion.ObtenirConnexion();
        con = maConnexion.ObtenirConnexion();
 
       try
       {
           //création de la requete
           statement = con.prepareStatement("SELECT password FROM users WHERE login ='"+login+"' ");
           resultat = statement.executeQuery();
 
           if(resultat.next())
           {
               String motDePass = resultat.getString(1);
               if(motDePass.equals(mdp))
               {
                   JOptionPane.showMessageDialog(null, "Connexion reussie !", "SUCCESS", JOptionPane.PLAIN_MESSAGE);
                    RessourcesHumaine rh = new RessourcesHumaine();
                    rh.setVisible(true);
           }
           else
           {
               JOptionPane.showMessageDialog(null, "Mot de passe incorect !","Erreur",(1));
           }
           }
           else
           {
               JOptionPane.showMessageDialog(null, "Login incorrect !","Erreur",(1));
           }
 
           //Recuperation de la requete dans la variable
           resultat = statement.executeQuery();
 
          // con.close();
       }
       catch(SQLException e)
               {
                   System.out.println(e.getMessage());
               }

[Médinoc]

Je ne connais pas les DAO de Java, mais le GetString(1) me paraît suspect: Intuitivement, j'aurais plutôt mis GetString(0).

PS:

statement = con.prepareStatement("SELECT password FROM users WHERE login ='"+login+"' ")

Mauvais, ça: http://xkcd.com/327/

[MarieKisSlaJoue]

Il y à souvent des confusion avec le getString mais c'est bien le numéro de la colonne qui faut mettre. Et donc 1.

String getString(int columnIndex)
throws SQLException

Retrieves the value of the designated column in the current row of this ResultSet object as a String in the Java programming language.

Parameters:
columnIndex - the first column is 1, the second is 2, ...
Returns:
the column value; if the value is SQL NULL, the value returned is null
Throws:
SQLException - if the columnIndex is not valid; if a database access error occurs or this method is called on a closed result set

Ce qui me parait suspect moi c'est que nul par t une déclare le type de ta variable résultat. Enfin pas dans ce que tu nous montre en tous cas. Mais ça serai plus jolie si tu utilisais les balise code pour poster ton code sur le forum.

Sinon pour ton problème je te suggère d'utiliser le débuger de netbean pour pouvoir comparer les valeur retournés.

[Sennad]

Salut !

Comme l'a dit le panda , utilise le débuggeur de NetBeans, tu pourras alors voir quel est le retour de la requête, et quel est le mot de passe stocké (Entrée Utilisateur).
Tu pourras donc cibler un peu plus ton problème.
N'hésite pas à revenir vers nous.

[Sennad]

Salut,

As-tu résolu ton problème ?

Si oui pense au petit
Avec la solution tant qu'à faire !

Sinon nous sommes là pour t'aider ^^

[kolodz]

On ne fait jamais un select du mot de passe pour le vérifier !

Code SQL :

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

select 1 from users WHERE login = ? and password = ?

Code :

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

1
2
3
con.prepareStatement("select 1 from users WHERE login = ? and password = ?");
updateSales.setString(1, login);
updateSales.setString(2, password);

cf: Using Prepared Statements
Ainsi le mot de passe réel de l'utilisateur n'est jamais chargé dans l'application. La base de donnée n'envoie jamais l'information !
Après, le mot de passe stocké en clair, ce n'est pas une bonne idée non plus.

Cordialement,
Patrick Kolodziejczyk.

[Sennad]

Kolodz a raison, il ne faut pas faire de select sur tes mdp , et encore moins les stocker en clair dans ta DB.
Si tu souhaites avoir plus d'infos sur ce sujet, il y a un sujet très intéressant déjà ouvert ICI

Bon courage !

[Sennad]

Salut !

As-tu résolu ton problème ?

Penses au