Discussion :

[mB.milen]

Bonsoir , SVP j'aimerais que vous m'aidiez à créer un login en java à mon application en entrant dans le premier champ un numéro et le second le mot de pass. J'ai essayé plusieurs codes en vain.

[joel.drigo]

Salut,

1. Si tu montres ton code, on pourra t'aider à le faire fonctionner
2. Au minimum précise-nous quel environnement graphique... Desktop (Swing, JavaFX, SWT...), Web (JSP, JSF...)
3. Un login ce n'est pas seulement un problème de GUI... quel système d'authentification as-tu derrière l'interface ?

[mB.milen]

Bonjour joel.drigo
Voici mon code dont tu m' a demandé.

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
 
  public void connexion_1() {
 
	AcceuilInterface acc=new AcceuilInterface();
    String num = acc.getNumCommercant_tfd().getText(); // obtention du numero saisi
    String password = acc.getPassword_tfd().getText();  // obtention du mot de pass saisi
    Connection connect =DataBaseAccess.open();
 
        try{
            state = connect.createStatement();
            String sql = "SELECT motDePass FROM Commercant WHERE  numCommercant = '"+num+"' ";
            resultSet = state.executeQuery(sql);
 
            if(resultSet.next()){
 
            String motDePass = resultSet.getString(2);
 
            if(motDePass.equals(password)){
 
                 JOptionPane.showMessageDialog(null,"Connexion réussie ! ","Success",JOptionPane.PLAIN_MESSAGE);
            }else {
 
                JOptionPane.showMessageDialog(null,"Mot de passe incorrect ! ","Error",1);
            }
            }else {
 
                JOptionPane.showMessageDialog(null,"Login incorrect ! ","Error",1);
            }
                connect.close();
 
        }catch (SQLException e4) {
 
            System.out.println(e4.getMessage());
        }
    }

[joel.drigo]

Salut,

String motDePass = resultSet.getString(2); veut dire que tu récupères le mot de passe dans le deuxième champ du select, or il n'y a qu'un seul champ !

Cela, dit, plutôt que de procéder ainsi, en récupérant le mot de passe et en le comparant, ce serait plus sécuris en faisant une requête du type select 1 where login='login' and motdepasse='motdepasse', et s'il y a un résultat le login est ok, sinon il est ko. Ainsi, le vrai mot de passe reste toujours côté serveur. Ensuite, il est toujours mieux de procéder avec un PreparedStatement :

Code :

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

1
2
3
4
5
6
7
try(PreparedStatement statement = connect.preparedStatement("SELECT 1 FROM Commercant WHERE  numCommercant = ? and motDePass = ?")) {
statement.setString(1,num);
statement.setString(2,password);
ResultSet resultSet = statement.executeQuery();
 
boolean loginOk = resultSet.next(); // s'il y a un résultat alors le couple num/password existe, sinon le login est ko
}

Attention de bien libérer tes ressources (avec close()), dans mon code fait avec try-with-resource.

[mB.milen]

Salut joel.drigo
J'ai essayé de corriger les erreurs tel que tu m'a montré mais je n'obtiens pas ce dont je veux.
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
 
public void connexion_11() {
 
	AcceuilInterface acc=new AcceuilInterface();
    String num = acc.getNumCommercant_tfd().getText(); // ( le numéroest de type int ). Obtention du numero saisi 
    String password = acc.getPassword_tfd().getText();  // Obtention du mot de pass saisi
    Connection connect =DataBaseAccess.open();
 
        try{
            prepa_state = connect.prepareStatement("SELECT 1 FROM Commercant WHERE  numCommercant = ? and motDePass = ? ");
            prepa_state.setString(1, num);
            prepa_state.setString(2, password);
            resultSet = prepa_state.executeQuery();
            boolean loginOk = resultSet.next();
 
            if(loginOk){
 
            String motDePass = resultSet.getString(2);
 
            if(motDePass.equals(password)){
                 JOptionPane.showMessageDialog(null,"Connexion réussie ! ","Success",JOptionPane.PLAIN_MESSAGE);
            }else {
 
                JOptionPane.showMessageDialog(null,"Mot de passe incorrect ! ","Error",1);
            }
            }else {
            	System.out.println("a");
                JOptionPane.showMessageDialog(null,"Login incorrect ! ","Error",1);
            }
         //  connect.close();
 
        }catch (SQLException e4) {
 
            System.out.println(e4.getMessage());
        }
    }

la console n'affiche pas d'erreur.
Voici son affichage:

Code :

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

1
2
3
 
Connection éffective !
a

[joel.drigo]

1. 
   
   Mais ça :
   

   Code :

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

   1
   2
   3
    String motDePass = resultSet.getString(2);
    
               if(motDePass.equals(password)){

   Il ne faut plus le faire ! Je t'ai dit que le mot de passe n'était plus récupéré par le client : le select récupère 1.
   
2. 
   Si ça affiche "a", c'est parce que tu testes avec un login/password incorrect. Tu peux afficher en console temporairement le num et mot de passe saisis pour vérifier que c'est bien les valeurs saisies que tu utilises (le problème vient peut-être de ta classe AcceuilInterface)
   
3. 
   Et tu t'es arrangé pour refaire un code où les ressources (statement/resultset) ne sont pas libérées !!!