Discussion :

[nolookpass8]

Bonjour,

J'essaie de faire un système d'authentification, avec le code que j'ai le problème c'est que mon login et mon mdp sont toujours corrects que je rentre rien ou n'importe quoi. getText() est-elle la bonne méthode pour récupérer ce que l'user à rentré dans les champs ??

Voici mon code en espérant que vous pourrez m'aider :

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
 
 
private void buttonSeConnecterActionPerformed(java.awt.event.ActionEvent evt) {                                                  
        PreparedStatement test = null;
        String login = textFieldIdentifiant.getText();
        String mdp = passwordFieldMotDePasse.getText();
        ResultSet rs = null;
        String url = "jdbc:mysql://localhost/...";
        String driver = "sun.jdbc.odbc.JdbcOdbcDriver";// connexion driver jdbc
        String user = "root";
        String password = "...";
 
        try {
 
            Class.forName(driver);
            try (Connection connection = (Connection) DriverManager.getConnection(url, user, password)) {// connexion avec la bdd
                String sql = "SELECT login FROM UTILISATEUR WHERE login= ? AND mdp = ?";
                test = connection.prepareStatement(sql);
                test.setString(1, login);
                test.setString(2, mdp);
                rs = test.executeQuery();
                if (rs == null) {
                    System.out.println("Echec de la connexion");
                } else {
                    System.out.println("Connexion réussie");
                    setVisible(false);// fermer la frame courante auth
                    new accueil().setVisible(true);// ouvre la frame accueil
                }
            }
        } catch (ClassNotFoundException | SQLException cnfe) {
            System.err.println(cnfe);
        }
    }

[olivier.pitton]

C'est normal, ton test de vérification est incorrect.

La méthode executeQuery te renverra toujours un ResultSet, ou une exception, mais jamais null. Ce que tu dois tester c'est s'il y a au moins une ligne dans le ResultSet. S'il y a une ligne, cela signifie qu'il existe une entrée dans ta base contenant un login X pour un password Y, donc c'est bon. S'il n'y a aucune ligne, alors soit le login / password est faux, soit la ligne n'existe pas.

Essaye cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
String sql = "SELECT login FROM UTILISATEUR WHERE login= ? AND mdp = ?";
test = connection.prepareStatement(sql);
test.setString(1, login);
test.setString(2, mdp);
rs = test.executeQuery();
if (!rs.next()) {
  System.out.println("Echec de la connexion");
} else {
  setVisible(false);// fermer la frame courante auth
  new accueil().setVisible(true);// ouvre la frame accueil
}

PS : Pense à ne charger le Driver qu'une seule fois et pas à chaque clic. De plus, pense à fermer les ressources comme ResultSet et PreparedStatement.