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

b08519b41319ff124e54e4b44f0a41f5 #28

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

Conversation

lap110303
Copy link

O código foi refatorado de maneira a torná-lo mais organizado e consistente durante todo o programa, para seu fácil entendimento e compreensão. Comentarei as mudanças de acordo com os tópicos especificados no Moodle.
*Nomenclatura - A nomenclatura foi organizada de forma clara e simples, visando tornar o entendimento de cada função e variável do código possível apenas pelo nome. Além disso, a escolha do nome de cada função e variável segue um padrão bem definido durante todo o programa, já as classes seguem um padrão um pouco diferente, facilitando a diferenciação entre as responsabilidades.
*Formatação - A formatação foi feita de maneira bem espaçada, seguindo um padrão durante todo o programa, visando manter o código limpo e simples.
*Comentários - Como o código é bem simples e a nomenclatura facilita bastante o entendimento, foram feitos poucos comentários, já que sua adição não seria necessária e apenas faria com que o código ficasse mais desorganizado.
*Aplicação Correta de OO - As classes são simples e realizam apenas poucas funções de acordo com suas responsabilidades. As variáveis foram declaradas de modo privado, utilizando gets e sets para sua manipulação fora das suas respectivas classes e a memória alocada no heap é liberada no final.
*Modularização - Os arquivos, antes feitos todos em .hpp, agora são divididos em .hpp e .cpp, facilitando o entendimento da implementação e da especificação.

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