Jim din praxis att ha öppna och nära anslutningsmetoder i en annan klass är mycket gammal, modern .NET-utveckling följer ett mönster som detta nuförtiden:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
se här:Stänger SqlConnection och SqlCommand c# eller sök i SO efter hundratals frågor och svar som alla säger detsamma, stäng anslutningen omedelbart antingen med en användning som i detta exempel eller med ett försök/äntligen i samma metod, inget behov av en metod för att öppna och på metod för att stänga det, bara benägna att fel om något händer däremellan.