c# - 处理 SQL 命令并关闭连接

Question

直到现在我总是使用类似的结构从数据库中获取数据并填充数据表

public static DataTable GetByID(int testID)
        {
        DataTable table = new DataTable();
        string query = @"SELECT * FROM tbl_Test AS T WHERE T.testID = @testID";

        using (SqlConnection cn = new SqlConnection(Configuration.DefaultConnectionString))
        {
            SqlCommand cmd = new SqlCommand(query, cn);
            cmd.Parameters.Add("@testID", SqlDbType.Int).Value = testID;

            cn.Open();
            table.Load(cmd.ExecuteReader());
        }

        return table;
    }

现在我在构建分析中看到了一些警告:

TestService.cs (37):CA2000:Microsoft.Reliability:在方法“TestService.GetByID(int)”中，对象“表”未沿所有异常路径放置。在对对象“表”的所有引用超出范围之前调用 System.IDisposable.Dispose。

TestService.cs (42):CA2000:Microsoft.Reliability:在方法“TestService.GetByID(int)”中，在对对象“cmd”的所有引用超出范围之前调用 System.IDisposable.Dispose。

我应该更改我的代码吗

    public static DataTable GetByID(int testID)
    {
        DataTable table = new DataTable();
        string query = @"SELECT * FROM tbl_Test AS T WHERE T.testID = @testID";

        using (SqlConnection cn = new SqlConnection(Configuration.DefaultConnectionString))
        {
            using (SqlCommand cmd = new SqlCommand(query, cn))
            {
                cmd.Parameters.Add("@testID", SqlDbType.Int).Value = testID;

                cn.Open();
                table.Load(cmd.ExecuteReader());
            }
        }

        return table;
    }

如何处理 DataTable 对象？将 SqlCommand 放在 using 中是一个好习惯吗？

谢谢

干杯

Answer 1

你也应该这样做:

using (SqlDataReader reader =
            cmd.ExecuteReader
                (CommandBehavior.CloseConnection))
        {
            table.Load(reader);
        }

加载表格时