Por que não usar um for
simples?
private double getValorVenda() {
double valor = 0.0;
for (Produto produto : produtos) {
valor += produto.getValor(Venda.class);
}
return valor;
}
O método forEach
serve para "fazer algo" com cada elemento da lista, tanto que ele recebe como parâmetro um Consumer
(uma operação que faz algo com o elemento e não retorna nada).
Se você tentar fazer o mesmo algoritmo acima com forEach
, o código não compila:
produtos.forEach(p -> {
valor += p.getValor(Venda.class);
});
Esse código dá um erro de compilação:
local variables referenced from a lambda expression must be final or effectively final
Isso acontece porque variáveis declaradas fora de um lambda só podem ser usadas dentro dele se forem final
. Mas se eu declará-la como final
:
final double valor = 0.0;
Aí eu não posso modificá-la, e o código dá outro erro:
cannot assign a value to final variable valor
Enfim, se você quer sumarizar resultados, um loop tradicional já resolve.
Mas se quiser, também pode usar streams, uma alternativa é fazer assim:
double valor = produtos
// cria o stream
.stream()
// obtém os valores das vendas de todos os produtos
.mapToDouble(produto -> produto.getValor(Venda.class))
// soma tudo
.sum();
Com isso, valor
terá a soma dos valores, ou zero caso a lista de produtos seja vazia.
Só lembrando que streams são legais, mas têm o seu custo e geralmente serão mais lentas que um loop tradicional, e temos que balancear a clareza do código e outros benefícios que as streams trazem (vai da opinião de cada um) versus a velocidade. Claro que para casos simples como esse talvez nem faça tanta diferença, mas é algo a se considerar sempre.