Jim, jouw gewoonte om open en close-verbindingsmethoden binnen een andere klasse te gebruiken, is heel oud, moderne .NET-ontwikkeling volgt tegenwoordig een patroon als dit:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
zie hier:SqlConnection en SqlCommand c# sluiten of zoek in SO naar honderden vragen en antwoorden die allemaal hetzelfde zeggen, sluit de verbinding onmiddellijk met een gebruik zoals in dit voorbeeld of met een try/finally binnen dezelfde methode, het is niet nodig om één methode te openen en een methode om te sluiten het, alleen gevoelig voor fouten als er iets tussendoor gebeurt.