Question

.net/C# 数据访问层 OleDbDataReader的写法

public OleDbDataReader GetDataReader(string Sqlstring)
{
Open();

OleDbCommand cmd = new OleDbCommand(Sqlstring, Connection);
OleDbDataReader reader = cmd.ExecuteReader();

Close();
return reader;
}

之前是这么写的，但while (drd.Read()) {}这样的就读不出数据，报错：
[InvalidOperationException: Invalid attempt to call Read when reader is closed.]

后来就把Close();与return reader;互换了位置，成了：
public OleDbDataReader GetDataReader(string Sqlstring)
{
Open();

OleDbCommand cmd = new OleDbCommand(Sqlstring, Connection);
OleDbDataReader reader = cmd.ExecuteReader();

return reader;
Close();
}

这样就很正常，能读出数据。但后来在一些需要执行update类的sql语言的页面上，又报错“无法保存；正被别的用户锁定(Access数据库的错误)”的错误。查了一下，说是数据库没及时关闭。
这样一来，感觉上面代码里的Close();就形同虚设了，不得不手动在执行update类的sql语言的代码下面添加如db.Close();之类的关闭数据库的代码。

我觉得这不是办法，我的问题是，是不是我数据访问层里 OleDbDataReader的写法根本就不对？或者while (drd.Read()) {}这样的使用DataReader的办法不对？
或者是Close();的写法不对
public void Close()
{
if (Connection != null)
{Connection.Close();}
}

不清楚究竟哪里有问题。想要一个比较优雅的解决办法。

Answer 1

我认为理想的 优雅的解决方案就是：
1、GetDataReader 方法中不要写 Open();与Close();，把 Open()拿到外面写。

2、在 while (drd.Read()) {} 之后写 Close();

3、 Open();与Close();尽量不要写在某个方法体内，这样写灵活性差，代码就容易出现你说的问题。

4、我的理论就是 在你要访问DB的时候才 Open();，在你访问结束以后就马上Close();

例如：
Open(); // 开

drd = GetDataReader(xx,xx);

while (drd.Read())
{
//...
}
Close(); // 关

Answer 2

你有没有发现你的
return reader;
Close();
close是放在 return后面 程序不会执行到这的 当然就不能关闭啦
写法不对吧. 要用OleDbDataReader connection是必须不能关闭的
你可以在这个方法把OleDbDataReader 处理
可以在你用完OleDbDataReader 时调用connection关闭方法 而且要先关闭OleDbDataReader
reader.close()
再关闭connection .close()
还有不清楚 继续问吧!

Answer 3

public OleDbDataReader GetDataReader(string Sqlstring)方法中Close()方法没有执行。这样数据库就没有关闭，接下来对数据库的操作危险可想而知。。。建议您可以把数据库的打开关闭操作放在这个方法外面执行在调用这个方法之前connection.Open下，用完这个方法马上connection.Close()。这样就可以了

Answer 4

Close写法不对
if (conn.State == ConnectionState.Open)
{
conn.Close();
}