Discussion :

[benji1801]

Bonjour,

J'utilise le driver jdbc pour me connecter à ma bdd Postgresql.

J'ai créé une classe BddConnexion qui me permet d'instancier une connexion. (Je n'utilise sans doute pas le bon vocabulaire je suis très débutant).

C'est dans la classe BddConnexion que je mets tous les paramètres pour se connecter à la bdd.

Et dans ma classe Main, quand je veux faire une requête sql, je fais :

Statement state = BddConnexion.getInstance().createStatement();
state.executeQuery(maRequete);
etc....

A chaque fois que je voulais faire une requête je faisais comme ça.

Est-ce correct de faire :

Connection conn = BddConnexion.getInstance();

Statement state = conn.createStatement();
state.executeQuery(maRequete);

Statement state2 = conn.createStatement();
state2.executeQuery(maRequete2);


Ca évite de faire BddConnexion.getInstance() à chaque fois.
Ca marche bien, mais est-ce "bon" ?

Merci.

[tchize_]

oui, il vaut mieux réutiliser au maximum la connexion, ca coute moins cher que d'en faire une nouvelle à la fois. Non parce qu'il manque certaines choses dans votre code. Typiquement vous devriez avoir quelque chose 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
16
17
18
19
20
21
 
Connection conn = BddConnexion.getInstance();
try{
   Statement state = conn.createStatement();
   try{
       resultat = state.executeQuery(maRequete);
       //traiter resultat 
   } finally {
       state.close();
   } 
   Statement state2 = conn.createStatement();
   try{
       resultat = state2.executeQuery(maRequete2);
       //traiter resultat 
   } finally {
       state2.close();
   }
} finally {
   if (conn!=null)
       conn.close();
}

ou mieux puisque les statements sont réutilisables

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
 
Connection conn = BddConnexion.getInstance();
try{
   Statement state = conn.createStatement();
   try{
       resultat = state.executeQuery(maRequete);
       //traiter resultat 
       resultat = state.executeQuery(maRequete2);
       //traiter resultat 
   } finally {
       state.close();
   } 
} finally {
   if (conn!=null)
       conn.close();
}

[benji1801]

D'accord merci beaucoup