SqlHelper类编写前奏：DataReader关闭链接出现问题

浏览数：26 / 时间：2015年06月12日

SqlHelper是一个执行数据库操作的助手类，但是当我们没学过DataSet之前，要想使用using搭配SqlConnection和SqlCommand写出一个真正独立的SqlHelper都是不太可能的。

比如：一个常规的ExecuteReader方法如果使用上述做法，代码如下：

using System.Data.SqlClient;

namespace ExecuteScalar.libs
{
    class SqlHelper
    {
        public static SqlDataReader ExecuteScalar()
        {
            //使用using管理资源
            using (SqlConnection conn = new SqlConnection("server=.;database=WebSite;uid=sa;pwd=123456"))
            {
                using (SqlCommand cmd = conn.CreateCommand())
                {
                    cmd.CommandText = "select * from UserInfo";
                    conn.Open();

                    return cmd.ExecuteReader();
                }
            }
        }//end ExecuteScalar

    }
}

我们在窗体的按钮点击事件中使用这个类的ExecuteScalar方法获取的SqlDataReader对象

private void button1_Click(object sender, EventArgs e)
{
    //使用dr变量接收ExecuteReader方法产生的SqlDataReader对象
    SqlDataReader dr = libs.SqlHelper.ExecuteReader("select * from UserInfo");

    MessageBox.Show(dr.HasRows.ToString());
}

执行点击事件，发现代码报异常：阅读器关闭时尝试调用 HasRows 无效

因为使用using在using的作用域结束之前会自动调用Dispose方法，导致连接关闭。而SqlDataReader对象读取的是服务器的数据，你通过ExecuteReader返回的一个SqlDataReader对象值保存了指向服务器结果集的指针并没有数据，数据还是要依赖于conn来读取的。结论：因此这里不能使用using

既然不using，自然不会报错，conn释放资源怎么办呢。于是就想到了这种办法

using System.Data.SqlClient;

namespace ExecuteScalar.libs
{
    class SqlHelper
    {
        //将conn定义为静态成员，要可以在外部手动释放掉
        public static SqlConnection conn;

        public static SqlDataReader ExecuteScalar()
        {
            //使用using管理资源
            using (conn = new SqlConnection("server=.;database=WebSite;uid=sa;pwd=123456"))
            {
                using (SqlCommand cmd = conn.CreateCommand())
                {
                    cmd.CommandText = "select * from UserInfo";
                    conn.Open();

                    return cmd.ExecuteReader();
                }
            }
        }//end ExecuteScalar

    }
}

使用的时候可以这样用：

private void button1_Click(object sender, EventArgs e)
{
    //使用dr变量接收ExecuteReader方法产生的SqlDataReader对象
    SqlDataReader dr = libs.SqlHelper.ExecuteReader("select * from UserInfo");

    MessageBox.Show(dr.HasRows.ToString());
    //关闭SqlDataReader
    dr.Close();
    libs.SqlHelper.conn.Close();

}

这样就达到了释放conn链接资源的目的。

且不说这种方法多么的不规范，多么违背面向对象程序设计的思想。光说这个手动释放，有多少程序员能够准确记得这一步。

这样的做法不能使SqlHelper成为一个真正独立真正封装的类。

基于此，我们就该在SqlHelper中放弃using 和 SqlDataReader的搭配。转而使用DataSet和SqlDataAdapter方式。

DataSet就是一个离线数据集，方便管理和遍历。

因此真正的SqlHelper.cs应该是这样写的：

using System.Data.SqlClient;
using System.Data;

namespace ExecuteScalar.libs
{
    class SqlHelper
    {
        public static DataSet GetDataSet(string sql)
        {
            SqlDataAdapter sda = new SqlDataAdapter(sql,"server=.;database=WebSite;uid=sa;pwd=123456");

            DataSet dSet = new DataSet();

            sda.Fill(dSet);

            return dSet;

        }

    }
}