Discussion :

[questionneuse]

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]

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]

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_]

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]

Bonjour,

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

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_]

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.

[questionneuse]

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]

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

[stephane.combes]

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.

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