昨天一个朋友使用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的目的,就是为了解决数据库的关闭问题的。
当然,我的数据量非常少,不能说明问题。希望更多的朋友说说自己的想法。