关闭没有的DataReader

[incompletewild]

这是出错代码的类

 public  static class DBHelper
    {
        private static SqlConnection connection;
   //private static SqlDataAdapter da;
        public static SqlConnection Connection
        {
            get
            {
                string connectionString = ConfigurationManager.ConnectionStrings["XCgo"].ConnectionString;
                if (connection == null)
                {
                    connection = new SqlConnection(connectionString);
                    connection.Open();
                }
                else if (connection.State == System.Data.ConnectionState.Closed)
                {
                    connection.Open();
                }
                else if (connection.State == System.Data.ConnectionState.Broken)
                {
                    connection.Close();
                    connection.Open();
                }
                return connection;
            }
        }
      public static DataTable GetDataTable(string safeSql)
        {
            DataSet ds = new DataSet();
            SqlCommand cmd = new SqlCommand(safeSql, Connection);
            SqlDataAdapter da = new SqlDataAdapter(cmd);
            cmd.Dispose();

                da.Fill(ds);//异常出在这里，提示There is already an open DataReader associated with this Command which must be closed first. 


      
            da.Dispose();
          
            connection.Close();
            return ds.Tables[0];
        }
}

我在前台频繁点一个连接调用这个GetDataTable方法就会报一个datareader未关闭的错误,但是这个方法里没用到datareader

[人生就是赌]

获取datatable的底层还是使用datareader来读取的，所以会报那个错
那个错的来源是因为你把SqlConnection 定义为了static，数据层连接绝对不能这样做的
这样做意味着你的数据库一直处于一个单连接，并发的时候报错无疑

[syc958]

为什么自己不写一。每次new一个conn,用完就关掉并dispose呢....学人家用也不是错，你应该把它的所有代码都弄清了再用才好呀....

[readfuture]

[Quote=引用 1 楼 elevenbus 的回复:]
获取datatable的底层还是使用datareader来读取的，所以会报那个错
那个错的来源是因为你把SqlConnection 定义为了static，数据层连接绝对不能这样做的
这样做意味着你的数据库一直处于一个单连接，并发的时候报错无疑
[/Quote]
分析的挺到位，支持

[AuC]

SqlDataAdapter da = new SqlDataAdapter(safeSql, Connection);
                DataTable dt = new DataTable();
                da.Fill(dt);
                return dt;

我习惯这样写
一般都不报错的

[beckfun]

[Quote=引用 4 楼 zyaccp30 的回复:]
C# codeSqlDataAdapter da=newSqlDataAdapter(safeSql, Connection);
DataTable dt=newDataTable();
da.Fill(dt);returndt;

我习惯这样写
一般都不报错的
[/Quote]
如果你连续两次去调用就会报错，或者同一类中调用第二次也会报错

[lytgmy]

static SqlConnection connection是类的，不是实例的，只是单一连接了，是有问题的！

[我在地球]

学习

[e博运智慧物流]

[Quote=引用 1 楼 的回复:]
获取datatable的底层还是使用datareader来读取的，所以会报那个错
那个错的来源是因为你把SqlConnection 定义为了static，数据层连接绝对不能这样做的
这样做意味着你的数据库一直处于一个单连接，并发的时候报错无疑
[/Quote]
不错，就是这个原因