关闭“卸载”中的连接方法

Question

我继承了一个 Web 框架，以前的开发人员在页面生命周期的 init/unload 方法中打开和关闭了他的数据库连接。本质上构造函数是这样的（简化以演示要点）；

public class BasePage
{
   protected DBConnection _conn;

   public BasePage()
   {
      Init += StartConnection;
      Unload += EndConnection;
   }

   private void StartConnection(object sender, EventArgs e)
   {
      _conn = new DBConnection(Application["connectionstring"].ToString());   
   }

   private void EndConnection(object sender, EventArgs e)
   {
      if (_conn == null)
         return;

      if (_conn.Connection.State == ConnectionState.Open)
      {
     _conn.Close();
         _conn.Dispose();
      }
   }
}

自从我来到这里以来，发展速度相当快，所以我从未停下来考虑它。最近，访问量有所增加，我们开始收到可怕的“超时已过期。从池中获取连接之前超时时间已过...”错误。

我目前正在检查其余的代码，寻找可能的连接泄漏，但上面的代码从来没有完全适合我，我想消除它作为潜在的罪魁祸首。那么接下来的问题；

即使发生异常，我是否可以始终调用“卸载”方法？或者任何人都可以使用上述模式看到任何其他潜在问题，从而使其成为这些连接泄漏的主要嫌疑人？

干杯，

Mikey

编辑：在调试中，即使出现异常，卸载方法也总是被调用。我真的只需要知道不会调用此方法的任何场景，这样我就可以弄清楚这是否是我需要首先重构的部分。

编辑：感谢那些迄今为止做出回应的人，但请不要再提出有关 IDisposable 类或“使用”或“catch/finally”模式的建议 - 这不是我的问题！我的问题具体是，页面是否可以运行其“Init”事件，但随后无法运行“Unload”事件，以及为什么会发生这种情况。

Answer 1

我不知道这是否安全，但我浏览了 System.Web.UI.Page 类的源代码，卸载事件由私有 ProcessRequestCleanup() 触发，除非请求是异步的或跨页面请求。对 clean up 方法的调用位于与 ProcessRequest 周围的 try 块耦合的 finally 块内。处理请求正在触发从PreInit到Render的所有页面生命周期事件。这意味着即使发生异常，卸载也将始终被触发（异步和跨页情况除外）。

然而，在我的页面中包含此代码我会感到非常不安，因为卸载的行为没有准确记录。

Answer 2

我总是使用如下的 using block ，

using( SqlConnection)
{

}

创建一个类，那么它永远不会引起任何问题

这样如果您不想编写用于打开连接的代码并再次

public class SqlConnectionManager
{
    public SqlConnection GetSqlConnectionManager()
    {
       //create and return connection
       //SqlConnection con = new SqlConnection();
       //return con;
     }

}

在您的类文件中

SqlConnection conn = null;
using (conn = (new SqlConnectionManager()).GetSqlConnectionManager())
{
     //do work with connection
}

所以通过上述方式您不需要再次编写代码再次，也不需要编写代码来关闭连接，因为它会通过 using 块自动释放。

Answer 3

一个（非常）快速的测试（VS2010 Web 服务器，.net v4）向我表明，当引发未处理的异常时（至少在 Page_Load 中引发），会调用 Unload 事件，因此原理看起来就像它应该起作用一样。

示例中列出的模式仅在连接打开时dispose'ing 连接。

由于 _conn 受到保护，BasePage 的后代页面可以与 _conn 交互并修改其值。后代类有两种方法可以打破这种模式：

1. 直接调用_conn.Close()。如果连接未打开，则不会在 EndConnection 中将其释放。

2. 通过将 _conn 设置为 null 或为其分配 DBConnection 的新实例来修改它的值。

考虑更改 EndConnection 方法，以便始终处置 _conn。

private void EndConnection(object sender, EventArgs e)
{
    if (_conn == null)
    {
       return;
    }
    if (_conn.Connection.State == ConnectionState.Open)
    {
         _conn.Close();
    }
    _conn.Dispose(); // always dispose even if not actually open. It may have been closed explicitly elsewhere.
}

情况 2 无法被 EndConnection 捕获。考虑将 _conn 设为私有并提供 getter 属性：

private DBConnection _conn;

protected DBConnection Connection {
     get 
     {
         return _conn;
     }
}

这可以防止后代类更改 _conn 的值。

最后，DBConnection 是您自己的类吗？我只是在您引用“_conn.Connection.State”时才问，而不仅仅是_conn.State。如果是这样，只需仔细检查 DBConnection 的 Dispose 方法是否正确处置其 Connection 实例。

Answer 4

编辑：根据 PHeiberg 的回答，这在 .net 4 中绝对是可能的，并且可以假设卸载将始终被调用。

我也检查了.net 2.0代码，那里也是如此。

Answer 5

不，看下面的例子。

如果用户关闭浏览器会发生什么？然后，将不会调用 Unload 函数，并且您将拥有与数据库的打开连接。

这Unload 问题也有与您类似的问题。