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

Aluno #175 #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Aluno #175 #175

wants to merge 1 commit into from

Conversation

0lilauro
Copy link

Boa noite professor, tudo bom ? Segue a entrega do meu trabalho

Matricula: 2020423868
Nome: LAURO CÉSAR DE OLIVEIRA TEIXEIRA
Curso: Matemática Computacional

Lista de alterações feitas:

  • Modularização das classes:
    • As classes estavam muito bem descritas como heranças então eu apenas decidi separalas em módulos dieferentes, para isso eu criei 2 namespaces para comportar as classes. Além disso eu separei os aquivos headers na pasta include e os arquivos de corpo na pasta src. Os binários gerados, por padrão vão diretamente para a pasta bin.
  • Comentários:
    • Um ponto muito importante e relativamente simples de se fazer é comentar todo o código, e nesse projeto dediquei bastante em especificar cada uma das funções e variáveis dentro das classes
  • Redefinição das classes:
    • Eu peguei as classes existentes no programa e dei uma melhorada na nomenclatura, nos métodos, nos modificadores de acessos, no uso de sobrecarga e sobrescrita e retirei os métodos redundantes.
  • Funções Builders:
    • No main.cpp eu reduzi um pouco das linhas utilizadas apenas definindo um construtor para um vetor de Clientes e de Encomendas (aproveitando o principio de liskov para definir o tipo).
  • Redução de código:
    • Utilizei um For, para reduzir a quantidade de linhas gastas no arquivo de main.cpp

@douglasmacharet douglasmacharet changed the title 2020423868 - LAURO CÉSAR DE OLIVEIRA TEIXEIRA [ Exercício de Revisão ] Aluno #175 Mar 15, 2021
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