Jim, praktik Anda memiliki metode koneksi buka dan tutup di dalam kelas lain sudah sangat lama, pengembangan .NET modern mengikuti pola seperti ini saat ini:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
lihat di sini:Menutup SqlConnection dan SqlCommand c# atau cari di SO untuk ratusan pertanyaan dan jawaban semuanya mengatakan hal yang sama, segera tutup koneksi baik dengan menggunakan seperti dalam contoh ini atau dengan coba/akhirnya di dalam metode yang sama, tidak perlu satu metode untuk membuka dan metode untuk menutup itu, rentan terhadap kesalahan jika terjadi sesuatu di antaranya.