Eliminación de comandos SQL y cerrar la conexión

Question

hasta ahora Siempre he utilizado una estructura similar para obtener datos de base de datos y rellenar un 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; 
    } 

Ahora vi algunas advertencias en el análisis de la acumulación:

TestService. cs (37): CA2000: Microsoft.Reliability: en el método 'TestService.GetByID (int)', el objeto 'table' no se elimina a lo largo de todas las rutas de excepción. Call System.IDisposable.Dispose en object 'table' antes de que todas las referencias estén fuera del alcance.

TestService.cs (42): CA2000: Microsoft.Reliability: En el método 'TestService.GetByID (int)', llame a System.IDisposable.Dispose en el objeto 'cmd' antes de que todas las referencias estén fuera del alcance.

¿Debo cambiar mi código en

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

¿Qué hacer con objeto DataTable? ¿Es una buena práctica colocar SqlCommand dentro del uso?

Gracias

Saludos

Answer 1

También debe hacer esto:

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

al cargar la tabla

Answer 2

• El que llama a este método debe llamar al disponer de la DataTable devuelto cuando se hace uso de ella.
• Sí, es una buena práctica colocar SqlCommand dentro utilizando.

Answer 3

Para "arreglar" su problema con DataTable, quizás podría modificar su función.

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


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

No diciendo que esta es la solución óptima, pero resolverá la queja.