内部でSqlConnectionを適切に処分する

Question

SqlConnectionは適切に処分されていますか？

/// <summary> 
/// Executes the stored procedure using the parameters. 
/// </summary> 
/// <param name="storedProcedureName"></param> 
/// <param name="parameters"></param> 
/// <param name="timeoutValue"></param> 
/// <returns></returns> 
public DataSet ExecuteStoredProcedure(string storedProcedureName, List<SqlParameter> parameters, int timeoutValue = 0) 
{ 
    var ds = new DataSet(); 
    var sqlConnection = (SqlConnection)Database.Connection; 

    using (var command = new SqlCommand(storedProcedureName, sqlConnection)) 
    { 
     //setting CommandTimeout specified value 
     if (timeoutValue > 0) 
     { 
      command.CommandTimeout = timeoutValue; 
     } 

     command.CommandType = CommandType.StoredProcedure; 

     foreach (var param in parameters) 
     { 
      command.Parameters.Add(param); 
     } 

     _databaseHelper.OpenConnection(command); 
     var adapter = new SqlDataAdapter(command); 
     adapter.Fill(ds); 

     _databaseHelper.CloseConnection(command); 
    } 

    return ds; 
} 

Answer 1

ここで新しい接続を開くわけではないので、とにかく処理されることはありません。

SqlCommandあなたがusing構成を使用しているため、どの場合でも正常に終了します。たとえばvar adapter = new SqlDataAdapter(command);のためのこの行は、例外がスローされます場合は、_databaseHelper.CloseConnection(command);は明らかに文句を言わない

Answer 2

と呼ばれることSqlConnectionのが適切に配置されますしていますか？

号_databaseHelper.CloseConnection(command)は、それがDispose()を呼び出すことと同じですcommand.Connection.Close()コールを、含むかもしれないことを意味し、それはfinallyブロックではないので、これは「適切に」と記載されません。