0

veja bem, eu estou desenvolvendo o dashboard de um sistema web, em um dos filtros eu devo listar os 5 principais clientes e ordena-los por valor mais alto das somas de totais de notas fiscais emitidas. Eu já estou conseguindo listar tranquilamente, o problema é que o filtro está listando tudo (quando que deveria apenas listar 5) e está de forma desordenada. Segue abaixo o meu código:

public ActionResult Index()

        {
            var dashboard = new DashboardViewModel();


                var lista = new List<NotaFiscal>();


            var participantes = db.Participantes
                          .ToList();
            var notas = db.NotasFiscais
                          .ToList();

            foreach (var part in participantes)
            {
                var x = new NotaFiscal();
                var res = notas.Where(y => y.ClienteID == part.ParticipanteId).Sum(o => o.ValorTotalNota);
                x.ClienteID = part.ParticipanteId;
                x.ValorTotalNota = res;
                x.NomeCliente = part.NomeParticipante;
                lista.Add(x);
            }


            dashboard.NotasFiscais = lista;
            dashboard.NotasFiscais.Take(5).OrderByDescending(x => x.ValorTotalNota);
            return View(dashboard);
        }

Alguém saberia me apontar a solução para que esse filtro liste apenas os 5 registros com os valores maiores de notas fiscais?

Abaixo o Jean e o Filipe me deram alternativas, após fazer o sugerido em ambos os casos ocorrem o seguinte erro, veja a imagem: inserir a descrição da imagem aqui

2 Respostas 2

2

O filtro foi feito mas não foi atribuído novamente onde deveria

Você pode fazer assim:

dashboard.NotasFiscais = lista.Take(5).OrderByDescending(x => x.ValorTotalNota);
return View(dashboard);

E dá pra mudar um pouco o código por conta das consultas, já que quando um toList é feito, a consulta é executada e isso pode afetar performance do que você estiver implementando.

Aqui um exemplo de como pode ficar:

var dashboard = new DashboardViewModel();


var lista = dbParticipantes
    .Select(e => new NotaFiscal {  
        ClienteID = e.ParticipanteId,
        ValorTotalNota = e.NotasFiscais.Sum(n => n.ValorTotalNota)
    })
    .OrderByDescending(e => e.ValorTotalNota)
    .Take(5);

dashboard.NotasFiscais = lista;
return View(dashboard);
11
  • Então amigo, muito obrigado mais da forma que vc pontuou eu me deparo com o seguinte problemas: Não é possível converter implicitamente tipo "System.Linq.IOrderedEnumerable<Ebase.EmissorNFeWeb.Models.NotaFiscal>" em "System.Collections.Generic.ICollection<Ebase.EmissorNFeWeb.Models.NotaFiscal>". Existe uma conversão explícita (há uma conversão ausente?) Dai como eu resolveria isso agora????
    – WPfan
    24/05/2018 às 19:39
  • E a segunda forma que vc cita não é possível, pois eu tenho que passar pelo foreach de qualquer maneira!!
    – WPfan
    24/05/2018 às 19:41
  • Veja o erro, editei a pergunta com o anexo
    – WPfan
    25/05/2018 às 12:20
  • 1
    @WPfan tenta dar colocar assim dashboard.NotasFiscais = lista.ToList(); 25/05/2018 às 17:18
  • Filipe, até que daria certo sim, mais a função que você fez ai contém um erro, nessa linha não consigo recuperar o NotasFiscais: ValorTotalNota = e.NotasFiscais.Sum(n => n.ValorTotalNota)
    – WPfan
    25/05/2018 às 17:24
2

Explicação

O problema do seu código que não ordena conforme o esperado está no trecho abaixo:

 dashboard.NotasFiscais.Take(5).OrderByDescending(x => x.ValorTotalNota);

Podemos ver que antes da ordenação, você seleciona cinco registros através do método Take(5). Para que funcione corretamente, altere a order para a exemplificada abaixo:

dashboard.NotasFiscais.OrderByDescending(x => x.ValorTotalNota).Take(5);

Sugestão

Altere este bloco de código:

   foreach (var part in participantes)
            {
                var x = new NotaFiscal();
                var res = notas.Where(y => y.ClienteID == part.ParticipanteId).Sum(o => o.ValorTotalNota);
                x.ClienteID = part.ParticipanteId;
                x.ValorTotalNota = res;
                x.NomeCliente = part.NomeParticipante;
                lista.Add(x);
            }

Para este:

var lista = db.NotasFiscais
                 .Where(x => db.Participantes.Any(y => y.ParticipanteID == x.ClienteID))
                 .GroupBy(z => z.ClienteID)
                 .Select(x => new NotaFiscal(){
                        ClienteID = x.First().ClienteID,
                        ValorTotalNota = x.Sum(n => n.ValorTotalNota)
                  })
                 .OrderByDescending(x => x.ValorTotalNota)
                 .Take(5)
                 .ToList();

dashboard.NotasFiscais = lista;
return View(dashboard);

Veja funcionando no Ideone

4
  • Jean, no meu caso eu segui o seu exemplo e continuo deparando com o mesmo problema relatado anteriormente, quando é feito 'dashboard.NotasFiscais = lista;' da Não é possível converter implicitamente tipo "System.Linq.IOrderedEnumerable<Ebase.EmissorNFeWeb.Models.NotaFiscal>" em "System.Collections.Generic.ICollection<Ebase.EmissorNFeWeb.Models.NotaFiscal>". Existe uma conversão explícita (há uma conversão ausente?)
    – WPfan
    25/05/2018 às 10:47
  • E vc cita que o problema de não ordenar corretamente é devido a isso aqui: dashboard.NotasFiscais.Take(5).OrderByDescending(x => x.ValorTotalNota); mais não, porque anteriormente eu tinha feito do jeito que você colocou abaixo e tava desordenado da mesma forma
    – WPfan
    25/05/2018 às 10:49
  • editei a pergunta com o anexo do erro, veja lá, do jeito que vc sugeriu
    – WPfan
    25/05/2018 às 12:21
  • Fiz uma alteração, adicione um ToList() ao final do novo trecho de código. 25/05/2018 às 13:33

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 .