Feedback da avaliação

[paaatcha]

Avaliação do projeto

• Integrantes:
  • Fernando Azevedo Peres
  • Luis Eduardo Câmara

Obs: Meu nome não precisa contar como colaborador uma vez que eu não fiz parte do desenvolvimento.

Neste documento você encontra a avaliação geral do seu projeto. Primeiro avalio os requisitos que foram cumpridos. Na sequência, faço comentários sobre
o tema, usabilidade, código, etc.

Qualquer dúvida ou problema, sinta-se livre para entrar em contato.

Requisitos

Primeiramente, segue a avaliação de cada um dos requisitos técnicos obrigatórios que constam na descrição do trabalho. Os que foram cumpridos estão demarcados com um checkbox.

• Conter pelo menos duas Activities
• Conter pelo menos 4 telas diferentes
• Consumir dados de uma API e/ou usar Banco de Dados
  • Se API é usada para salvar dados, você não precisa ter o banco local
  • Se você não usar uma API, você é obrigado a usar o banco localmente
• Utilizar Fragments
• Utilizar Navigation
• Utilizar pelo menos uma Recycler View
• Utilizar um padrão de projeto de livre escolha (Recomendável o MVVM)
• Utilizar boas práticas de codificação
• Usar arquivos strings.xml e colors.xml
  • Usou parcialmente a internacionalização
  • Tem texto injetado direto no código
• Seguir padrões de nomenclaturas dentro do projeto
  • Tanto no Kotlin quanto no XML
• Construir a aplicação em pelo menos duas línguas através da ferramenta de internacionalização

Outros requisitos eram desejáveis (mas não são obrigatórios):

• Utilizar Arquitetura MVVM
• Utilizar o padrão de projeto Singleton
• Criar cobertura de testes para o projeto (haverá pontuação extra para este item)
• Utilizar Firebase
  • Principalmente para gerenciamento de contas de usuários (se o seu app permitir)

Requisitos não técnicos, mas obrigatórios:

• Documentação de código
  • parcial
• Documentação de Funcionalidades
• Documentação em vídeo

Comentários gerais

De maneira geral, e sendo bem sincero, o projeto foi feito a toque de caixa apenas para cumprir (quase todos) os requisitos. Comparado aos outros apps de Pokedex que a turma fez, esse é o que tem menos funcionalidade.

A parte visual poderia ter tido melhor cuidado. Mas ok, não era um critério técnico (não somos designers). Porém, algumas decisões técnica, agora sim no campo da engenharia, foram ruins (e acredito que vocês tem consciência disso). Na sequência vocês encontram comentários técnicos e de usabilidade.

Por fim, analisando o Github de vocês, além do uso parecer ser bem rudimentar, eu só vi commit do Luis. Como foi a divisão de tarefas?

Comentários técnicos

• Poderia ter usado a SnackBar ao invés do Toast
• Em termos de uso de API vocês fizeram algo muito ruim. Ele pega todos os Pokemons de uma única vez ao invés de pegar apenas um grupo (como seus colegas fizeram)
  • A consequência disso é que a tela inicial demora muito tempo para carregar
  • Com certeza vocês tem consciência de que isso é horrível. Não façam isso no mercado de trabalho (se fizerem e falarem que foi meu aluno eu nego 😂)
• O app reconhece que eu já loguei e não preciso digitar a senha toda hora, o que é bom.
• A internacionalização é parcial. Mesmo em Português, existem vários termos em inglês (defense, speed, Base Stats, etc)
  • Além disso tem texto em Português injetado direto no código
• A documentação do código está parcial. Tem função/método/classe sem documentar ou com documentação fraca
• Não usou Fragment e nem Navigation 😓
• Existe string ingetada no código que deveria ser usada como constante/estático
  • Exemplo: todas as chaves do bundle da MainActivity
  • Detalhe, vocês tem uma classe de Constantes, era só incluir lá

Comentários sobre usabilidade

• Falta um botão de voltar na tela de cadastro e na tela que mostra os pokemons

  • Isso poderia ser facilmente resolvido com uma barra superior (o botão vem de graça)

• Poderiam ter colocado um launch icon personalizado para o aplicativo

• Do ponto de vista de usuário, o app poderia entregar mais. Vocês poderiam ter implementado bem mais coisas (exemplos: busca, batalha de pokemon, divisão por regiao, etc)

Nota final

Pontuação de cada parte do trabalho:

• Nota da implementação: 6.0
• Nota da documentação: 8.0
• Nota dos testes: 0