Pool de connexion ou singleton

**questionneuse** · 04/04/2012, 20h45

Bonsoir,

J'ai une petite question à propos de connexion a une base de données.
J'ai développé un pattern singleton pour me connecter à ma base, c'est pratique puisque du coup je ne ferme jamais ma connexion et plusieurs utlisateurs peuvent du coup l'utiliser, voici mon code:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
	public static DBConnection getInstance() throws Exception {		 
		if (DBConnection.dbconnection == null) {			
			//String path = "blabla";
			String path = "blabla";
			props.load(new FileInputStream(path));	
			//mettre en place le fichier de log	la première fois
			PropertyConfigurator.configure(props.getProperty("pathLog4j"));
			setLog4j(Logger.getLogger("DBConnection.class"));
			if(Logger.getLogger("DBConnection.class") == null)
				System.out.println("La mise en place du système de log n'a pas fonctionné ");
			else
				log4j.log(Level.INFO,"Le système de log est mis en place: "+ getLog4j().getName());
			log4j.log(Level.INFO, "pathLog4j: "+props.getProperty("pathLog4j"));
			DBConnection.dbconnection = new DBConnection();
			log4j.log(Level.INFO, "DBConnection.dbconnection = new DBConnection();");
		}
		return DBConnection.dbconnection;
	}
 
	public Connection getConnection() throws Exception {
 
		if (DBConnection.dbconnection == null)
			throw new Exception("Singleton non initialisé");
		if (DBConnection.conn != null && !DBConnection.conn.isClosed())			
		{
			return DBConnection.conn;
		}
		else
			log4j.log(Level.INFO, "Connection == null");
		try {
			 DBConnection.conn = getConnection3();
		} catch (Exception ex) {
			log4j.log(Level.FATAL, "Erreur lors du chargement du dataSource");
			ex.printStackTrace();
			throw new Exception("Erreur lors du chargement du dataSource");
		}
		return conn; 
	}
 
	private static Connection getConnection3() throws Exception {
		if (DBConnection.conn != null && !DBConnection.conn.isClosed())
		{
			return DBConnection.conn;
		}
 
		user = props.getProperty("user");
		password = props.getProperty("password");
		host = props.getProperty("host");
		port = props.getProperty("port");
		dbname = props.getProperty("dbname");
 
		try {
			String url = "jdbc:mysql://" + host +":"+port+ "/" + dbname;
			log4j.log(Level.INFO, "url: "+ "jdbc:mysql://" + host +":"+port+ "/" + dbname);
			Class.forName("com.mysql.jdbc.Driver").newInstance();
			DBConnection.conn = DriverManager.getConnection(url, user, password);
			log4j.log(Level.INFO, "Connection DB réussie");
		} catch (Exception e) {
			log4j.log(Level.FATAL, "Echec de la connection DB" + e);
		} 
		return DBConnection.conn;
	}

Et j'ai lu par ci par là que ce n'était pas bien qu'il fallait passer par un pool de connexion, je ne comprend pas pourquoi c'est mieux. Ou est l'avantage? et surtout est ce ma méthode est dangereuse?

Merci pour votre aide précieuse,

**stephane.combes** · 05/04/2012, 10h15

Bonjour,

Une partie de la réponse est que si tu laisses une connexion ouverte sans activité le serveur de base de données va la fermer automatiquement au bout d'un certain temps. Du coup, a l'exécution de la prochaine requéte, ton programme va lever une SQLException.

Le serveur de base de données est paramétré pour accepter un certain nombre de connexions simultanées. Quand le quota est atteint les connexion suivantes sont rejetées tant que les autres ne sont pas libérées. Alors, par courtoisie, il est recommandé de libérer une connexion lorsqu'elle n'est plus utilisée.

Il est généralement recommandé d'utiliser un pool de connexion car le temps de connexion à une base de données est assez long. Le pool a en charge d'ouvrir les connexions auprès du serveur et de les maintenir ouvertes. Cela permet que lorsque tu demandes l'accès à la base tu n'as pas a attendre l'ouverture de la connexion pour envoyer ta requéte. Après, je ne sais pas comment le pool gére les connexions pour ne pas squatter les connexions comme précisé dans le point précédent.

Voila, la réponse est incomplète mais j'espère qu'elle t'aidera tout de même.

S. Combes

**fr1man** · 05/04/2012, 11h42

Déjà ton pattern singleton ne respecte pas les bonnes pratiques.
Constructeur privé, initialisation...

Ensuite, c'est une très mauvaise idée de partager une connexion entre plusieurs utilisateurs.

Soit tu crées une connexion à chaque besoin, puis tu la fermes quand tu ne t'en sers plus.
Soit tu utilises un pool (ce qui n'empêche pas de fermer la connexion, pour la rendre au pool)

**Jimmy_** · 05/04/2012, 12h20

Bonjour,

En principe un pool de connexion ne garde pas de connexion ouverte pour rien, si une première requête arrive et si aucune connexion n'est ouverte le pool se chargera de l'ouvrir. C'est à l'utilisation que cela devient intéressant, car les requêtes suivantes profiteront de la connexion déjà ouverte.
En fonction de son paramétrage si une connexion n'est plus utilisée il la fermera de toute manière.

**questionneuse** · 10/04/2012, 09h41

Bonjour,

Merci beaucoup pour vos réponses, voici mes commentaires:

Envoyé par stephane.combes

Bonjour,

Une partie de la réponse est que si tu laisses une connexion ouverte sans activité le serveur de base de données va la fermer automatiquement au bout d'un certain temps. Du coup, a l'exécution de la prochaine requéte, ton programme va lever une SQLException.

J'ai déjà réglé ce problème en augmentant la durée de non activité avant déconnexion dans le fichier de conf de ma base.

Le serveur de base de données est paramétré pour accepter un certain nombre de connexions simultanées. Quand le quota est atteint les connexion suivantes sont rejetées tant que les autres ne sont pas libérées. Alors, par courtoisie, il est recommandé de libérer une connexion lorsqu'elle n'est plus utilisée.

Oui mais justement je n'utilise qu'une connexion donc je ne risque donc pas d'atteindre le quota.

Il est généralement recommandé d'utiliser un pool de connexion car le temps de connexion à une base de données est assez long. Le pool a en charge d'ouvrir les connexions auprès du serveur et de les maintenir ouvertes. Cela permet que lorsque tu demandes l'accès à la base tu n'as pas a attendre l'ouverture de la connexion pour envoyer ta requéte.

Justement je ne me connecte qu'une fois et je laisse ouverte cette connexion. Donc je ne perd pas de temps à me reconnecter..

Ensuite, c'est une très mauvaise idée de partager une connexion entre plusieurs utilisateurs.

Soit tu crées une connexion à chaque besoin, puis tu la fermes quand tu ne t'en sers plus.
Soit tu utilises un pool (ce qui n'empêche pas de fermer la connexion, pour la rendre au pool)

J'ai dit que j'avais entendu que ce n'était pas bien et qu'il fallait apparement utiliser un pool de connexion mais ma question est POURQUOI.
Donc ta réponse ne m'aide en rien.

En principe un pool de connexion ne garde pas de connexion ouverte pour rien, si une première requête arrive et si aucune connexion n'est ouverte le pool se chargera de l'ouvrir. C'est à l'utilisation que cela devient intéressant, car les requêtes suivantes profiteront de la connexion déjà ouverte.
En fonction de son paramétrage si une connexion n'est plus utilisée il la fermera de toute manière.

ma connexion unique n'est pas ouverte pour rien, elle est la seule utilisée. Et toutes mes requetes profitent de cette connexion ouverte.

Je ne comprend toujours pas pourquoi certains disent que ce n'est pas bien. D'autres sur certains site web recommande cette technique du singleton sur une connexion base de données.
N'y a t-il personne qui est capable de me dire POURQUOI ca n'est pas bien, est ce que cela génére des erreurs.

Merci d'avance

**Jimmy_** · 10/04/2012, 09h57

Le principal soucis est de partager ta connexion entre différents utilisateurs. Ensuite ton SGBD fermera probablement une connexion trop longtemps ouverte au boût d'un certain temps.

**stephane.combes** · 10/04/2012, 10h06

Envoyé par questionneuse

POURQUOI ca n'est pas bien

En principe ce n'est pas bien car les ressources du système ne sont pas illimitées alors il est recommandé de les libérer lorsque l'on en a plus besoin.

Envoyé par questionneuse

est ce que cela génére des erreurs

Mis à part le fermeture de la connexion par le serveur déjà évoquée je n'en connais pas d'autres.

S. Combes

**questionneuse** · 10/04/2012, 10h17

Envoyé par Jimmy_

Le principal soucis est de partager ta connexion entre différents utilisateurs.

En quoi est ce un souci que plusieurs users utilisent la même connexion?
J'essaie de comprendre les risques d'erreurs.

**fr1man** · 10/04/2012, 10h26

Si deux utilisateurs partagent la même connexion, ils ne pourront pas exécuter leur requête en même temps. Tu auras donc des problèmes de performance à terme.

Donc autant partir sur des bonnes pratiques, que de vouloir réinventer la roue et en faire une carrée

**questionneuse** · 10/04/2012, 10h42

Envoyé par fr1man

Si deux utilisateurs partagent la même connexion, ils ne pourront pas exécuter leur requête en même temps. Tu auras donc des problèmes de performance à terme.

Donc autant partir sur des bonnes pratiques, que de vouloir réinventer la roue et en faire une carrée

c'est le genre de truc auquel j'ai pensé..
Es tu sur qu'on ne peut pas effectuer 2 requetes en même temps sur une meme connection. As tu un lien?
Merci

ps: je ne réinvente pas la roue, certaine personnes propose cette idée..

**fr1man** · 10/04/2012, 10h49

J'étais tombé sur un lien concernant l'implémentation Mysql.

Je ne sais pas qui propose ça, en tout cas, certainement pas des personnes reconnues.

**questionneuse** · 10/04/2012, 10h59

Envoyé par fr1man

J'étais tombé sur un lien concernant l'implémentation Mysql.

Je ne sais pas qui propose ça, en tout cas, certainement pas des personnes reconnues.

Si tu trouves le lien ca serait sympa de me le filer.
J'ai pas trouvé si c'est possible d'avoir plusieurs requetes simultanés sur une connexion.

**fr1man** · 10/04/2012, 11h35

Je n'ai pas retrouvé le lien.

Une connexion représente une connexion physique vers la base de données, au sens socket. Il n'est indiqué nul part que l'implémentation doive gérer du multithreading pour traiter plusieurs requêtes en même temps.

**questionneuse** · 10/04/2012, 12h32

J'ai un doute sur le fait que les requetes ne s'effectuent pas en parallèle parceque lors de mes tests ou j'avais prés de 300 users je n'ai pas vu ce genre d'erreurs, attentes.
Là, en lancant le test par exemple sur mon appli avec 2 users qui se connectent avec 2 session différentes, les 2 effectuant la même recherche, celle ci dure 3, 4 secondes (temps de la requete SQL).
Je lance les 2 recherches à une seconde prés je devrais avoir un décalage entre les 2 de 3, 4 secondes. Or il y a même pas une seconde de décalage.

**fr1man** · 10/04/2012, 15h03

Admettons, mais peut-être qu'une autre implémentation du driver JDBC ne le gère pas comme cela.

Et puis comment tu gères tes transactions, puisqu'elles sont associées à la connexion ?

Pool de connexion ou singleton

Développement Web en Java

Discussions similaires

Partager

Partager