DataReader的关闭问题疑惑篇

DataReader的关闭问题疑惑篇

昨天一个朋友使用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的目的，就是为了解决数据库的关闭问题的。

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