如何为CA2000修复SqlCommand？

Question

我试图像下面这样执行一个SQL查询，但是Visual抱怨一个CA2000。

public static IDictionary<string, string> TemplateDirectories(string connectionString) {
    var directories = new Dictionary<string, string>();
    using (var connection = new SqlConnection(connectionString)) {
        connection.Open();
        using (var command = new SqlCommand { Connection = connection, CommandText = "select * from PATHS" }) {
            var reader = command.ExecuteReader();
            while (reader.Read())
                directories[reader["CODE"].ToString()] = reader["PATH"].ToString();
                reader.Close();
        }
    }
    return directories;
}

Error CA2000 ...object‘()’并不是沿所有异常路径释放的。在所有对对象的引用超出作用域之前，在对象'new ()‘上调用SqlCommand。

我试过几种方法来解决这个问题，但没有人成功。那怎么解决呢？

Answer 1

这是因为在使用对象初始化器时存在缺陷。

初始化SqlCommand的方式，如果在初始化对象时出现某些异常，则SqlCommand将不被释放。

那么解决方案是什么。用老式的方式声明这个物体，以消除警告-

using (var command = new SqlCommand())
{
    command.Connection = connection;
    command.CommandText="select * from PATHS";
    ...
}

我找到了一篇很好的关于这个问题的文章，它给出了解决这个问题的更多细节和方法-- http://haacked.com/archive/2013/01/11/hidden-pitfalls-with-object-initializers.aspx/。

尽管如此，对于这个复杂的问题，最好使用SqlCommand的构造函数，并像下面这样传递命令文本和连接对象(提供:Damien_ the _Unbeliever的注释)

 string commandText = "select * from PATHS";
 using (var command = new SqlCommand(commandText, connection))
 {
  ...
 }

Answer 2

尝试显式地分配command参数：

using (var command = new SqlCommand())
{
    command.Connection = connection;
    command.CommandText="select * from PATHS";
    var reader = command.ExecuteReader();
    while (reader.Read())
        directories[reader["CODE"].ToString()] = reader["PATH"].ToString();
    reader.Close();
}