如何找到未关闭的DataReader？

Question

我在 ASP.Net 1.1 中维护一个大型应用程序，该应用程序已被许多人（包括学生）更改。显然，代码很少被审查，所以有很多……呃……愚蠢的错误。最有问题的是从未关闭的 DataReader。 而且到处都是，每页最多十个。由于这个项目大约有一百页，轻松有 300 多个类，一想到要找到所有未关闭的 DataReader，我就感到沮丧。

我知道这是没有希望的，但是有没有一种简单的方法可以找到所有这些未关闭的DataReader？一些软件或 Visual Studio 2003 调整...

Answer 1

那么，使用 DataReader 的“正确”方法是

using (SqlDataReader dr = ...) {
    ...
}

或者

SqlDataReader dr = ...
try {
    ...
} finally {
    dr.Close();
}

因此，简单的解决方案就是对 SqlDataReader 和 IDataReader 进行完整项目搜索。如果它

• 不在 using 子句中或者
• 没有紧随其后 try

那么它很可能使用不正确并且需要修复。

请注意，以下代码：

SqlDataReader dr = ...
...
// do something
...
dr.Close();

是不正确的，因为在 // do some 期间引发的异常将使 DataReader 保持打开状态。

Answer 2

尝试查找和替换下的文件中替换功能。使用正则表达式。

1. 将 {SqlDataReader.*}$ 替换为 using(\1) \{。
2. 然后将 using({using(.*)} \{ 替换为 \1。
3. 编译并检查每个错误并仅放置右大括号。

但是 make确保在开始之前备份文件。

这样，您就可以跳过那些已经有 using 块来关闭数据读取器的文件。