Optimiser l'utilisation d'une connexion MySql

Version imprimable

Voir 40 message(s) de cette discussion en une page

Ta classe de connexion est vraiment mal pensée... tu as défini la connexion en static !
Si tu as 2 demandes simultanées, tu piles la première ouverte...

Bref, les bons principes sont :
- une classe utilitaire qui ne s'occupe QUE de renvoyer une connexion
- des méthodes qui utilisent la connexion au travers de PreparedStatement ou Statement (ou autre...)

La classe utilitaire pourrait ressembler à ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
public class ConnectionUtils
{
    public static Connection getConnection()
    {
        try 
        {            
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            String url = "jdbc:mysql://ip:port/bd_name";
            String user = "user_name";
            String passwd = "password";
            return DriverManager.getConnection(url, user, passwd);
        } 
        catch (Exception e) 
        {
            e.printStackTrace();            
        }
        return null;
    }
}

Tes méthodes utilisatrices

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
...
Connection connection = null;
PreparedStatement pstmt = null;
 
try
{
    connection = ConnectionUtils.getConnection();
    pstmt = connection.prepareStatement("....");
     ...
}
catch (Exception e)
{
    // on signale
}
finally
{
    if (pstmt != null) pstmt.close();
    if (connection != null) connection.close();
}

25/02/2015, 14h01
junior222

Ok merci pour votre aide
25/02/2015, 15h28
fr1man
Tu peux sortir le chargement du driver dans un bloc static; pas besoin de l'exécuter à chaque appel de la méthode.
Code:

1 2 3 4 static { Class.forName("sun.jdbc.odbc.JdbcOdbcDriver"); }
25/02/2015, 16h08
junior222

Et je met ça à quel niveau?
27/02/2015, 10h49
junior222

Citation:

Envoyé par OButterlin

Ta classe de connexion est vraiment mal pensée... tu as défini la connexion en static !
Si tu as 2 demandes simultanées, tu piles la première ouverte...

Je ne comprend pas quand vous parlez de 2 connexion simultanée vous voulez dire sur la même machine ou dans le réseau?
27/02/2015, 11h24
OButterlin

Un développement d'application Swing met en oeuvre des Threads, il se pourrait très bien que 2 Threads traitent des données DB simultanément et là, problème avec une connexion "static".
Alors ce n'est peut-être pas le cas de ton application, met là, j'émets des doutes sur le respect des bonnes manières, tu auras certainement codé tes méthodes dans le thread AWT ce qui est fortement déconseillé (tu peux figer ton ihm)

Citation:
Envoyé par fr1man
Tu peux sortir le chargement du driver dans un bloc static; pas besoin de l'exécuter à chaque appel de la méthode.
Code:

1 2 3 4 static { Class.forName("sun.jdbc.odbc.JdbcOdbcDriver"); }
Il y a un truc que je n'avais pas vu ; pour MySQL il faut faire :

Code:

Class.forName("com.mysql.jdbc.Driver");

et non pas appeler le JdbcOdbcDriver (qui ne sert que pour utiliser une base Access ...)

donc on pourrait se dire : "pourquoi ça marche quand même ?"

parce que le com.mysql.jdbc.Driver est, en principe, de type 4 et donc s'autocharge en service au lancement de l'application avant même son utilisation.
Ma science est toute neuve c'est en lisant un post récent de professeur shadoko que j'ai appris que les pilotes JDBC de type 4 n'avait plus besoin de Class.forName("com.mysql.jdbc.Driver").

Moralité : on peut supprimer le Class.forName et la classe utilitaire de OButterlin devient :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 public class ConnectionUtils { public static Connection getConnection() { try { String url = "jdbc:mysql://ip:port/bd_name"; String user = "user_name"; String passwd = "password"; return DriverManager.getConnection(url, user, passwd); } catch (Exception e) { e.printStackTrace(); } return null; } }

27/02/2015, 11h49
Népomucène

Citation:

Envoyé par junior222

Je ne comprend pas quand vous parlez de 2 connexion simultanée vous voulez dire sur la même machine ou dans le réseau?

Sur la même machine.
27/02/2015, 12h47
junior222

Citation:

Envoyé par OButterlin

Un développement d'application Swing met en oeuvre des Threads, il se pourrait très bien que 2 Threads traitent des données DB simultanément et là, problème avec une connexion "static".
Alors ce n'est peut-être pas le cas de ton application, met là, j'émets des doutes sur le respect des bonnes manières, tu auras certainement codé tes méthodes dans le thread AWT ce qui est fortement déconseillé (tu peux figer ton ihm)

La je ne comprend pas bien où vous voulez en venir mais ce que je peux dire c'est que j'utilise le thread juste pour exécuter ma class main qui est mon interface de login

Citation:

Envoyé par Népomucène

Il y a un truc que je n'avais pas vu ; pour MySQL il faut faire :

Code:

Class.forName("com.mysql.jdbc.Driver");

et non pas appeler le JdbcOdbcDriver (qui ne sert que pour utiliser une base Access ...)

donc on pourrait se dire : "pourquoi ça marche quand même ?"

parce que le com.mysql.jdbc.Driver est, en principe, de type 4 et donc s'autocharge en service au lancement de l'application avant même son utilisation.
Ma science est toute neuve c'est en lisant un post récent de professeur shadoko que j'ai appris que les pilotes JDBC de type 4 n'avait plus besoin de Class.forName("com.mysql.jdbc.Driver").

Moralité : on peut supprimer le Class.forName et la classe utilitaire de OButterlin devient :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public class ConnectionUtils
{
    public static Connection getConnection()
    {
        try 
        {            
            String url = "jdbc:mysql://ip:port/bd_name";
            String user = "user_name";
            String passwd = "password";
            return DriverManager.getConnection(url, user, passwd);
        } 
        catch (Exception e) 
        {
            e.printStackTrace();            
        }
        return null;
    }
}

Merci beaucoup j'ai testé et voilà un exemple:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
 
 
public class ConnectionUtils {
    public static Connection getConnection()
    {
        try 
        {            
            String url = "jdbc:mysql://ip:port/bd_name";
            String user = "user";
            String passwd = "password";
            return DriverManager.getConnection(url, user, passwd);
        } 
        catch (Exception e) 
        {
            e.printStackTrace();            
        }
        return null;
    }
    public static void main(String [] args){
    	PreparedStatement pstmt=null;
    	Connection connection=ConnectionUtils.getConnection();
    	try
    	{
    	    connection = ConnectionUtils.getConnection();
    	    pstmt = connection.prepareStatement("select * from client");    	
    	    System.out.println("exécuté");
    	    ResultSet res=pstmt.executeQuery();
    	    while(res.next())
    	    	System.out.println(res.getInt(1));
    	}
    	catch (Exception e)
    	{
    		e.printStackTrace();
    	}
    	finally
    	{
    	    if (pstmt != null)
				try {
					pstmt.close();
				} catch (SQLException e) {
					e.printStackTrace();
				}
    	    if (connection != null)
				try {
					connection.close();
				} catch (SQLException e) {
					e.printStackTrace();
				}
    	}
    }
}

28/02/2015, 15h34
MasterMbg

Bonjour,
Tu te donnes de la peine à fermer la ressource occupée par PreparedStatement pour rien. Utilise try with ressources disponible depuis Java 7 pour te simplifier la tâche.
28/02/2015, 16h35
junior222

Citation:

Envoyé par MasterMbg

Bonjour,
Tu te donnes de la peine à fermer la ressource occupée par PreparedStatement pour rien. Utilise try with ressources disponible depuis Java 7 pour te simplifier la tâche.

Vous avez un exemple svp?
28/02/2015, 16h50
MasterMbg
Bah, un truc du genre :
Code:

1 2 3 4 5 6 7 try(PreparedStatement pstmt = connection.prepareStatement( "select * from client" )){ //Suite de ton code.... } catch (Exception e){ e.printStackTrace(); }
Dans ce cas, tu te passes du bloc finally, qui lui aussi aurait besoin d'un bloc try/catch... Le code devient plus lisible et tu développes un peu plus rapidement.
28/02/2015, 16h51
Népomucène

Citation:

Envoyé par MasterMbg

Bonjour,
Tu te donnes de la peine à fermer la ressource occupée par PreparedStatement pour rien. Utilise try with ressources disponible depuis Java 7 pour te simplifier la tâche.

D'après la doc Oracle, ça ne marcherait qu'avec des drivers JDBC 4.1.
Est-on sûr que MySql a un driver de ce type ?
Quand je vois la doc du JDBC MySql, j'ai un doute... Mais j'ai peut-être mal cherché ?
28/02/2015, 16h57
junior222
Citation:
Envoyé par MasterMbg

Bah, un truc du genre :

Code:

1 2 3 4 5 6 7 try(PreparedStatement pstmt = connection.prepareStatement( "select * from client" )){ //Suite de ton code.... } catch (Exception e){ e.printStackTrace(); }

Dans ce cas, tu te passes du bloc finally, qui lui aussi aurait besoin d'un bloc try/catch... Le code devient plus lisible et tu développes un peu plus rapidement.
Ok et pour l'objet de connexion?
28/02/2015, 16h59
MasterMbg

Citation:

Envoyé par Népomucène

D'après la doc Oracle, ça ne marcherait qu'avec des drivers JDBC 4.1.
Est-on sûr que MySql a un driver de ce type ?
Quand je vois la doc du JDBC MySql, j'ai un doute... Mais j'ai peut-être mal cherché ?

Les drivers de MySQL depuis la version 3 sont tous de type 4.

Voir 40 message(s) de cette discussion en une page