0

quisiera saber que estoy haciendo mal en mi código; deseo obtener los datos de una tabla de mi BD; en la tabla tengo 2 registros .. pero el método sólo está que me devuelve un resultado. Este es el método que tengo en mi Capa_Persistencia:

public DataTable ListarUsuarios()
    {
        DataTable dataTable = new DataTable();
        try
        {
            SqlCommand comando = conexion.obtenerComandoSP("SP_ObtenerUsuarios");
            SqlDataReader resultado = comando.ExecuteReader();
            if (resultado.Read())
            {
                dataTable.Load(resultado);
            }
            resultado.Close();
            return dataTable;
        }
        catch (Exception ex)
        {
            throw ex;
        }
    }
1
  • Prueba a quitar la sentencia if y deja sólo el dataTable.Load(resultado); Commented el 7 jul. 2020 a las 18:26

1 respuesta 1

1

El resultado que obtienes en un SqlDataReader es "forward-only", esto significa que una vez que lees un registro, no lo puedes volver a leer.
En tu método, la instrucción if (resultado.Read()) lee y descarta el primer registro y por esa razón no aparece en el DataTable.
Para validar si la consulta devuelve resultados, debes cambiar la instrucción if (resultado.Read()) por if( resultado.HasRows ). La propiedad HasRows devuelve true cuando el SqlDataReader contiene uno o más registros.

Con este cambio, el método quedaría de la siguiente manera:

public DataTable ListarUsuarios()
{
    DataTable dataTable = new DataTable();
    try
    {
        SqlCommand comando = conexion.obtenerComandoSP("SP_ObtenerUsuarios");
        SqlDataReader resultado = comando.ExecuteReader();
        if (resultado.HasRows)
        {
            dataTable.Load(resultado);
        }
        resultado.Close();
        return dataTable;
    }
    catch (Exception ex)
    {
        throw ex;
    }
}

Otras recomendaciones relacionadas con el método pero no con la pregunta:

  • En el bloque catch: si el objetivo es reenviar la excepcion tal como se recibe, entonces utiliza solamente throw;. La forma como lo utilizaste, reemplaza el 'stack trace' de la excepción original. ver mas

  • Agrega un bloque finally para liberar (Dispose) los recursos utilizados. ver más

    //Ejemplo:
    catch {...}  
    finally
    {
        resultado?.Dispose();
    }
    
  • Es recomendable abrir y cerrar la conexión a la base de datos cada vez que se necesite.

1
  • Gracias! me solucionó el problema! :D
    – nandofer
    Commented el 8 jul. 2020 a las 1:13

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.