Jim, la tua pratica di avere metodi di connessione aperti e chiusi all'interno di un'altra classe è molto vecchia, lo sviluppo moderno di .NET segue uno schema come questo al giorno d'oggi:
using (SqlConnection conn = new SqlConnection("connection string here"))
using (SqlCommand cmd = new SqlCommand("sql query", conn))
{
// execute it blah blah
}
vedere qui:Chiudere SqlConnection e SqlCommand c# o cerca in SO centinaia di domande e risposte che dicono tutte uguali, chiudi immediatamente la connessione con un utilizzo come in questo esempio o con un tentativo/finalmente all'interno dello stesso metodo, non è necessario un metodo per aprire e un metodo per chiudere esso, solo soggetto a errori se succede qualcosa nel mezzo.
