ASP .net中连接数据库是不是不宜用封装啊

Question

为什么我看书上的例子 在每个按钮下都把 SqlConnection SqlCommand SqlReader写了一遍
然后我自己想用封装的方法 结果报错为 未将对象引用设置到对象的实例。
public class DataAccess
{
public static string ConnStr="Data Source=.\\sqlexpress;Initial Catalog=学生信息管理;Integrated Security=True;Pooling=False";
    public SqlDataReader GetReader(string sqlText)
    {
        SqlConnection conn = new SqlConnection(ConnStr);
        SqlCommand command = new SqlCommand(sqlText,conn);
        SqlDataReader dr = null;
        try
        {
            conn.Open();
            dr = command.ExecuteReader(CommandBehavior.CloseConnection);
        }
        catch (Exception e)
        {
            dr.Close();/就是这里报错了
            conn.Dispose();
            command.Dispose();
            throw new Exception(e.ToString());//
        }
        return dr;
    }
}
这是登陆按钮下的事件
protected void Btnlogin_Click(object sender, EventArgs e)
    {
        string name = this.TBname.Text.Trim();
        string password = this.TBpwd.Text.Trim();
        DataAccess da=new DataAccess();        string sqlText="select * from 用户信息 where 用户名=‘"+name+"' and 密码='"+password+"’";        SqlDataReader reader = da.GetReader(sqlText);
        if (reader.Read()==true)
        {
            Session["userName"] = name;
            this.Lblstate.Text = "用户" + name + "登陆成功";
        }
}

Answer 1

因为SqlDataReader dr = null;为空，我觉得就没有必要抛异常时候关闭；弄个SQL帮助类  任意调用，多使用！！

Answer 2

先关闭dateReader
再关闭dataAdapter
最后关闭数据库

Answer 3

把dr.close删了就搞定了 这是咋回事呢

Answer 4

SqlConnection conn = new SqlConnection(ConnStr);
        SqlCommand command = new SqlCommand(sqlText,conn);
        SqlDataReader dr = null;
        try
        {
            conn.Open();//说不定你这句就报错了，然后catch里面的代码！
这时候dr=null,dr.Close()当然要报错
            dr = command.ExecuteReader(CommandBehavior.CloseConnection);
      }
        catch (Exception e)
        {
            dr.Close();/就是这里报错了[/color
[color=#FF0000]]//在这加上一个判断，如果dr！=null而且dr的状态是open的才关闭！
            conn.Dispose();
            command.Dispose();
            throw new Exception(e.ToString());//
        }

Answer 5

CommandBehavior.CloseConnection
楼主加了这句，所以再dr.close()就报错了

Answer 6

把dr.close不要删除，SqlDataReader在读数据的时候是一边读，一遍显示，你删了那里面的数据就没有了，当然报错啊，

Answer 7

你可以参考微软的petshop 4.0中的sqlhelper类的写法。一般的用法，我都是用那个sqlhelper类

Answer 8

问题是 “在执行dr = command.ExecuteReader(CommandBehavior.CloseConnection);这行代码时出现异常，然后被catch捕获，此时dr 对象为空。”
这是由程序过程的设计漏洞“dr并不是一定在有实例对象时被关闭”导致，所以需要改变程序过程以下是我在你代码上做的修改：
 public SqlDataReader GetReader(string sqlText)
        {
            using (SqlConnection conn = new SqlConnection(ConnStr))
            {
                SqlCommand command = new SqlCommand(sqlText, conn);
                SqlDataReader dr = null;
                try
                {
                    conn.Open();
                    dr = command.ExecuteReader(CommandBehavior.CloseConnection);
                    dr.Close();
                }
                catch (Exception e)
                {
                    throw new Exception(e.ToString());//
                }
                finally
                {
                    conn.Close();
                    command.Dispose();
                }
            }
        }

Answer 9

对不起 上面的代码是我本地的，没有完全改成符合你的情况 
上面程序的dr.Close(); 可以不要 
在代码的最后加上 return dr; 即可由于dr 为null是 不需要close();所以无需做关闭操作