使用带有 connection.open 的语句

Question

我正在查看一些代码并与同事讨论。

特别是一段看起来像这样的代码。

    [Test]
    public void TestNormalWay()
    {
        using(var cn = GetConnection())
        {
            cn.Open();
            // do stuff
        }
    }

问题来了：

“为什么不将cn.Open移到GetConnection方法中。”

我说过如果“Open”抛出异常 dispose 将不会被调用。他的回答是

“那又怎样。连接没有打开，为什么需要获取 关闭（或处置）？”

对我来说，这只是不想知道我是否需要处置/关闭，所以我会在代码中重复 cn.Open，而不是将其移动到共享函数中。

但是这很有趣......所以我在SQL Server Connection Pooling (ADO.NET)做了一些阅读

对我来说，不清楚是否存在调用 cn.Open 并抛出异常以及需要调用 dispose 的情况。

所以在我下面的例子中，“TestNormalWay”和“WhyNotDoItThisWay”之间真的有什么区别

    protected static DbConnection GetConnection()
    {
        DbConnection cn = new SqlConnection("SomeConnecitonstring... ");
        return cn; 
    }

    protected static DbConnection GetConnectionDangerousVersion()
    {
        DbConnection cn = new SqlConnection("SomeConnecitonstring... ");
        cn.Open();  // this will throw.. .dispose not called
        return cn; 
    }

    [Test]
    public void TestNormalWay()
    {
        using(var cn = GetConnection())
        {
            cn.Open();
            // do stuff
        }
    }

    [Test]
    public void WhyNotDoItThisWay()
    {
        using(var cn = GetConnectionDangerousVersion())
        {
            // do stuff
        }
    }

Answer 1

您编写代码的方式总是希望在连接创建后立即打开，因此没有区别。

但是，您可以多次打开和关闭连接，并且在设计用于执行此操作的代码中存在很大差异。

我可能想编写一些代码，其中我有一个长时间运行的例程，该例程需要一个连接对象并随着时间的推移打开和关闭它。该例程可能不关心连接对象是如何创建的。因此，将创建连接的行为与打开和关闭连接的行为分开是一个优势。

关于资源管理问题，我同意这不是问题。创建 SQL 连接对象本身不会锁定任何资源，打开它的行为会获取池连接。如果 open 返回异常，我认为假设连接未打开是合理的。

Answer 2

我倾向于只返回SqlConnection 的实例而不在您的方法中调用Open()。如果需要，就应该这样做。您的实用程序函数中不需要它。

其中一个原因是，有些对象需要SqlConnection，但不一定需要打开它们。例如，SqlDataAdapter 接受SqlConnection，并在其内部处理它的打开和关闭。当然，您可以在传递连接之前打开它，但是您必须明确关闭它。

退后几步，调用代码应该负责处理与SqlConnection 的具体关系。

Answer 3

您可以在第二个“危险”版本中的 .Open() 调用周围放置一个 try/catch，这样如果它在打开时抛出异常，您仍然可以处理连接。

Answer 4

在 SqlConnection 的内部达到顶峰之后，我几乎确信这真的无关紧要。快速测试似乎证实了这一点：

static void Main( string[] args )
{
   Func<SqlConnection> getConnection =
            () =>
               {
                  var connection =
                     new SqlConnection(
                        "Initial Catalog=myDatabase;Server=(local);Username=bogus;password=blah;Connect Timeout=10;" );

                  connection.Open();
                  return connection;
               };

   while(true)
   {
      try
      {
         using( var connection = getConnection() )
         {
            var cmd = new SqlCommand( "SELECT 1", connection ) {CommandType = CommandType.Text};
            cmd.ExecuteNonQuery();
         }
      }
      catch ( Exception )
      {
         // ignore exception
      }
   }
}

我让这段代码运行了几分钟，并附加了一个分析器。它不仅运行速度非常快，而且还没有泄漏任何内存。这几乎让我相信在 GetConnection 方法中打开连接是可以的。

当然，这里发布的所有其他论点仍然有效；仅当您要立即使用时才应打开连接。