0

Nececito optimizar este codigo a una unica consulta LINQ

Mi codigo:

  public IList<SucursalModel> RemoverSucursalesMalas(IList<SucursalModel> sucursales)
    {
        var sucursalesBuenas = new List<SucursalModel>();
        var cabecera = new List<SucursalCabeceraModel>();
        var detalle = new List<SucursalDetalleModel>();
        foreach(var item in sucursales)
        {
            cabecera = item.Cabecera.Where(c => c.Controla != "M").ToList();
            if (cabecera.Count > 0)
            {
                detalle = item.Detalle.Where(d => d.Controla != "M").ToList();
                if (detalle.Count > 0)
                    sucursalesBuenas.Add( item);
            }
        }

        return sucursalesBuenas;
    }

introducir la descripción de la imagen aquí

introducir la descripción de la imagen aquí

2 respuestas 2

1

Estás haciendo un Select (o SelectMany, en este ejemplo es irrelevante).

Select sirve para mapear datos de una colección a otra. Literalmente estás diciendo que se mapeen todos los elementos de la colección resultSucursales a una colección con todas las Cabecera (filtrados) de cada elemento.

Deberías devolver una copia de todo el objeto, pero con las Cabecera filtradas.

var res = sucursales.Select<Sucursal, Sucursal>(sucursal =>
{
    return new Sucursal()
    {
        Cabeceras = sucursal.Cabeceras.Where(c => c.Controla != "M").ToList(),
        Detalles = sucursal.Detalles
    };
});

Te dejo aquí el ejemplo.

3
  • ME DICE : IList<SucursalModel>' no contiene una definición para 'Select' y la mejor sobrecarga del método de extensión 'Queryable.Select<SucursalCabecera, SucursalDetalle>(IQueryable<SucursalCabecera>, Expression<Func<SucursalCabecera, SucursalDetalle>>)' requiere un receptor del tipo 'IQueryable<SucursalCabecera>' Commented el 17 may. 2020 a las 0:45
  • ¿Has importado el namespace using System.Linq;?
    – VRoxa
    Commented el 17 may. 2020 a las 0:49
  • VRoxa ...Gracias ...Voy a editar mi pregunta para una mejor comprension..por que con tu ayuda mejoro 80% lo que ya tenia Commented el 17 may. 2020 a las 1:04
0

De esta forma resolvi :

var sucursalesBuenas = resultSucursales.Where(x => x.Cabecera.Count(c => c.Controla != "M") > 0 
&& x.Detalle.Count(d => d.Controla != "M") > 0).ToList();

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.