Jim votre pratique d'avoir des méthodes de connexion ouvertes et fermées à l'intérieur d'une autre classe est très ancienne, le développement .NET moderne suit un modèle comme celui-ci de nos jours :
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
voir ici :Fermer SqlConnection et SqlCommand c# ou recherchez dans SO des centaines de questions et réponses qui disent toutes la même chose, fermez la connexion immédiatement soit avec un using comme dans cet exemple ou avec un try/finally à l'intérieur de la même méthode, pas besoin d'une méthode pour ouvrir et d'une méthode pour fermer il, juste sujet aux erreurs si quelque chose se passe entre les deux.
