Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

7171f5d8d78ce95d69ff736e13105042 #77

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mateuskrause
Copy link

Propostas de mudanças ao código:

O código base possui diversos problemas, desde mais explícitos como por exemplo a não padronização nos diversos elementos e não ter como ser mantido.

Proponho uma revisão e refatoração mais focada na orientação à objetos, e sua capacidade de organizar a ponto de deixar mais aberto à mudanças futuras, como uma possível extensão do programa.

Modificações:

  • Padronização nos nomes de variáveis, em português - lowerCamelCase para nomes gerais e UPPER_SNAKE_CASE para include guards ;
  • Aplicação de um guia de estilo para indentações e espaços;
  • Remoção de código comentado ou inutilizado;
  • Definição mais clara das classes e suas hierarquias;
  • Modularização. Separação em pastas.
  • Encapsulamento coeso;
  • Definição de tipos adequados às variáveis;
  • Organização da função main;
  • Armazenamento dos dados em vectors;
  • Divisão em funções.
  • Criação de um Makefile
  • Adicionado .gitignore

Durante o processo foi realizado um planejamento futuro - caso sejam adicionadas mais famílias de animais, facilitando a extensão.

Comentários:

Mesmo com as modificações, ainda existem práticas, que devido o requerimento de mesmo output, foram mantidas, como adicionar animais depois de imprimir, por exemplo.

Ao fim, o programa apresenta a mesma saída, mas de uma forma mais organizada e extensível.

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.

1 participant