Skip to content

Conversation

@victorfdovale
Copy link
Collaborator

@victorfdovale victorfdovale commented Apr 14, 2022

Copy link

@labenu-bot labenu-bot 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 pela entrega!

Requisitos do projeto ✅

Implementações Feito
É possível ver uma lista com todos os pokemons
É possivel fazer uma busca por um pokemom específico
É possivel filtrar pokemons por um ou mais parâmetros.
Contém paginação -
Transformou o arquivo exel em json
Usou algum banco de dados
Enviou os dados json já existentes para banco de dados
Separou responsabilidades usando os princícios de arquitetura de softWare
Criou uma entidade para pokemon
Criou documentação do Postman
Criou um 'README' explicando como executar o projeto -
Usou docker. -
Fez testes unitários -
Feedback do código Feito
Isolou as configurações do servidor das lógicas de negócio
Isolou as entidades da aplicação
Tratou corretamente os erros

Comentários da pessoa avaliadora

Olá, Victor! Gostaria de primeiramente parabenizá-lo por concluir o quarto case de final de curso da Labenu! \o/

Pude perceber que você conseguiu encaminhar bem o seu fluxo de projeto, implementando alguns endpoints funcionais, utilizando classes e orientação à objetos! ;)

De toda forma, gostaria de pontuar algumas observações que podem te ajudar nas próximas atividades:

  1. Uma boa prática no desenvolvimento de endpoints e API´s consiste em tornar a sua utilização eficaz e entendível. No caso do seu código, senti falta de uma explicação mais clara de como o usuário consumiria e/ou reproduziria seus endpoints. Como sugestão, você poderia incluir estas instruções no README.md, descrevendo o seu projeto, o deploy do projeto, suas limitações e também dando essas instruções para que qualquer pessoa que eventualmente vá acessar suas requisições consiga fazê-lo de forma prática. ;)

  2. Ao observar o endpoint de pegar todos os pokemons, senti falta de alguma estrutura de restrição de busca (paginação) :/ Ainda que não fosse uma obrigatoriedade, pensando no fluxo de utilização do frontend, poderia ser interessante tal implementação, pois muitos dados podem gerar demora na requisição. Pense nisso! ;)

  3. Senti que a sua restrição de dados de pokemons limitou muito a utilização dos dados armazenados. Pensando no contexto de pokemons, faz sentido que tenhamos, por exemplo, os status de attack, defense, stamina, entre outros, que definam e configurem dado pokemon. Ainda que não seja um erro, gostaria que você pensasse na aplicação final do seu frontend. Quais informações o client estaria interessado nos pokemons? Pra que finalidade? Pense nisso! ;)

No mais, muito sucesso na sua jornada e parabéns pela entrega! :)

Análise realizada por: Bruno

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.

3 participants