Rodada cases semana4 - #57
Conversation
labenu-bot
left a comment
There was a problem hiding this comment.
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:
-
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. ;) -
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! ;)
-
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
Deploy no heroku : https://case-redfox-labenu.herokuapp.com/
Documentação da API : https://documenter.getpostman.com/view/18386367/Uyr4JKSA