Discussion :

[blocnote]

Salut, svp voila un simple code qui contient une interface graphique avec deux boutons dont un avec le code de connexion a la bd et il fonctionne chez moi.

Bon mon problème est : est-ce que ce code est correcte théoriquement, cad il faut un seul ficher classe ou plus, l'ordre des classes et méthodes est juste en deux mots ce qui m'intéresse c'est les bonnes pratiques donnez moi des remarques et merci d'avance.

[lnotray]

Salut,
- Autant que possible, on met chaque classe / interface / enum /annotation dans un fichier séparé portant le même nom, et on les fait commencer par une majuscule.
- On essaye de donner des noms explicites aux classes et aux méthodes
- Appeler une classe 'Interface' est un peu bizarre car Interface est un élément du langage.
- Quand on utilise le réseau, on le libère à la fin du traitement en utilisant finally, dans ton exemple ça donnerait cela :

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
 
		Connection connexion = null;
		Statement instruction = null;
		ResultSet resultat = null;
		try {
			connexion = DriverManager.getConnection(
					"jdbc:mysql://localhost:3306/test", "root", "estest");
			try {
				instruction = connexion.createStatement();
				try {
					resultat = instruction
							.executeQuery("SELECT col1 FROM table1");
					try {
						while (resultat.next()) {
							System.out.println("Resultat: "
									+ resultat.getString("col1"));
						}
					} catch (SQLException e) {
						throw e;
					} finally {
						resultat.close();
					}
				} catch (SQLException e) {
					throw e;
				} finally {
					instruction.close();
				}
			} catch (SQLException e) {
				throw e;
			} finally {
				connexion.close();
			}
		} catch (SQLException e) {
			System.out.println("echec pilote : " + e);
		}

Il n'y a pas de garbage collection pour les sockets laissés ouverts.

[legentil]

Salut !

En ce qui concerne les classes EcouteBoutonEnvoi1 et EcouteBoutonEnvoi2, ça me choque pas que ce soit dans la même classe.
Moi en fait plutôt que de redéfinir une classe listener comme ça, je fais directement :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
JButton1.addActionListener(new ActionListener(){
    public void actionPerformed(ActionEvent ev)
		{ 
			System.out.println("Connexion...");
			clique1 clq = new clique1();
			clq.meth1();
 
		}
});

Ce qui revient au même, au final je sais pas si il y en a un qui est à préférer à l'autre...

par contre pour ta classe clique1 (qui devrai commencer par un majuscule d'ailleur !), ça serai plus propre de faire un fichier à part avec une classe genre AccesBDD un truc comme ça
Cette classe serai paramétrable (pour l'adresse, login, password), la méthode meth1 s'appellerai connexion (et d'autres méthodes permettraient de faire diverses actions sur la base, comme lancer du SQL par exemple, fermer la connexion).
Comme ça tu te retrouverai avec une classe entièrement réutilisable !

Ensuite ta classe Connexion ne sert pas à grand chose, tu pourrai mettre le

Code :

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

1
2
3
4
5
 
public static void main(String[] args)
	{	Interface1 inter1 = new Interface1();
		inter1.methode1();
	}

directement dans la classe Interface

Dernier détail, si ta classe principale de ton fichier s'appelle Interface1, ton fichier doit s'appeler Interface1.java

Voilà en espérant que ça t'aidera !

[legentil]

lnotray je viens de voir ton message, il y a pas d'intérêt à catcher 3 fois la même exception pour faire le même traitement derrière, ton code est simplifiable :

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
 
Connection connexion = null;
		Statement instruction = null;
		ResultSet resultat = null;
		try {
			connexion = DriverManager.getConnection(
					"jdbc:mysql://localhost:3306/test", "root", "estest");
			try {
				instruction = connexion.createStatement();
				resultat = instruction.executeQuery("SELECT col1 FROM table1");
				while (resultat.next()) {
					System.out.println("Resultat: "
									+ resultat.getString("col1"));
				}
			} catch (SQLException e) {
				throw e;
			} finally {
				resultat.close();
			}
		} catch (SQLException e) {
			System.out.println("echec pilote : " + e);
		}

ça aura le même comportement si je me trompe pas?
Sinon par contre tout à fait d'accord sur le reste !

[lnotray]

Peut-être est-ce suffisant dans la plupart des cas et que mon code est un peu ceinture - bretelles, mais en toute rigueur avec ta méthode :
- si connexion.createStatement() lève une exception tu vas te retrouver avec un NPE sur resultat.close()
- à aucun moment tu ne fermes ta connection.
Après, je suis bien incapable de te dire ce que cela va impliquer, je ne sais même pas ce qu'il advient des connections qu'on laisse ouvertes, mais à mon avis ce n'est pas optimal.

[legentil]

euh sauf erreur de ma part (j'ai copié collé ton code et peut être supprimé un peu vite) mon code et le tiens ont le même effet

Code :

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

1
2
 
connexion = DriverManager.getConnection(				"jdbc:mysql://localhost:3306/test", "root", "estest");

Va être catché par

Code :

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

1
2
3
4
 
} catch (SQLException e) {
			System.out.println("echec pilote : " + e);
		}

et

Code :

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

1
2
3
4
5
6
7
 
instruction = connexion.createStatement();
resultat = instruction.executeQuery("SELECT col1 FROM table1");
				while (resultat.next()) {
					System.out.println("Resultat: "
									+ resultat.getString("col1"));
				}

Toutes les instructions du code ci dessus seront catchés par

Code :

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

1
2
3
4
5
6
 
} catch (SQLException e) {
				throw e;
			} finally {
				resultat.close();
			}

(et donc le close est fait dans ton exemple comme dans le mien)
Par contre, dans ton exemple, le close sera fait 3 fois alors que dans le miens il est fait qu'une fois!

[rolfone]

je ne suis pas sur de moi mais la j'ai un doute

Code :

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

resultat.close();

j'aurais dit plutot

Code :

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

connexion.close();

[lnotray]

+1
J'ai un gros doute sur le fait que ResultSet.close() ferme aussi le Statement et la Connection.
Je viens de vérifier, Statement.close() ferme les ResultSet associés, on peut donc supprimer un try / catch dans mon code. Par contre je ne crois pas qu'un Connection.close() ferme systématiquement les Statement.

[rolfone]

suite a un doute je suis allé voir la FAQ JDBC - Comment fermer une connexion ?

[legentil]

Ah oui au temps pour moi !
J'avais pas vu que tu fermais des trucs différents à chaque fois!
Je pensais que tu fermais 3 fois la connexion en fait...

En fait dans ma tête, le code que je laissait était :

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
 
Connection connexion = null;
		Statement instruction = null;
		ResultSet resultat = null;
		try {
			connexion = DriverManager.getConnection(
					"jdbc:mysql://localhost:3306/test", "root", "estest");
			try {
				instruction = connexion.createStatement();
				resultat = instruction.executeQuery("SELECT col1 FROM table1");
				while (resultat.next()) {
					System.out.println("Resultat: "
									+ resultat.getString("col1"));
				}
			} catch (SQLException e) {
				throw e;
			} finally {
				connexion.close();
			}
		} catch (SQLException e) {
			System.out.println("echec pilote : " + e);
		}

Par contre effectivement, pour les statements ouverts, pas terrible même ça
J'ai lu un peu trop en diagonale, j'ai un peu de la m... dans les yeux (matin oblige )

à + !