Authentification MySQL/Swing/Java Fermé

Question

Bonjour à tous !

Voilà j'ai décidé de commencer à faire du développement IHM sous Java. J'ai donc créé des interfaces plutôt simple puis un peu plus compliqué avec des events et tout. 

Ensuite j'ai voulu me connecter à une DB MySQL, ce que j'ai réussi à faire.

Mon problème aujourd'hui est l'authentification, j'arrive à vérifier dans ma DB si le login correspond au mot de passe. Ce que je souhaite c'est grâce à cela, créer un utilisateur unique. Hop design pattern Singleton sur une classe User ! 

Je ne pensais pas avoir de problème ... Sauf que je n'arrive pas à récupérer mon utilisateur dans la classe main qui me servira pour communiquer avec l'utilisateur.

Je vous montre le code :

public class User {
	
	private static User instance;
	String login;
	String mdp;
	String droit;

private User(String login, String mdp, String droit) 
	{
		this.login = login;
		this.mdp = mdp;
		this.droit = droit;
	}
	
	public static synchronized User getInstance(String login, String mdp, String droit)
	{
		if(instance == null)
		{
			instance = new User(login, mdp, droit);
		}
		return instance;
	}

@Override
	public String toString() {
		return "User [login=" + login + ", mdp=" + mdp + ", droit=" + droit + "]";
	}

}


D'ailleurs je ne sais toujours pas si j'ai le droit de créer un constructeur de Singleton avec des paramètres comme je le fais ...

public class Login extends JFrame {
	
	Connection cn;
	Statement st;
	ResultSet rs;

JLabel labLogin, labPass;
	JPanel principal, panLogin, panPass, panButton;
	JTextField login, pass;
	JButton b;
	
	String user,password;
	
	static User utilisateur;

public Login()
	{
		
		frame();
	}
	
	public void connect()
	{
		try
		{
			Class.forName("com.mysql.jdbc.Driver");
			System.out.println("Worked");
		}
		catch(ClassNotFoundException e)
		{
			e.printStackTrace();
		}
	}
	
	public void connectMySQL()
	{
		connect();
		String host = "jdbc:mysql://localhost/bdd";
		String user = "root";
		String pass = "";
		try
		{
			cn = DriverManager.getConnection(host, user, pass);
			st = cn.createStatement();
			/*PreparedStatement statement = (PreparedStatement) connect.prepareStatement("SELECT * FROM etudiant");
			
			ResultSet data = statement.executeQuery();
			while(data.next())
			{
				System.out.println("Name : "+data.getObject("nom")+ " Prenom : "+data.getObject("prenom"));
			}
			*/

System.out.println("it works");
		}
		catch(SQLException e)
		{
			e.printStackTrace();
		}
	}
	
	public void deconnexion()
	{
		try
		{
			st.close();
			cn.close();
		}
		catch(SQLException e)
		{
			System.out.println(e.getMessage());
		}
		
	}
	
	public void frame()
	{
		this.setTitle("User Login");
		this.setPreferredSize(new Dimension(400, 220));
		this.setBackground(Color.WHITE);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setLayout(new BorderLayout());

principal = new JPanel();
		principal.setBackground(Color.WHITE);

panLogin = new JPanel();
		panLogin.setBackground(Color.WHITE);
		panLogin.setPreferredSize(new Dimension(220, 60));
		panLogin.setBorder(BorderFactory.createTitledBorder("User Login"));
		labLogin = new JLabel("Your login : ");
		login = new JTextField();
		login.setPreferredSize(new Dimension(100, 20));
		panLogin.add(labLogin);
		panLogin.add(login);
		
		panPass = new JPanel();
		panPass.setBackground(Color.WHITE);
		panPass.setPreferredSize(new Dimension(220, 60));
		panPass.setBorder(BorderFactory.createTitledBorder("User Password"));
		labPass = new JLabel("Your password : ");
		pass = new JTextField();
		pass.setPreferredSize(new Dimension(100, 20));
		panPass.add(labPass);
		panPass.add(pass);
		
		panButton = new JPanel();
		panButton.setBackground(Color.WHITE);
		b = new JButton("Login");
		b.addActionListener(new ActionListener() {
			
			@Override
			public void actionPerformed(ActionEvent e)
			{
				try
				{
					connectMySQL();
					user = login.getText().trim();
					password = pass.getText().trim();
					
					String sql = "SELECT login,pass FROM users WHERE login = '"+user+"'AND pass = '"+password+"'";
					rs = st.executeQuery(sql);

int count = 0;
					
					while(rs.next())
					{
						count++;
					}
					
					if(count == 1)
					{
						JOptionPane.showMessageDialog(null, "User Found, Acces Granted");
					
					}
					else if(count > 1)
					{
						JOptionPane.showMessageDialog(null, "Duplicate User, Acces Denied");
					}
					else
					{
						JOptionPane.showMessageDialog(null, "User doesn't exist");
					}
					deconnexion();
					
				}
				catch(Exception ex)
				{
					ex.getMessage();
				}

}
		});
		panButton.add(b);
		
		principal.add(panLogin);
		principal.add(panPass);

this.getContentPane().add(principal, BorderLayout.CENTER);
		this.getContentPane().add(panButton, BorderLayout.SOUTH);
		pack();
		this.setVisible(true);
	}
	
	public void returnUser()
	{
		utilisateur = User.getInstance(user, password, "admin");
	}

Et voici le soucis dans le Main.java :

public class Main {
	public static void main(String[] args) {
		Login l = new Login();
		System.out.println(l.user);
	}
	
}

Je comprends que dans le main lorsque je souhaite afficher le User créé lors de la validation de la connexion, celui-ci soit null car le programme fait le new Login et en même temps avant que j'ai pu finir mon traitement dans ma frame, affiche le User donc il sera à null ...

Voilà je suis plus qu'ouvert à tous conseils que vous pourriez me donner, je débute vraiment dans ce domaine donc j'aimerai apprendre et avoir une bonne façon de faire ^^

PS: Désolé pour tout le code, la frame prend de la place ...

Merci d'avance !

KX · Answer

Bonjour,

Créer un singleton avec des paramètres n'a pas de sens.

Tout les User vont utiliser le même singleton, donc si tu précises login/mdp à ton singleton, ça veut dire que tout les User auront le même login/mdp !

Exemple :

User toto = User.getInstance("toto", "123", "admin");
System.out.println(toto); // User [login=toto, mdp=123, droit=admin]

User tata = User.getInstance("tata", "789", "visitor");
System.out.println(tata); // User [login=toto, mdp=123, droit=admin]

System.out.println(toto == tata); // true
Dans ton code, un singleton pertinent pourrait être pour créer une unique connexion à la base de données, DriverManager.getConnection serait donc utilisé une seule fois pour te renvoyer toujours la même connexion (les user/mdp de base de données ne changent pas d'un appel à l'autre).

Hurobaki · Answer

Salut KX, 

Merci de ta réponse, si je ne souhaite justement qu'il n'y ai qu'un seul utilisateur ce code serait-il correct ? La seule différence que je souhaite mettre en place serait si cet utilisateur est un admin ou non.

Mais je ne parviens pas à le récupérer dans mon Main.java car il est à null, il faudrait que j'attende d'avoir fini avec la frame pour ensuite continuer les instructions du Main.

Est-ce que cela est possible ? Ou est-ce qu'il y aurait une autre méthode ?

Hurobaki · Answer

Cette implémentation plus robuste, je ne doute pas qu'elle soit correcte mais pour ce qui est du multithread comment fait-elle pour palier ce problème ? Je me suis beaucoup renseigner et j'ai vu que le principal problème était le multithread sur un singleton, et que c'est pour cela qu'il fallait utiliser synchronized dans ma méthode.

Oui je mélange un peu à force chercher des solutions j'ai fais un gros mélange ...

public static void main(String[] args) {
    LoginDialog dialog = new LoginDialog();
    int result = dialog.requestCredentials(); // demande user/mdp
    if (result == OK_OPTION) { // l'utilisateur appuie sur OK
        User user = dialog.getUser();
        boolean isValid = UserService.getInstance().isValid(user);
	if (isValid) { // on utilise l'application avec user
        }
        else { // on rejette l'accès à l'application
        }
    } else { // on rejette l'accès à l'application
    }
}


Juste pour être sûr que j'ai compris, LoginDialog serait ma classe Login.
La méthode requestCredentials() est la Frame qui s'affiche pour demander le login/mdp. Lors du clic sur le bouton valider/ok, je stocke la valeur dans un entier => result et ce n'est qu'à ce moment là que je vais créer mon User. 
Le seul soucis que j'ai c'est la fonction isValid, elle va ma permettre d'initialiser le login, mdp et droit de mon User ?

Authentification MySQL/Swing/Java

3 réponses

Discussions similaires