1

Irei explicar o que quero fazer.
Tenho esta List:

public static List<Cliente> Clientes = new List<Cliente>();

public class Cliente
{
    public string Nome { get; set; }
    public List<ProdutoComprado> Produtos { get; set; }
}

public class ProdutoComprado
{
    public string NomeProduto { get; set; }
    public string ValorProduto { get; set; }
}

Como podem ver, cada cliente tem seus produtos. O que estou precisando é somar os valores da constante ValorProduto de todos os produtos de determinado cliente. Creio que para fazer isso terei que usar Sum para somar as variáveis. Estava tentando algo como:

Cliente cliente = Clientes[listaClientes.SelectedIndex];

int total = 0;
foreach (var s in cliente.Produtos)
{
    total = s.ValorProduto.Sum(x => Convert.ToInt32(x));
}

string soma = string.Format("\n\n\nValor total: {0}", total);

Isso não está funcionando. Estou tendo como saída o valor 317.

1 Resposta 1

1

Tente alterar o código para o seguinte:

Cliente cliente = Clientes[listaClientes.SelectedIndex];

int total = cliente.Produtos.Sum(x => Convert.ToInt32(x.ValorProduto));

string soma = string.Format("\n\n\nValor total: {0}", total);

O Sum do LINQ atua percorrendo e somando os elementos da lista, deste modo, o foreach externo é desnecessário. Outro ponto que está falho no seu código é que você está sempre sobrescrevendo o total e não o incrementando (trocando o = por +=).

7
  • haha' eu sabia que alguém perguntaria isso. Eu particularmente gosto de usar string para armazenar valores, independentemente do tipo. Quando preciso realizar alguma operação com determinado valor, me utilizo da conversão do mesmo. Nesse meu caso, converto a string para int com o Convert.ToInt32.
    – M. Victor
    Commented 7/11/2014 às 16:40
  • @user17636 Tudo bem quanto a sua preferência pessoal, mas quando se trabalha em time, sugiro que você repense a respeito disto. Essa atitude é bem insegura e não consigo imaginar um bom motivo para defendê-la. Imagine quantos problemas de conversão você pode ter... as linguagens possuem vários tipos por uma razão. Mas quanto à pergunta original, funcionou? Commented 7/11/2014 às 16:50
  • Não funcionou, não estou recebendo o valor esperado.
    – M. Victor
    Commented 7/11/2014 às 16:55
  • @user17636 Alterei a resposta. Commented 7/11/2014 às 17:02
  • Funcionou perfeitamente. Muito obrigado pela ajuda, grande abraço! :)
    – M. Victor
    Commented 7/11/2014 às 17:06

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 .