C# - Rellene la lista genérica de SqlDataReader

Question

¿Cómo puedo agregar valores que SqlDataReader devuelve a una lista genérica? Tengo un método donde utilizo SqlDataReader para obtener CategoryID de una tabla Category. Me gustaría agregar todos los CategoryID una lista genérica.

Esta dosis no funciona porque devuelve solo un categoryID y ese es el último. Quiero agregar todos los categoryID a la lista y luego devolverlos.

¿Cómo puedo hacer eso?

SqlConnection connection = null; 
SqlDataReader reader = null; 
SqlCommand cmd = null; 

try 
{ 
    connection = new SqlConnection(connectionString); 
    cmd = new SqlCommand("select CategoryID from Categories", connection); 

    connection.Open(); 

    List<int> catID = new List<int>(); 
    dr = cmd.ExecuteReader(); 
    while (dr.Read()) 
    { 
     catID.Add(Convert.ToInt32(dr["CategoryID"].ToString())); 
    } 
} 
finally 
{ 
    if (connection != null) 
     connection.Close(); 
} 
return catID; 

Answer 1

su código actual debería funcionar, asumiendo catID es realmente declarado antes del bloque try, de lo contrario no se compilará.

Answer 2

Trate de este tipo, que es mejor, más segura, utiliza la carga diferida, menos código, trabajo, ...:

public IEnumerable<int> GetIds() 
{ 
    using (var connection = new SqlConnection(connectionString)) 
    using (var cmd = connection.CreateCommand()) 
    { 
     connection.Open(); 
     cmd.CommandText = "select CategoryID from Categories"; 
     using (var reader = cmd.ExecuteReader()) 
     { 
      while (reader.Read()) 
      { 
       yield return reader.GetInt32(reader.GetOrdinal("CategoryID")); 
      } 
     } 
    } 
} 

y luego:

List<int> catIds = GetIds().ToList(); 

Answer 3

AS BrokenGlass explicó esto es la demostración

SqlConnection connection = null; 
     SqlDataReader dr= null; 
     SqlCommand cmd = null; 
List<int> catID = new List<int>(); 
     try 
     { 
      connection = new SqlConnection(connectionString); 
      cmd = new SqlCommand("select CategoryID from Categories", connection); 

      connection.Open(); 



      dr = cmd.ExecuteReader(); 
      while (dr.Read()) 
      { 
       catID.Add(Convert.ToInt32(dr["CategoryID"].ToString())); 
      } 


     } 
     finally 
     { 
      if (connection != null) 
       connection.Close(); 
     } 
     return catID; 

, así como cambia la declaración

SqlDataReader reader = null; 

a

SqlDataReader dr= null; // Because you are using dr in the code not reader 

Answer 4

Esto debería funcionar, pero yo sugeriría que usted utilice using con su connections

SqlConnection connection = null; 
    SqlDataReader reader = null; 
    SqlCommand cmd = null; 
    List<int> catID = new List<int>(); 
    try 
    { 
     connection = new SqlConnection(connectionString); 
     cmd = new SqlCommand("select CategoryID from Categories", connection); 

     connection.Open(); 



     dr = cmd.ExecuteReader(); 
     while (dr.Read()) 
     { 
      catID.Add(Convert.ToInt32(dr["CategoryID"].ToString())); 
     } 


    } 
    finally 
    { 
     if (connection != null) 
      connection.Close(); 
    } 
    return catID; 

Answer 5

 List<int> s = new List<int>(); 
     conn.Open(); 
     SqlCommand command2 = conn.CreateCommand(); 
     command2.CommandText = ("select turn from Vehicle where Pagged='YES'"); 
     command2.CommandType = CommandType.Text; 
     SqlDataReader reader4 = command2.ExecuteReader(); 
     while (reader4.Read()) 
     { 
      s.Add(Convert.ToInt32((reader4["turn"]).ToString())); 
     } 
     conn.Close();