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的目的，就是为了解决数据库的关闭问题的。

当然，我的数据量非常少，不能说明问题。希望更多的朋友说说自己的想法。