[debutant]problème avec une authentification

**sissi25** · 29/01/2007, 20h44

J'ai réalisé une authentification toute simple avec le login et le mot de passe et je voudrais mettre une condition si le login n'est pas dans la bdd

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
 public void Identif(String login, String pass) {
              Connection conn = null;
              String message = new String();
 
        try {         
                       Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
                       conn = DriverManager.getConnection("jdbc:odbc:BiorG","","");
                       System.out.println("Connexion établie avec BiorG");
                       Statement s = conn.createStatement ();
                       Statement s1 = conn.createStatement ();
                       s1.executeQuery("SELECT Utilisateurs.Login FROM Utilisateurs");
                       s.executeQuery ("SELECT Utilisateurs.Password, Utilisateurs.Login FROM Utilisateurs WHERE (Utilisateurs.Login)=\'"+login+"\'");
                       ResultSet rs = s.getResultSet ();
                       ResultSet rs1 = s1.getResultSet ();
                       while (rs1.next()){
                            String log = rs1.getString("Login");
                             System.out.println("fct" + log);
                           if (log.equals(login)){
                                  while (rs.next()){ 
                            String mdp = rs.getString ("Password");
 
                          if (mdp.equals(pass))
                          {
 
        message = "Connection réussie";
        JOptionPane.showMessageDialog(null,""+ message );
 
                        }
                       else { message = "Connection échouée";
        JOptionPane.showMessageDialog(null,""+ message );
 
                       }
                                  }
                                        rs1.close(); s1.close();rs.close(); s.close();         
        }
 
 
 
                       else{
                           message = "Le nom entré n'est pas dans la base de donnée";
        JOptionPane.showMessageDialog(null,""+ message );
                       }
                       }
        }
                          catch(Exception e) {System.err.println(e);}
                          finally { if (conn != null) {
                try {
                    conn.close(); System.out.println("Connexion terminée");}
                       catch(Exception e) {System.err.println ("Pb de fermeture de connexion ");}
                    }                
            }  
 
 
    }
 
 
}

Le problème c'est que log prend toutes les valeurs de la bdd. Ainsi si je met le 3eme nom de la base de donnée, cela me met que le nom n'est pas dans la base de donnée

Merci de votre aide

**sissi25** · 30/01/2007, 12h38

J'ai essayé comme dans le tutoriel d'utiliser la méthode execute qui renvoi true s'il y a un résultat et false sinon mais quand je met un login qui n'est pas dans la base de donnée ca me met rien au lieu de me mettre "Le nom n'est pas dans la base de donnée

Vous pouvez m'aider svp
merci

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
 
public void Identif(String login, String pass) {
              Connection conn = null;
              String message = new String();
              ResultSet resultSet = null;
 
        try {         
              Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
              conn = DriverManager.getConnection("jdbc:odbc:BiorG","","");
              System.out.println("Connexion établie avec BiorG");
              Statement s = conn.createStatement ();
              boolean rs =  s.execute ("SELECT Utilisateurs.Password, Utilisateurs.Login FROM Utilisateurs WHERE (Utilisateurs.Login)=\'"+login+"\'");
                if(rs){
                    resultSet = s.getResultSet();
 
 
                                  while (resultSet.next()){ 
                            String mdp = resultSet.getString ("Password");
                        String log = resultSet.getString("Login");
                             System.out.println("fct" + log);
                          if (mdp.equals(pass))
                          {
 
        message = "Connection réussie";
        JOptionPane.showMessageDialog(null,""+ message );
 
                        }
                       else { message = "Connection échouée";
        JOptionPane.showMessageDialog(null,""+ message );
 
                       }
                                  }
                                      resultSet.close(); s.close();         
        }
 
                else{
  message = "Le nom n'est pas dans la bdd";
        JOptionPane.showMessageDialog(null,""+ message );
   }             
 
        }        
 
 
                          catch(Exception e) {System.err.println(e);}
                          finally { if (conn != null) {
                try {
                    conn.close(); System.out.println("Connexion terminée");}
                       catch(Exception e) {System.err.println ("Pb de fermeture de connexion ");}
                    }                
            }  
 
 
    }

**schniouf** · 30/01/2007, 12h53

Dans ta boucle while ( resultset.next() ) il te manque une condition d'arrêt si tu as trouvé un login et un mot de passe.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
...
boolean trouve = false ;
while ( resultset.next() && !trouve)
{
   ...
   if ( resultset.getString("password").equals(password) )
   {
      System.out.println("Connexion OK") ;
      trouve = true ;
   }
   else
      ...
}

**sissi25** · 30/01/2007, 15h54

ok mais si je fais ca et que je rentre un nom qui n'est pas dans la bdd, il ne se passe rien

je n'ai pas le message d'erreur "Le nom n'est pas dans la bdd"

**schniouf** · 30/01/2007, 17h05

Envoyé par sissi25

ok mais si je fais ca et que je rentre un nom qui n'est pas dans la bdd, il ne se passe rien

je n'ai pas le message d'erreur "Le nom n'est pas dans la bdd"

Alors tu fais comme ça :

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
...
boolean trouve = false ;
while ( resultset.next() && !trouve)
{
   ...
   if ( resultset.getString("password").equals(password) )
   {
      trouve = true ;
   }
}
 
if ( trouve )
    System.out.println("Connexion OK") ;
else
    System.out.println("Le nom n'est pas dans la bdd") ;

**sissi25** · 30/01/2007, 20h12

oui mais ca met le message d'erreur si je rentre un mauvais mot de passe. Si je rentre un mauvais nom, ca ne fait rien

**sironimo** · 30/01/2007, 20h18

Code :

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

1
2
 
if ( (resultset.getString("password").equals(password)) && (resultset.getString("login").equals(login))  )

C'est pas très propre car tu auras un message d'erreur pour 3 cas: mauvais mot de passe, mauvais login, mauvais login et mauvais mot de passe.

Mais disons que pour l'instant ça pourrait aller

je te conseille de faire quelques recherches sur google sur du code existant pour t'aider aussi.

**schniouf** · 30/01/2007, 21h14

Envoyé par sissi25

oui mais ca met le message d'erreur si je rentre un mauvais mot de passe. Si je rentre un mauvais nom, ca ne fait rien

Ben justement, c'est ça la sécurité ! Quand tu rentres un mauvais login, il faut pas dire que le login existe pas. Il faut dire "Mauvais login ou mot de passe". De même, quand le login est bon mais pas le mot de passe, si tu dis que tu as donné un mauvais mot de passe, alors le pirate sait que ce login existe et ça sera plus facile pour lui de pirater !

**sissi25** · 30/01/2007, 21h26

ok c'est vrai
tu as raison

mais pour ma curiosité personnelle (ca fait un bon moment que je suis dessus ...

)

comment je pourrai faire pour afficher un message d'erreur quand on rentre un mauvais login ????

**sissi25** · 30/01/2007, 22h27

pourquoi est ce que ce code ne fonctionne 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
 
        try {
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            conn = DriverManager.getConnection("jdbc:odbc:BiorG","","");
            System.out.println("Connexion établie avec BiorG");
            Statement s = conn.createStatement();
            rs = s.execute("SELECT Utilisateurs.Password, Utilisateurs.Login FROM Utilisateurs WHERE (Utilisateurs.Login)=\'"+login+"\' AND (Utilisateurs.Password)=\'"+pass+"\'");
            if (rs){
                resultSet = s.getResultSet();
                message = "Connection réussie";
                JOptionPane.showMessageDialog(null,""+ message );
            } else { message = "Connection échouée";
            JOptionPane.showMessageDialog(null,""+ message );
            }            
            resultSet.close(); s.close();                      
        }

il y a un pb au niveau du if(rs) parce que j'ai toujours connection réussie meme si le mdp et le login ne correspondent pas

merci de votre aide

**sironimo** · 30/01/2007, 22h31

Il faudrait appliquer le resulSet sur rs et non sur s. De plus, il faudrait vérifier qu'il ne renvoie rien ce resultSet.

**schniouf** · 30/01/2007, 22h51

Envoyé par sissi25

ok c'est vrai
tu as raison

mais pour ma curiosité personnelle (ca fait un bon moment que je suis dessus ...

)

comment je pourrai faire pour afficher un message d'erreur quand on rentre un mauvais login ????

Tout est une histoire d'algorithme, il faut réfléchir un peu :

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
...
resultset = s.executeQuery( "Select * from LOGIN WHERE login='"+login+"'" ) ;
boolean loginFound = false ;
boolean trouve = false ;
while ( resultset.next() && !trouve)
{
   loginFound = true ;
   ...
   if ( resultset.getString("password").equals(password) )
   {
      trouve = true ;
   }
}
 
if ( trouve )
    System.out.println("Connexion OK") ;
else if ( !loginFound )
    System.out.println("Le login n'est pas dans la bdd") ;
else
     System.out.println("Login trouvé, mais mauvais mot de passe")

Le mieux reste de faire :

Code :

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

1
2
3
4
5
resultset = s.executeQuery( "Select * from LOGIN WHERE login='"+login+"' AND password='"+password+"'" ) ;
if ( resultset.next() )
   System.out.println("Connexion OK") ;
else
   System.out.println("Mauvais login ou mot de passe") ;

[debutant]problème avec une authentification

JDBC Java

Discussions similaires

Partager

Partager