这是一种处理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为了这样做。

喜欢的东西：

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

只能在实现IDisposable确保即使有异常或块中的回报Dispose()方法将被调用对象使用的using。它的实现类似于：

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(); 
}