これはSQL接続を処分する良い方法ですか？

Question

私はこれを書いた。これは、開いているSQL接続を処分する良い方法ですか？私の先輩は、これがベストプラクティスだと言いましたが、依然として私はここで質問して自分のことを確かにしたいと思っています。

public bool SaveBulkpayment(List<ClsCombinePayment> list) 
{ 
    bool Result = false; 
    ClsDatabaseManager Manager = ClsDatabaseManager.InitializeDbManager(Constants.Databases.POSSystem); 
    Manager.Open(); 
    Manager.CreateParameters(10); 
    try 
    { 
     foreach (var item in list) 
     { 
      Manager.AddParameters(0, "@Batch", item.Batch); 
      Manager.AddParameters(1, "@TransactionDateTime", TransactionDateTime); 
      Manager.AddParameters(2, "@BatchNo", BatchNo); 
      Manager.AddParameters(3, "@Salary", item.Salary); 
      Manager.AddParameters(4, "@EDRRecord_ID", item.EDRRecord_ID); 
      Manager.AddParameters(5, "@User_ID", item.User_ID); 
      Manager.AddParameters(6, "@Branch_ID", item.Branch_ID); 
      Manager.AddParameters(7, "@PafFile_ID", item.PAFFile_ID); 

      Manager.AddParameters(8, "@PinCode", item.PinCode); 
      Manager.AddParameters(9, "@ifDiff", item.ifDiff); 
      Result = Manager.ExecuteNonQuery("usp_SaveBulkPayment").ToBool(); 
     } 
     Manager.Dispose(); 
    } 
    catch (Exception ex) 
    { 
     throw ex; 
    } 
    return Result; 
} 

Answer 1

ない安全に配置されます。これを行うにはusingを使用してください。以下のような

何か：

のみ IDisposableを実装するオブジェクトに使用することができます

using(var manager = ClsDatabaseManager.InitializeDbManager(Constants.Databases.POSSystem); Manager.Open()) 
{ 
    //Your code 
} 

usingは例外またはブロックで戻りがあってもDispose()メソッドが呼び出されることを確認してください。

public void Using<T>(T input, Action<T> predicate) where T : IDisposable 
{ 
    try 
    { 
     predicate(input); 
    } 
    finally 
    { 
     input.Dispose(); 
    } 
} 

Answer 2

オープンSQL接続は、できるだけ早期に処理する必要があります。

あなたのコードでは、SQL接続をfinallyブロックに配置することになります。このようにしても、例外の場合には、接続は、それが配置されることはありません例外の場合には...本当に

try 
{ 
    //Code   
} 
catch (Exception ex) 
{ 
    //Handle exception 
} 
finally{ 
    //Dispose the Connection 
    Manager.Dispose(); 
}