0

Hola gente lo que quiero hacer es sustituir el .Max() en c#. Quiero crear un método para llamar el último registro de la base de datos de producto y sumarle uno, en caso de que la base este vacía que me traiga uno. A continuación les muestro el código que tengo.

public int ObtenerSiguienteCodigo()
{
    using (var context = new Quiosco2Container())
    {
        var resultado = context.Productos.OfType<Acceso_Datos.Producto>().Any()
            ? context.Productos.OfType<Acceso_Datos.Producto>().Max(x => x.Codigo) + 1
            : 1;

        return resultado;
    }
}

Desde ya muchas gracias y espero ser claro en lo que necesito.

1
  • Los comentarios no deben usarse para discusiones extendidas; esta conversación ha sido trasladada a un chat.
    – Jorgesys
    Commented el 6 feb. 2019 a las 21:55

2 respuestas 2

1

Si no queres usar Enumerable.Max, un pequeño truco puede ser encontrarlo usando Enumerable.OrderByDescending, ya que con este, ordenas según el Id, así luego tomas el primero y lo incrementas.

Para ejemplificarlo,

Asumamos que tengo la siguiente clase

public class Ej
{
    public string o1 { get; set; }
    public int Id { get; set; }
}

y tengo la siguiente lista de objetos

IEnumerable<Ej> objList = new List<Ej>
{
    new Ej { o1 = "uno", Id = 1 },
    new Ej { o1 = "tres", Id = 3 },
    new Ej { o1 = "dos", Id = 2 },
    new Ej { o1 = "cinco", Id = 5 },
    new Ej { o1 = "cuatro", Id = 4 },
};

Para encontrar el siguiente id, lo podría hacer de la siguiente manera:

using System.Linq;
int nextId = objList.OrderByDescending(x => x.Id).FirstOrDefault()?.Id+1 ?? 1;
  1. Primero, uso Enumerable.OrderByDescending para ordenar de mayor a menor, y de esa manera tomar el Id más alto.
  2. Luego utilizo Enumerable.FirstOrDefault para obtener, el objeto con el Id más alto, o null en su defecto.
  3. En caso de ser null, asigno 1 a la variable nextId
5
  • Todo muy lindo.. ojala no tengas 3 millones de datos ;)
    – gbianchi
    Commented el 6 feb. 2019 a las 16:39
  • @gbianchi la manera óptima de hacer esto sería usar Max() o cual es? Commented el 6 feb. 2019 a las 16:40
  • 1
    creo que ninguna de estas maneras me resulta optima.. pero es mi opinion y necesitaria mas contexto.. tirarle el problema a EF en este caso me parece sobrecargar un lugar erroneo.. la db ya deberia saber cual es el maximo sin tanto lio.
    – gbianchi
    Commented el 6 feb. 2019 a las 16:41
  • @JuanSalvadorPortugal no uso lista uso base de datos SQLServer como lo haria si quiero tomar el ultimo registro en base de datos de la tabla Producto?
    – Estanislao
    Commented el 6 feb. 2019 a las 16:43
  • @Estanislao ¿no estas usando entity framework? Commented el 6 feb. 2019 a las 16:56
0

Podrias ordernar descendente y tomar el primer de los codigos, si existe el valor lo tomas, sino al cero le sumas uno por lo tanto sera el primer codigo

public int ObtenerSiguienteCodigo()
{
    using (var context = new Quiosco2Container())
    {
        var codigo = context.Productos
                                .Select(x=>x.Codigo)
                                .OrderByDescending()
                                .FirstOrDefault();
        int ultimo = 0;

        if(codigo != null){
            ultimo = codigo;
        }

        return ultimo++;
    }
}

Igual sigo pensando que el Max() es la mejor opcion

public int ObtenerSiguienteCodigo()
{
    using (var context = new Quiosco2Container())
    {
        var resultado = context.Productos.Select(x => x.Codigo)
                                            .DefaultIfEmpty()
                                            .Max();

        return resultado++;
    }
}

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.