Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR - Daniel - Pratica de Lambda #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DanielJBMediote
Copy link

Olha ai Deivid

Copy link
Owner

@DeividFrancis DeividFrancis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabéns por se comprometer a desenvolver as atividades, em uma analise geral o código está bem satisfatório, adicionei umas dicas para evoluir seu código.

OBS: Formulei a atividade VIII incorretamente, queria realmente dizer:

VIII Mostra a categoria onde com os produtos mais caro;
VIII - Mostrar a categoria onde a soma dos produtos seja mais caro;


import java.math.BigDecimal;

public class Produto {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dica: Legal a estrutura que você criou a classe (muitos sente dificuldade), uma dica seria para o atributo de categoria utilizar um enum ia deixar seu código mais profissional ; )

}

private static void atividadeA() {
List<Produto> prodFiltroListA = produtoList.stream().filter(prod -> prod.getCategoria().equals("Tecnologia"))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DICA para evitar erros como NullPointerException seria interessante fazer o filtro assim:
.filter(prod -> "Tecnologia".equals(prod.getCategoria())

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não sabia dessa, alterar a ordem... Significar que ele estaria comparando aquela String ("Tecnologia") que no caso sempre será existente, com a string do atributo de Produto (que pode ser um objeto ou null), certo?


}

public static void listarProdutos(List<Produto> prodList) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DICA: Bastante legal esse método para reutilizar nos demais desafios, uma sugestão seria ter implementado o toString() da classe Produto pois o java olha esse método como padrão assim podia ser simplificado.


}

private static void atividadeC() {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Falta implementa o filtro somente com estoque;

@DeividFrancis DeividFrancis changed the title 20221003-daniel-atividade-java-stream PR - Daniel - Pratica de Lambda Oct 4, 2022
@DanielJBMediote
Copy link
Author

Então, eu fiquei na dúvida nesta atividade VIII, no começo eu ia fazer por soma da maior categoria, mas ai me falaram que era a categoria com o produto de maior preço.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants