SqlDataReader的关闭问题
昨天一个朋友使用Repeater绑定数据源时,老是出现"阅读器关闭时尝试调用 FieldCount 无效。"错误。
我看了他的代码,使用的是SqlHelper类下面的ExecuteReader方法,返回一个SqlDataReader进行绑定。
public static SqlDataReader ExecuteReader(CommandType cmdType, string cmdText, params SqlParameter[] cmdParms) { SqlCommand cmd = new SqlCommand(); SqlConnection conn = new SqlConnection(CONN_STRING); // we use a try/catch here because if the method throws an exception we want to // close the connection throw code, because no datareader will exist, hence the // commandBehaviour.CloseConnection will not work try { PrepareCommand(cmd, conn, null, cmdType, cmdText, cmdParms); SqlDataReader rdr = cmd.ExecuteReader(CommandBehavior.CloseConnection); cmd.Parameters.Clear(); return rdr; } catch { conn.Close(); throw; } }
然后,他自己又编写了一个类,GetData,用其中的一个方法来调用SqlHelper.
public SqlDataReader GetReader() { using (SqlDataReader reader = SqlHelper.ExecuteReader(CommandType.StoredProcedure, "GetAllUser", null)) { return reader; } }
其中GetAllUser是一个不带参数,查询所有用户的存储过程,最后通过返回的reader进行数据源绑定。
他出现的这个错误,明显是在读取reader之前,reader已经被关闭(using的原因)。因此会读不到数据,就会报那种错误。因此,把他的调用方法改了一下:
public SqlDataReader GetReader() { try { SqlDataReader reader=SqlHelper.ExecuteReader(CommandType.StoredProcedure, "GetAllUser", null); return reader; } catch (Exception) { return null; } }
这样,就不会报错了,UI层进行数据绑定也没有任何问题。但是,随之一个新的问题出现了:SqlDataReader没有手动关闭,会不会造成连接池达到最大值?
我们注意到,在SqlHelper里面,使用到了CommandBehavior.CloseConnection,这个的作用是"关闭SqlDataReader 会自动关闭Sqlconnection." 也就是说,关闭Sqlconnection是在关闭SqlDataReader的前提下进行,还是需要手动关闭SqlDataReader。又要返回SqlDataReader,又要关闭它,怎么办呢?有的网友提出:在return reader之前,先关闭reader,即进行reader.Close();
实际上这样是不行的,会报与前面一样的错误,在读取数据之前,已经被关闭。
CommandBehavior.CloseConnection用于关闭数据库连接,这是肯定的,但它会不会一起把SqlDataReader也一起关闭了呢。也就是说,用了CommandBehavior.CloseConnection,是不是就不用再手动关闭SqlDataReader了。
我们中以使用SqlHelper,然后在前台网页里面进行测试
protected void bind() { SqlConnection conn = new SqlConnection(ConfigurationManager.ConnectionStrings["constr"].ToString()); conn.Open(); SqlCommand cmd = new SqlCommand("GetAllUser", conn); SqlDataReader sdr = cmd.ExecuteReader(CommandBehavior.CloseConnection); repeater1.DataSource = sdr; repeater1.DataBind(); Response.Write(sdr.IsClosed.ToString()+"<br/>"); Response.Write(conn.State.ToString()); }
输出的结果是
True
Closed
说明SqlConnection和SqlDataReader都已经被关闭了。
如果把CommandBehavior.CloseConnection去掉,输出的结果则是:
False
Open
由此可见,使用了CommandBehavior.CloseConnection之后,读取完数据后,系统自动关闭了SqlDataReader和SqlConnection。听说当初微软弄出CommandBehavior.CloseConnection的目的,就是为了解决数据库的关闭问题的。
当然,我的数据量非常少,不能说明问题。希望更多的朋友说说自己的想法。