Relatório antes da refatoração
❯ cloc main
4 text files.
4 unique files.
0 files ignored.
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Java 4 32 6 180
-------------------------------------------------------------------------------
SUM: 4 32 6 180
-------------------------------------------------------------------------------
Apos a refatorção
❯ cloc main
5 text files.
5 unique files.
0 files ignored.
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Java 5 33 4 177
-------------------------------------------------------------------------------
SUM: 5 33 4 177
-------------------------------------------------------------------------------
Apesar de não ter impactado consideravelmente na quantidade de linhas, houve até um aquivo a mais, o projeto ficou mais legível pois houve uma troca de int para ENUM na textura do solo, commit ENUMS e o uso de arrays para substituir uma estrutura condicional de escolha para fontes de potássio e fosforo, commit Arrays.
Análise do projeto realizado por Guilherme Eiti Akita Miua.
Problemas encontrados:
- Nomenclatura incorreta para o atributo "aluminium" na classe Ground.java
- Testes para os métodos de teores do solo se encontram na classe AppTest.java, ficando confuso encontrar onde está os testes de teores do solo Commit: https://github.com/guilhermemiua/arquitetura-software-1/commit/ef6ad94e4b4c0f8cbdfd928352cd9776e64ecc85
- Fonte de fósforo está com o valor inteiro, dificultando saber qual valor deve ser aplicado
- Não havia testes para o método de nutrientes adicionais e suas quantidades
- Ao criar teste para o método de nutrientes adicionais e suas quantidades, houve um erro de tamanho do Array de Nutrientes e também estava errado o cálculo da quantidade Commit: https://github.com/guilhermemiua/arquitetura-software-1/commit/5ca6773ffc8143278e20ace81fadc6d9d20bb6ef
- Fonte de potássio está com o valor inteiro, dificultando saber qual valor deve ser aplicado
- Não havia testes para o método de nutrientes adicionais e suas quantidades
- Não possuía os cálculos: Participação atual do potássio na CTC do solo, participação ideal do potássio na CTC e participação do potássio na CTC após correção Após a análise, foi criado os métodos que faltavam seguindo a arquitetura do projeto e seus devidos testes Commit: https://github.com/guilhermemiua/arquitetura-software-1/commit/2ed1bb5bf3f70ff9a14c371f52c3e9090b6cfd97
- Existência de uma classe App.java sem utilidade