當引發異常時，確保SQL連接關閉的正確方法是什麼？

我使用的模式看起來像這樣經常。我想知道這是否正常，或者是否有我不在這裏申請的最佳做法。當引發異常時，確保SQL連接關閉的正確方法是什麼？

具體我想知道;在引發異常的情況下，我在finally塊中的代碼足以確保連接適當關閉？

public class SomeDataClass : IDisposable 
{ 
    private SqlConnection _conn; 

    //constructors and methods 

    private DoSomethingWithTheSqlConnection() 
    { 
     //some code excluded for brevity 

     try 
     { 
      using (SqlCommand cmd = new SqlCommand(SqlQuery.CountSomething, _SqlConnection)) 
      { 
       _SqlConnection.Open(); 
       countOfSomething = Convert.ToInt32(cmd.ExecuteScalar()); 
      } 
     } 
     finally 
     { 
      //is this the best way? 
      if (_SqlConnection.State == ConnectionState.Closed) 
       _SqlConnection.Close(); 
     } 

     //some code excluded for brevity 
    } 

    public Dispose() 
    { 
     _conn.Dispose(); 
    } 
}

來源

2008-09-26 Eric Schoonover

呃......你爲什麼要檢查的連接關閉之前已經關閉？爲什麼使用類成員來存儲連接，如果你要使用它的唯一方法關閉它？ – Shog9 2008-09-26 18:46:33

spoon16包含短語「//爲了簡潔排除某些代碼」。由此我推斷這不是使用它的唯一方法。 – 2008-09-26 18:52:41

是正確的，這絕對是我能想到的最簡單的例子。我認爲，由於連接池，我不需要嘗試存儲每個方法範圍之外的SqlConnection對象。所以只要使用USING作爲其他建議。 – 2008-09-26 20:19:35

總結你的數據庫處理代碼內部的「使用」

using (SqlConnection conn = new SqlConnection (...)) 
{ 
    // Whatever happens in here, the connection is 
    // disposed of (closed) at the end. 
}

來源

2008-09-26 18:46:37 Sklivvz

由於您使用IDisposables反正。你可以使用'using'關鍵字，它基本上等同於在finally塊中調用dispose，但它看起來更好。

來源

2008-09-26 18:46:46

我猜，通過「_SqlConnection.State == ConnectionState.Closed」你的意思！=。

這肯定會起作用。我認爲習慣上把連接對象本身包含在using語句中，但是如果你想由於某種原因重用相同的連接對象，你所擁有的就很好。

但是，您應該改變的一件事是Dispose（）方法。您不應該在處理中引用連接對象，因爲它可能已經在此處完成。您應該遵循推薦的Dispose模式。

來源

2008-09-26 18:49:52

-4

我可以建議是：


    class SqlOpener : IDisposable 
    { 
     SqlConnection _connection; 

     public SqlOpener(SqlConnection connection) 
     { 
      _connection = connection; 
      _connection.Open(); 

     } 

     void IDisposable.Dispose() 
     { 
      _connection.Close(); 
     } 
    } 

    public class SomeDataClass : IDisposable 
    { 
     private SqlConnection _conn; 

     //constructors and methods 

     private void DoSomethingWithTheSqlConnection() 
     { 
      //some code excluded for brevity 
      using (SqlCommand cmd = new SqlCommand("some sql query", _conn)) 
      using(new SqlOpener(_conn)) 
      { 
       int countOfSomething = Convert.ToInt32(cmd.ExecuteScalar()); 
      } 
      //some code excluded for brevity 
     } 

     public void Dispose() 
     { 
      _conn.Dispose(); 
     } 
    }

希望幫助:)

來源

2008-09-26 18:51:39