Vứt bỏ lệnh SQL và đóng kết nối

Question

đến bây giờ tôi luôn luôn sử dụng một cấu trúc tương tự để lấy dữ liệu từ DB và điền vào một DataTable

public static DataTable GetByID(int testID) 
     { 
     DataTable table = new DataTable(); 
     string query = @"SELECT * FROM tbl_Test AS T WHERE T.testID = @testID"; 

     using (SqlConnection cn = new SqlConnection(Configuration.DefaultConnectionString)) 
     { 
      SqlCommand cmd = new SqlCommand(query, cn); 
      cmd.Parameters.Add("@testID", SqlDbType.Int).Value = testID; 

      cn.Open(); 
      table.Load(cmd.ExecuteReader()); 
     } 

     return table; 
    } 

Bây giờ tôi thấy một số cảnh báo trong phân tích xây dựng:

TestService. cs (37): CA2000: Microsoft.Reliability: Trong phương thức 'TestService.GetByID (int)', đối tượng 'table' không được xử lý dọc theo tất cả các đường dẫn ngoại lệ. Gọi System.IDisposable.Dispose trên đối tượng 'table' trước khi tất cả các tham chiếu đến nó nằm ngoài phạm vi.

TestService.cs (42): CA2000: Microsoft.Reliability: Trong phương thức 'TestService.GetByID (int)', hãy gọi System.IDisposable.Dispose trên đối tượng 'cmd' trước khi tất cả các tham chiếu đến nó nằm ngoài phạm vi.

Tôi có nên thay đổi mã của tôi trong

public static DataTable GetByID(int testID) 
    { 
     DataTable table = new DataTable(); 
     string query = @"SELECT * FROM tbl_Test AS T WHERE T.testID = @testID"; 

     using (SqlConnection cn = new SqlConnection(Configuration.DefaultConnectionString)) 
     { 
      using (SqlCommand cmd = new SqlCommand(query, cn)) 
      { 
       cmd.Parameters.Add("@testID", SqlDbType.Int).Value = testID; 

       cn.Open(); 
       table.Load(cmd.ExecuteReader()); 
      } 
     } 

     return table; 
    } 

Phải làm gì với đối tượng DataTable? Có thực hành tốt để đặt SqlCommand bên trong việc sử dụng không?

Cảm ơn

Cheers

Answer 1

Bạn cũng nên làm điều này:

using (SqlDataReader reader = 
      cmd.ExecuteReader 
       (CommandBehavior.CloseConnection)) 
     { 
      table.Load(reader); 
     } 

khi tải bảng

Answer 2

• Người gọi của phương pháp này nên gọi dispose của DataTable trả về khi nó được thực hiện sử dụng nó.
• Có, đó là một thực hành tốt để đặt SqlCommand bên trong sử dụng.

Answer 3

Để "khắc phục" sự cố của bạn với DataTable, có lẽ bạn có thể sửa đổi chức năng của mình.

public static void GetByID(DataTable table, int testID) 
{ 
    // bla bla bla 
} 


// calling the function 
using(DataTable table = new DataTable()) 
{ 
    TestService.GetByID(table, 5); 
} 

Không nói đây là giải pháp tối ưu nhưng sẽ giải quyết khiếu nại.