Problème avec ResultSet et la communication avec la base

Bonsoir,
Je développe une application de gestion pour mon PFE.
Au niveau de l'interface de connexion et après la requête, j'ai un problème ou niveau de la boucle if qui gère l'authentification en décidant qu'il s'agit d'un Admin, Ingénieur ou autres. La base de donnée fonctionne correctement, je l'ai testé avec les autres interfaces de l'application.
Pouvez-vous m'aider SVP? Merci.
Voici le code:

Code:

---

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

Class.forName("sun.jdbc.odbc.JdbcOdbcDriver"); String url = "jdbc:odbc:gmi"; Connection con = DriverManager.getConnection(url, "", ""); Statement creerconnec = con.createStatement(); String sql = "select * from compte_utilisateur where login = '" + jTextField1.getText() + "' and password ='" + jTextField2.getText() + "';"; ResultSet RS = creerconnec.executeQuery(sql);   while (RS.next()) { if (RS.getString("login").equals(jTextField1.getText()) && RS.getString("password").equals(jTextField2.getText())) { if (RS.getString("fonction").equals("Administrateur")) { interface_menu i_m = new interface_menu(); i_m.setVisible(true); } else if (RS.getString("fonction").equals("Ingenieur")) { interface_menu_Ingénieur i_m = new interface_menu_Ingénieur(); i_m.setVisible(true); } else { interface_menu_technicien i_m = new interface_menu_technicien(); i_m.setVisible(true); } } }   con.close();

---

Aider. A quoi ?

Il n'y a rien qui va dans ce code!

Pour commencer, en Java, il y a des conventions de nommage:

Code:

---

1 2	interface_menu i_m = new interface_menu();

---

Doit être remplacé par:

Code:

---

1 2	InterfaceMenu im = new InterfaceMenu();

---

Code:

---

1 2	interface_menu_Ingénieur i_m = new interface_menu_Ingénieur();

---

Doit être remplacé par:

Code:

---

1 2	InterfaceMenuIngénieur im = new InterfaceMenuIngénieur();

---

En java, les classes (et interface...) commencent par des majuscules et les variables par des minuscule. Par ailleurs, quand on change de mot, on met une majuscule (et non une minuscule). De plus, on évite les accents comme "é"'.

Toujours pour ce code, il serait bien d'utiliser le polymorphisme, on aurait le Design Pattern Factory ( https://fr.wikipedia.org/wiki/Fabriq...de_conception) ).

Comme:

Code:

---

1 2 3 4

public interface MenuInterface{ //Des trucs }

---

Code:

---

1 2 3 4

public class MenuBase implements MenuInterface{ //Des trucs }

---

Code:

---

1 2 3 4

public class MenuIngenieur implements MenuInterface{ //Des trucs }

---

Code:

---

1 2 3 4

public class MenuTechnicien implements MenuInterface{ //Des trucs }

---

La fin code deviendrait:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

Optional<MenuInterface> oMenu = Optional.empty() while (rs.next()) { if (rs.getString("login").equals(jTextField1.getText()) && rs.getString("password").equals(jTextField2.getText())) { if (rs.getString("fonction").equals("Administrateur")) { oMenu = Optional.of(new MenuBase()); } else if (rs.getString("fonction").equals("Ingenieur")) { oMenu = Optional.of(new MenuIngenieur()); } else { interface_menu_technicien i_m = new interface_menu_technicien(); oMenu = Optional.of(new MenuTechnicien()); } } } if(oMenu.isPresent()){ oMenu.get().setVisible(true); }

---

Code:

---

1 2	ResultSet RS = creerconnec.executeQuery(sql);

---

Non, non et non.

Code:

---

1 2	ResultSet rs = creerconnec.executeQuery(sql);

---

Convention Java. Les majuscules, c'est pour les classe.

Bon, on va maintenant aborder ce qui est vraiment problématique et choquant!

La requête SQL

Code:

---

1 2	String sql = "select * from compte_utilisateur where login = '" + jTextField1.getText() + "' and password ='" + jTextField2.getText() + "';";

---

Ça, c'est vraiment non.
Si on a comme texte de login:

Code:

---

1 2	Delete FROM COMMANDE;

---

C'est la cata, on a une injection SQL. C'est pour ça qu'il faut utiliser des PreparedStatement ( https://java.developpez.com/faq/jdbc...paredStatement ):

Code:

---

1 2 3 4 5 6

String sql = "select * from compte_utilisateur where login =? and password ?;"; PreparedStatement creerconnec = con.prepareStatement() creerconnec.setString(1,jTextField1.getText()); creerconnec.setString(2,jTextField2.getText()); ResultSet rs = creerconnec.executeQuery(sql);

---

A noter qu'il faudrait refaire la requête SQL pour ne remonter que ce qui est intéressant (ici la fonction):

Code:

---

1 2	select fonction from compte_utilisateur where login =? and password ?;

---

si aucun utilisateur ne correspond au couple login/password, le resultset sera vide.

Séparation des rôle (SOLID)
La fin correspond à SOLID ( https://fr.wikipedia.org/wiki/SOLID_(informatique) )
Ton code fait trop de chose, car il mélange des composants interfaces (JText...) et de BDD.
Or, il faut séparer.

Ton code doit être une méthode de login, qui donnerait le rôle.

Code:

---

1 2 3 4

Optional<String> getRole(String login,String password){ //Ton code }

---

Ou mieux:

Code:

---

1 2 3 4

Optional<User> getUser(String login, String password){ //Ton code }

---

Et une fois qu'on a l'utilisateur:

Code:

---

1 2 3 4

MenuInterface getMenu(User user){ //Ton code }

---

J'ai aussi oublié de préciser que:

Code:

---

1 2	if (rs.getString("login").equals(jTextField1.getText()) && rs.getString("password").equals(jTextField2.getText()))

---

Ne sert à rien, vu que par la requête SQL, ce qui est ramené sera toujours vrai.

Il y a aussi l'usage d'un pont odbc jdbc qui est douteux.
Quelle est la base de données cible ? Il n'existe pas de driver JDBC pour elle ? Si oui, mieux vaut l'utiliser.