4

Tenho esse método com um foreach dentro

[Route("")]
        [HttpGet]
        [ResponseType(typeof(List<MarkupListResponse>))]
        public IHttpActionResult Get(int resellerId)
        {
            var catalogs = _catalogService.GetAllByResellerId(resellerId).ToList();
            var model = new List<MarkupListResponse>();
            foreach (var catalog in catalogs)
            {
                model.Add(new MarkupListResponse()
                {
                    CreatedOn = catalog.CatalogDate,
                    CatalogId = catalog.Id,
                    ItemsQuantity = catalog.Items.Count
                });
            }

            return Ok(model);
        }

Nesse foreach possuo mais de 400 itens em catalogs. Como eu faço para pssar para uma lambda, pois acho que isso possa melhorar a performance, acho.

Estou usando Entity Framework para essas consultas e MVC.

10
  • 3
    Não, só piorará a performance. Já respondi sobre, pra vc inclusive: pt.stackoverflow.com/a/53857/101
    – Maniero
    Commented 7/03/2019 às 13:15
  • @Maniero, dentro do foreach, eu dou um new, ou seja, instancio a cada iteração o objeto MarkupListResponse e populo as propriedades da dele. Isso pode interferir na performance? Pois nesse loop, eu tenho uma péssima performance, entre outras nesse projeto.
    – pnet
    Commented 7/03/2019 às 13:31
  • Sim, mas nem sempre pode escapar disto. Em alguns sim, mas aí cabe uma análise profunda. Talvez tirar esse ToList() possa ajudar, mas não acho que muito. Se tá ruim, o problema deve ser outro porque 400 é pouca coisa.
    – Maniero
    Commented 7/03/2019 às 13:33
  • 1
    Comece otimizando a sua query no banco, retorne apenas as informações que vai utilizar, já refletindo uma List<MarkupListResponse> e verifique os índices... É muito mais provável que o seu gargalo esteja nesse ponto do que no foreach... Commented 7/03/2019 às 13:55
  • 2
    Então é diferente, nem deveria ter cogitado usar de outra forma, o EF vai converter isso em query em vez de trazer tudo e processar na memória. Se você não coloca essa informação totalmente relevante e necessárias fica complicado responder adequadamente.
    – Maniero
    Commented 7/03/2019 às 15:54

1 Resposta 1

3

fiz isso e ganhei muito. Agora está com uns 3s para executar, o que levava mais de 2min.

var catalogs = _catalogService.GetAllByResellerId(resellerId).Select(c => new MarkupListResponse
            {
                CreatedOn = c.CatalogDate,
                CatalogId = c.Id,
                ItemsQuantity = c.Items.Count()
            }).ToList();

            return Ok(catalogs);
3
  • 3
    Entenda, que nesse caso, muito provavelmente não é expressão lambda que melhorou a sua performance... mas sim a query gerada para consultar o banco de dados. Commented 7/03/2019 às 14:52
  • @LeandroAngelo, faz sentido o que você disse, mas essas coisas são muito conceituais, por exemplo, para iterar e enviar uma query ao banco, dizem que Linq/Lambda é mais performático, pelo que tenho lido. Iterar uma lista, creio que o foreach/for seja mais performático, pelo que acabei de ler no post de um amigo meu. Como disseram, depende de como e no que usar.
    – pnet
    Commented 7/03/2019 às 15:52
  • 1
    Faça o teste, compare queries anteriores e a atual Commented 7/03/2019 às 15:55

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .