-
Notifications
You must be signed in to change notification settings - Fork 55
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
0b6e7d61b7bd5fa931e13b5e3bf3641e #50
Open
JoaoSaude
wants to merge
31
commits into
pds2-dcc-ufmg:main
Choose a base branch
from
JoaoSaude:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+336
−332
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Alterações de nomenclatura: Diversas alterações de nomenclatura foram feitas para adequar o padrão "camelCase" seguido durante todo o código. Além disso, nomes em inglês, letras maiúsculas e outros fora desse padrão, tanto em métodos como em atributos foram alterados.
Alterações de formatação: O código base possuía diversos desalinhamentos de formatação. Haviam muitas linhas fora de identação que foram corrigidas.
Foram adicionados comentários explicativos para facilitar o entendimento de quem lê o código e explicar brevemente as funcionalidades gerais
Os conceitos de POO foram utilizados de forma mais clara principalmente na modularização do código. O código base possuía uma programa muito desprotegido com os dados e com baixo grau de encapsulamento. Acrescentei arquivos cpp para as implementações do método, assim como utilizei de namespaces para criar uma corrente lógica entre os elementos do código. Vi como importante para melhorar a eficiência do desenvolvimento, principalmente na recompilação, a criação de um arquivo Makefile juntamente com a separação dos códigos fonte em duas devidas pastas. Alterei também a instanciação de alguns animais, tendo em vista que não faria sentido um Animal genérico ser instanciado. Coloquei para a superclasse(Animal) o atributo de espécie, já que ele é compartilhado pelas classes derivadas. Também, diversos atributos foram devidamente protegidos com modificadores de acesso e seus métodos get e set foram criados para utilização desses em outras partes do programa. Finalmente, algumas partes do código foram desconsideradas por serem repetições desnecessárias que comprometiam a leitura do programa, assim como partes de código que não faziam sentido lógico de serem empregados, como cabeçalhos com nomes equivocados, variáveis não utilizadas e repetições na implementação de variáveis no escopo main.
Dessa maneira, foram implementadas diversas boas práticas de programação atreladas aos conceitos de POO.