C# IDisposable Using: Best Practice

Question

Od pewnego czasu pakuję obiekty OracleConnection i OracleCommand w instrukcje USING, jednak po uruchomieniu analizatora kodu odkryłem, że OracleParameter również implementuje IDisposable. Czy poniższy kod jest prawidłowy? Czy istnieje lepsza technika czytelności lub struktury? Na pierwszy rzut oka wydaje się być zagraconym za pomocą instrukcji USING:

using (OracleConnection conn = new OracleConnection(connectionstring)) 
{ 
    conn.Open(); 
    using (OracleCommand cmd = new OracleCommand(sql, conn)) 
    { 
     cmd.BindByName = true; 

     using (OracleParameter param1 = new OracleParameter("p1", OracleDbType.Int32, System.Data.ParameterDirection.Input)) 
     { 
      param1.Value = int.Parse(value1); 
      cmd.Parameters.Add(param1); 
     } 

     using (OracleParameter param2 = new OracleParameter("p2", OracleDbType.Varchar2, System.Data.ParameterDirection.Input)) 
     { 
      param2.Value = value2; 
      cmd.Parameters.Add(param2); 
     } 

     using (OracleDataReader dr = cmd.ExecuteReader()) 
     { 
      // loop data here... 
     } 
    } 
} 

Answer 1

Chcesz pozbyć parametrów dopiero na samym końcu ich wykorzystania, w tym podczas kwerendy (i ewentualnie odczyt wyników):

using (OracleConnection conn = new OracleConnection(connectionstring)) 
{ 
    conn.Open(); 
    using (OracleCommand cmd = new OracleCommand(sql, conn)) 
    { 
     cmd.BindByName = true; 

     using (OracleParameter param1 = new OracleParameter("p1", OracleDbType.Int32, System.Data.ParameterDirection.Input)) 
     using (OracleParameter param2 = new OracleParameter("p2", OracleDbType.Varchar2, System.Data.ParameterDirection.Input)) 
     { 
      param1.Value = int.Parse(value1); 
      cmd.Parameters.Add(param1); 
      param2.Value = value2; 
      cmd.Parameters.Add(param2); 

      using (OracleDataReader dr = cmd.ExecuteReader()) 
      { 
       // loop data here... 
      } 
     } 
    } 
} 

Zauważ że można umieścić wiele using oświadczenia w wiersz. To dlatego, że, podobnie jak w rachunku if,

1. using oświadczenie jest uważane za proste stwierdzenie (nawet z blokiem); i
2. Instrukcja A może mieć blok lub instrukcję poniżej.

Answer 2

Nie jestem pewien, czy będzie działać poprawnie. Należy wziąć pod uwagę, że po zakończeniu użytkowania oba parametry powinny zostać usunięte. Fakt, że Twój obiekt cmd.Parameters nadal zawiera odniesienie do nich, nie wyklucza tego, co może się wydarzyć w metodzie Dispose w postaci OracleParameter. Dla wszystkich intensywnych celów deweloper tego obiektu może czyścić pola, które twój OracleCommand spodziewa się wypełnić.

Istnieje pewne niebezpieczeństwo. Jeśli jesteś absolutnie pewny, że chcesz prawidłowo zutylizować swój OracleParameters, sugeruję, abyś pozbył się ich po użyciu przy użyciu OracleDataReader.

Należy pamiętać, że generalnie wywołuje się Dispose po zakończeniu używania obiektu. Mówisz, żeby zwolnił wszystkie zasoby, które wstrzymuje z pulą. Jeśli nie używasz przedmiotu, nie wyrzucaj go przedwcześnie.

Answer 3

Nie jest to prawdą, ponieważ wyrzucasz parametry jeszcze przed ich użyciem.

Zamiast tego powinna więc to tak

OracleParameter param1 = new OracleParameter("p1", OracleDbType.Int32, System.Data.ParameterDirection.Input); 

param1.Value = int.Parse(value1); 
cmd.Parameters.Add(param1); 


OracleParameter param2 = new OracleParameter("p2", OracleDbType.Varchar2, System.Data.ParameterDirection.Input); 

param2.Value = value2; 
cmd.Parameters.Add(param2); 


using (OracleDataReader dr = cmd.ExecuteReader()) 
{ 
    // loop data here... 
} 

param1.dispose(); 
param2.dispose(); 

Answer 4

using (OracleConnection conn = new OracleConnection(connectionstring)) 
using (OracleCommand cmd = new OracleCommand(sql, conn)) 
using (OracleParameter param1 = new OracleParameter("p1", OracleDbType.Int32, 
     System.Data.ParameterDirection.Input)) 
using (OracleParameter param2 = new OracleParameter("p2", OracleDbType.Varchar2, 
     System.Data.ParameterDirection.Input)) 
} 
    conn.Open(); 
    cmd.BindByName = true; 

    param1.Value = int.Parse(value1); 
    cmd.Parameters.Add(param1); 

    param2.Value = value2; 
    cmd.Parameters.Add(param2); 

    using (OracleDataReader dr = cmd.ExecuteReader()) 
    { 
     // loop data here... 
    } 
} 

Answer 5

można spojrzeć na kod źródłowy połączenia i polecenia to wyrzucać parametry? jeśli obiekt połączenia lub polecenia udostępnia wzór, zawija parametry i usuwa je po ich usunięciu. powinieneś się tym martwić. które moim zdaniem powinno/powinno.

Answer 6

Ten kod jest niepoprawny. tworzone przez ciebie parametry są nadal używane poza zasięgiem instrukcji using, ponieważ dodajesz je do zbioru parametrów, ale instrukcja using wywoła Dispose dla parametrów po opuszczeniu kontrolki przez kontrolkę. Co oznacza, że ​​kiedy nadejdzie czas, aby użyć parametrów wewnątrz theyâ polecenia zostaną disoised z już

Answer 7

Według MSDN, trzeba tylko użyć using dla Connection i DataReader obiektów. Nigdy nie widziałem using (lub .Dispose()) używane z obiektami parametrów ADO.NET. Gdyby to było konieczne, a nawet pożądane, myślę, że pojawiłoby się to już jakiś czas w ciągu ostatnich 10 lat.