Używam wzoru, który wygląda tak często. Zastanawiam się, czy to jest w porządku, czy też istnieje najlepsza praktyka, której tutaj nie stosuję.

W szczególności zastanawiam się; w przypadku, gdy wyjątek jest zgłaszany, kod, który mam na końcu, wystarczy, aby zapewnić odpowiednie połączenie?

public class SomeDataClass : IDisposable
{
    private SqlConnection _conn;

    //constructors and methods

    private DoSomethingWithTheSqlConnection()
    {
        //some code excluded for brevity

        try
        {
            using (SqlCommand cmd = new SqlCommand(SqlQuery.CountSomething, _SqlConnection))
            {
                _SqlConnection.Open();
                countOfSomething = Convert.ToInt32(cmd.ExecuteScalar());
            }
        }
        finally
        {
            //is this the best way?
            if (_SqlConnection.State == ConnectionState.Closed)
                _SqlConnection.Close();
        }

        //some code excluded for brevity
    }

    public Dispose()
    {
        _conn.Dispose();
    }
}