Discussion :

[takinelinfo]

Bonjour,

Je viens de me rendre compte d'une chose surprenante, je n'arrive pas à fermer la connexion à ma base de données.
Techno: WPF
Base : PostgreSQL
Provider: NPGSQL

Voici un bout de code:

Code c# :

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
 
public partial class MainWindow : Window
{
	public MainWindow()
	{
		InitializeComponent();
 
 
		var connection = new NpgsqlConnection(ConnectionStringGet("..........."); //Step 0
		connection.Open(); //Step 1
 
		Thread.Sleep(3000);
 
		connection.Close();
		connection.Dispose();
		connection = null;
		//step2
	}
}

Je regarde les connexions ouvertes grâce à PgAdminIII
Step 0 : 0 connexion d'ouverte.
Step 1 : 1 connexion d'ouverte.
Step 2 : 1 connexion d'ouverte.

Par contre si je ferme l'application, la connexion à la base de données est coupée.
Je pense que c'est mieux d'ouvrir et fermer la connexion à chaque fois non?
Comment se fait il que ça ne marche pas dans mon code?
Merci d'avance

[Er3van]

Le code que tu donnes c'est celui que tu testes pour donner l'état 0/1/2 ?
Tu fais du pooling de connexions ?
Et si tu mets ça dans un TransactionScope ça fait pareil ?
Comment testes-tu que la connexion est ouverte ?

[tomlev]

A priori je dirais que c'est dû au pooling, et que c'est donc normal...

Essaie d'ajouter ça après avoir fermé la connection :

Code :

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

NpgsqlConnection.ClearAllPools();

ou bien ça avant de la fermer :

Code :

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

NpgsqlConnection.ClearPool(connection);

[takinelinfo]

@Er3wanLe code que tu donnes c'est celui que tu testes pour donner l'état 0/1/2 ?
Oui
Tu fais du pooling de connexions ?
Pour être très honnête, je ne sais pas ce que c'est.
Et si tu mets ça dans un TransactionScope ça fait pareil ?
TransactionScope, c'est l'utilisation du Using?
Comment testes-tu que la connexion est ouverte ?
PgAdminIII > Outils > Etat du serveur

@towlev

Code c# :

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

NpgsqlConnection.ClearAllPools();

ça fonctionne parfaitement avec ça.

D'après vos réponses, deux questions:
1) Faut il ou y a 'til une utilité à utiliser le pooling de connexion?
2) Je viens du monde PHP/MySQL, sur un client lourd faut il ouvir et fermer la connexion à la base de données à chaque requête?

Encore merci à vous deux

[Nathanael Marchand]

D'après vos réponses, deux questions:
1) Faut il ou y a 'til une utilité à utiliser le pooling de connexion?

Oui! Ouvrir une connexion est couteux en temps et en ressources.

2) Je viens du monde PHP/MySQL, sur un client lourd faut il ouvir et fermer la connexion à la base de données à chaque requête?

Tu peux le faire à chaque transaction si le pooling est activé sinon ca risque d'être trop couteux.

[takinelinfo]

@Nathanael

Code c# :

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
 
var connection = new NpgsqlConnection(ConnectionStringGet(new DatabaseSetting("152.18.80.215", "postgres", "a1b2c3d4", "geco_faa_faam", "5432", "UTF8", "35")));
 
Console.WriteLine("Query 1");
connection.Open();
var command = new NpgsqlCommand("SELECT * FROM usertest", connection);
command.ExecuteNonQuery();
connection.Close();
Thread.Sleep(3000);
 
Console.WriteLine("Query 2");
connection.Open();
var command2 = new NpgsqlCommand("SELECT * FROM usertest", connection);
command.ExecuteNonQuery();
connection.Close();
 
Thread.Sleep(3000);
connection.ClearPool();

En effet, lorsque je fais ça il ne ferme pas la connexion à la base de données entre les deux requêtes.
Si le pooling est activé, y a t'il une différence entre:

Code c# :

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

1
2
3
4
5
6
7
connection.Open();
//Requête
connection.Close();
 
connection.Open();
//Requête2
connection.Close();

et

Code c# :

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

1
2
3
4
connection.Open();
//Requête
//Requête2
connection.Close();

[tomlev]

Il y a une différence du point de vue de .NET, dans le sens où l'objet NpgsqlConnection sera dans l'état ouvert, puis fermé ; mais la connexion native sous-jacente restera dans l'état ouvert si le pooling est activé, et sera réutilisée la prochaine fois que tu ouvrira la connexion avec NpgsqlConnection

[Nathanael Marchand]

Je pense pas. Peut etre que le connection.Close() rollback la transaction courante si elle a pas été commitée auquel cas ca ferait une différence.

[SaumonAgile]

Si le pooling est activé, Open réserve une connexion du pool, Close la libère. Le pool se débrouille pour faire ce qu'il veut avec ses connexions sous-jacentes.

Donc pour résumer, si le pooling est activé:

Code :

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

1
2
3
4
C1.Open
C2.Open
C1.Close
C2.Close

Ceci récupère deux connexions différentes du pool.

Code :

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

1
2
3
4
C1.Open
C1.Close
C2.Open
C2.Close

Ceci peut potentiellement réutiliser la même connexion mais rien ne permet de prédire ce comportement à l'avance.

[takinelinfo]

Et du coup avec le pooling est ce que l'utilisation du pattern singleton pour l'accès au données ne perd pas un peu de son intérêt puisque même si l'on a une seule instance, plusieurs connexions peuvent être ouvertes à un instant T?

[tomlev]

Et du coup avec le pooling est ce que l'utilisation du pattern singleton pour l'accès au données ne perd pas un peu de son intérêt puisque même si l'on a une seule instance, plusieurs connexions peuvent être ouvertes à un instant T?

Bah en fait tu peux pas réouvrir une connexion fermée il me semble, il faut créer une nouvelle instance... donc le pattern singleton va pas beaucoup t'aider ici

[Nathanael Marchand]

Et du coup avec le pooling est ce que l'utilisation du pattern singleton pour l'accès au données ne perd pas un peu de son intérêt puisque même si l'on a une seule instance, plusieurs connexions peuvent être ouvertes à un instant T?

Le pattern singleton est de toute facon une mauvaise idée pour l'accès aux données...

[Robotech]

Le pattern singleton est de toute facon une mauvaise idée pour l'accès aux données...

Simple question.... pourquoi ?

[Nathanael Marchand]

Car un singleton ca se partage et qu'il n'est pas de bonnes pratique de partager les accès aux données. Chaque unité logique doit avoir la sienne. Typiquement en ASP.Net c'est un accès par requête utilisateur, en client lourd un par par requête.

[tomlev]

en client lourd un par par requête.

Ca fait beaucoup quand même... j'aurais plutôt dit "un par lot de requêtes"
(ouais je chipote ^^)

[Nathanael Marchand]

Oui alors non j'ai zappé un bout de ma phrase en fait!
Pour un service WCF c'est un par requête, pour un client lourd un par écran (sauf si c'est un écran un peu complexe avec de l'asynchrone et/ou du multi-thread).

[Robotech]

Merci de ta réponse