这样获得DataReader合理吗？请各位大哥帮我看看～

[xiaomaolover]

public SqlDataReader GetDataReader(string str_Sql)
{
Open();
SqlCommand cmd = new SqlCommand(str_Sql,cn);
SqlDataReader dataReader = cmd.ExecuteReader(System.Data.CommandBehavior.CloseConnection);
return dataReader;
}


这样DataReader关闭了吗?Connection关闭了吗？
如果用了好多这个。会不会很慢？
请各位大哥帮我看看～

[saucer]

it looks ok

>>这样DataReader关闭了吗?

no

>>Connection关闭了吗？

no

>>如果用了好多这个

make sure you call DataReader.Close() after you are done with the DataReader, it will automatically close the database connection since you used CommandBehavior.CloseConnection

The general guideline with database connection is, open them as late as possible, and close them as soon as possible

[heroman1979]

实际上,这种情况我觉得用DATASET生成DATAVIEW比较好,因为DATAREADER是非连接式的,他只是提供了一个命令通道,本身不保存命令产生的数据,所以,当连接关闭后,也就不能访问DATAREADER了,换句话来说,以上代码如果使用DATAREADER,那么,连接是不会立刻关闭的

[活靶子哥哥]

PetShop中是这样的
public void RunProc(string procName, out SqlDataReader dataReader)
{
SqlCommand cmd = CreateCommand(procName, null);
dataReader = cmd.ExecuteReader(System.Data.CommandBehavior.CloseConnection);
}

[wy_rover]

但返回DataReader读了之后,一定要Close(), 这样Connection才会Close