Jim, Ihre Praxis, offene und geschlossene Verbindungsmethoden innerhalb einer anderen Klasse zu haben, ist sehr alt, die moderne .NET-Entwicklung folgt heutzutage einem Muster wie diesem:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
siehe hier:Closing SqlConnection and SqlCommand c# oder suchen Sie in SO nach Hunderten von Fragen und Antworten, die alle dasselbe sagen, schließen Sie die Verbindung sofort, entweder mit einem using wie in diesem Beispiel oder mit einem try/finally innerhalb derselben Methode, ohne dass eine Methode geöffnet und eine Methode geschlossen werden muss es, nur anfällig für Fehler, wenn dazwischen etwas passiert.
