Skip to content

Instantly share code, notes, and snippets.

@rafarocha
Last active September 16, 2024 03:02
Show Gist options
  • Save rafarocha/0e38518a404ab73315f40bd5f126a690 to your computer and use it in GitHub Desktop.
Save rafarocha/0e38518a404ab73315f40bd5f126a690 to your computer and use it in GitHub Desktop.

Avaliação dos Requisitos

  • Design de Classes de Domínio → Ok
  • Separação em Camadas → Apenas serviço mas sem interfaces
  • Transição de Estados → Sim mas implícito para carteira de investimentos
  • Princípios SOLID → Infelizmente não houve

Avaliação do Projeto

  • Geral
    • Método main ficou muito extenso e generalizado
    • Implemente o padrão command ou strategy para evitar menus por switch case
    • Mostrar menu e imprimir informações ao usuário é papel da view + controller
    • Boa consideração e usabilidade de opções ao usuário
    • Não houve implementação do repositorio ou cadastro de usuários
    • Sem implementação por interfaces ou princípios SOLID
    • Ótimo design por carteira de investimento ao usuário e lógica para estimá-lo
    • Bom projeto e lógica estruturada em resolver o problema para investimentos
  • Servico
    • Documente interfaces e não implementação. Em ServicoInvestimento sem interface
    • Boa proposta em simularValorFuturo() com base na média da variação mensal
  • Dominio
    • Sem implementação do equals ou comparable
    • Maioria das classes de domínio ficaram sem comportamento. Apenas get/set.
    • Leia mais sobre modelo anêmico e como contornar em suas entidades de negócio
    • Método verificarSituacao() ficou bem extenso. Quebre em métodos menores

Avaliação Desafio 01

  • Apesar da pouca experiência com Java você teve uma ótima evolução
  • Procure praticar mais, de forma consistente, um pouco todo dia
  • E após implementar, avalie no que fez, o que poderia ter ficado melhor. Autocrítica!
  • Atenção na escrita das mensagens e frases em grafia ou falha no português
  • Implementou apenas o emprestar no service mas não compila. Atenção!
  • Nunca entregue algo que diz estar pronto sem ter certeza que funcionou.
  • Se por acaso não funciona, não tem problema. Você avisa e evita surpresas do outro lado.
  • A implementação do service não poderia ter a classe scanner. Isso fica no controller ou main
  • Faltou caprichar no readme ou incluir mais informações sobre as suas mudanças
  • Parabéns por participar e entregar mesmo que com dificuldades evidentes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment