using connection.open with statement

Question

私はいくつかのコードを見て、同僚と議論していました。

特にこのようなコードセクション。

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

質問が来た：

"なぜGetConnectionメソッドにcn.Openを移動しません。"

私は、 "オープン"が例外をスローするとディスパッチは呼び出されないと言った。彼の答えは

た「だから何な接続がそのように開かれていませんでした。なぜそれが が閉じて（または配置された）を取得する必要があるでしょうか？」

私にとっては、私がディスパージ/クローズする必要があるかどうかを知りたいのではなく、共有機能に移動するのではなく、コード内でcn.Openを繰り返すことです。

しかしそれは面白いですが...私はcn.Openを呼び出すと、それがスローするシナリオと例外処分は希望がある場合は私にSQL Server Connection Pooling (ADO.NET)

でいくつかの読書は、それが明確ではありませんでした呼び出される必要があります。以下の私の例ではそう

あなたは常には、すぐにそれはだとして、接続を開きたい、本当に「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

2回目の "危険な"バージョンで.Open（）呼び出しをtry/catchすることで、開いている間に例外がスローされても接続が破棄されるようにすることができます。

Answer 3

あなたのメソッドでOpen()を呼び出さずに、SqlConnectionのインスタンスを返すことになります。それが必要な場合は、それを行う必要があります。あなたのユーティリティ機能では必要ありません。

理由の1つは、SqlConnectionが必要なオブジェクトがいくつかありますが、必ずしもそれらを開く必要はありません。たとえば、SqlDataAdapterはSqlConnectionで、それ自体の開閉を処理します。確かに、あなたはそれを渡す前に接続を開くことができますが、あなたはそれを閉じることで明示的にする必要があります。

少し前に戻って、SqlConnectionと正確に何を処理するかは、呼び出し元コードの責任で行う必要があります。

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メソッドで接続を開いても大丈夫です。

もちろん、ここに掲載されているその他の議論はすべて有効です。あなたがすぐにそれを使用する場合は、接続を開く必要があります。