Джим, твоята практика да имаш отворени и затворени методи за свързване в друг клас е много стара, модерното развитие на .NET следва схема като тази в наши дни:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
вижте тук:Затваряне на SqlConnection и SqlCommand c# или потърсете в SO за стотици въпроси и отговори, всички казващи едно и също, незабавно затворете връзката или с използване като в този пример, или с опит/накрая в рамките на същия метод, няма нужда един метод да се отваря и един метод да затваря просто е склонен към грешки, ако нещо се случи между тях.
