SQLConnection用Using语句，从里面调用SQLDataReader吗？

Question

只是想确保这是调用连接并从数据库中获取数据的最佳方式，或者我应该如何在 using 语句之外调用 datareader？ （为了更快地关闭连接？）或者您个人对此有何更改？

using (SqlConnection cn = new SqlConnection(connStr))
        {
            using (SqlCommand cm = new SqlCommand(connStr, cn))
            {  
                cm.CommandType = CommandType.StoredProcedure;
                cm.CommandText = "GetExchRatesByDate";
                cm.Parameters.Add("@Date", SqlDbType.VarChar).Value = txtStartDate.Text;
                cn.Open();
                SqlDataReader dr = cm.ExecuteReader();

                while (dr.Read())
                {
                    firstName = (string)dr["GivenName"];
                    lastName = (string)dr["sn"];;
                }
                dr.Close();
            }
        }

Answer 1

您无法在 using 语句之外成功调用 datareader，因为它需要打开的连接来读取数据。

连接将以您所拥有的方式足够快地关闭，甚至不会真正“关闭”。它将返回到连接池（假设您正在使用一个）。由于您可能是这样，因此您无需担心在此上下文中连接关闭的速度有多快，因为需要连接的应用程序的其他部分将从池中获取可用的连接。假设您没有真正高流量的应用程序，在这种情况下它可能会变得很重要，但是有很多很多并发用户，您可以通过增加池中的连接数来缓解这个问题。

Chris 也提出了一个很好的观点：它应该在 using 语句中：

  SqlDataReader dr = cm.ExecuteReader();

                while (dr.Read())
                {
                    firstName = (string)dr["GivenName"];
                    lastName = (string)dr["sn"];;
                }
                dr.Close();

在这种情况下，如果你的阅读器抛出异常，它永远不会到达 dr.Close(); 所以它会保持打开状态的时间比需要的时间长得多（甚至可能在应用程序的生命周期内）。

数据读取器

Answer 2

您不仅无法在 using 语句之外调用 SqlDataReader，因为变量中声明的所有内容都将被释放，而且您需要一个打开的连接来读取数据，您最好编写一个将返回的对象，甚至是您的对象的列表。

public class MyObject {
    public string FirstName { get; set; }
    public string Surname { get; set; }
}

public IEnumerable<MyObject> GetObjects() {
    ICollection<MyObject> myObjects = new List<MyObject>();

    using (SqlConnection cn = new SqlConnection(connStr))
    {
        using (SqlCommand cm = new SqlCommand(connStr, cn))
        {  
            cm.CommandType = CommandType.StoredProcedure;
            cm.CommandText = "GetExchRatesByDate";
            cm.Parameters.Add("@Date", SqlDbType.VarChar).Value = txtStartDate.Text;
            cn.Open();

            using(SqlDataReader dr = cm.ExecuteReader()) 
                while (dr.Read()) {
                    MyObject myObject = new MyObject();
                    myObject.FirstName = (string)dr["GivenName"];
                    myObject.Surname = (string)dr["sn"];
                    myObjects.Add(myObject);
                }
        }
    }
    return myObjects;
}

Answer 3

datareader 实现了 IDisposable，因此它也应该包含在 using 子句中。其他一切看起来都不错。

Answer 4

其他人已经介绍了它，但我喜欢使用的另一件事是 AddWithValue:

cm.Parameters.Add("@Date", SqlDbType.VarChar).Value = txtStartDate.Text;

可以写为：

cm.Parameters.AddWithValue("@Date", txtStartDate.Text);

Answer 5

我会考虑将 SQLConnection 放入工厂方法中。 （如果它被称为负载）

另外，我不会在 dr.Read 周围有一个 while 循环
你只期待一个答案，那么当没有结果或有很多结果时会发生什么？？？
不太确定我是否喜欢选角，但可能还可以。

我认为您围绕此代码进行了单元测试，并传入了连接字符串等...（显然工厂的想法会使单元测试变得更加困难，所以也许不值得做...）

您的 using 语句对我来说看起来很好。