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 #163 #163

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

Aluno #163 #163

wants to merge 1 commit into from

Conversation

umaanaflavia
Copy link

Geral

  • Foi removida a chamada do namespace std, então string, endl foram alterados para std::string, std::endl
  • Ajustes na indentação e no espaçamento das linhas

Cliente.cpp

  • A chamada de std::cout foi quebrada em várias chamadas menores para ajudar na legibilidade do código
  • Foi acrescentada a implementação do construtor de Cliente

Cliente.hpp

  • Os atributos da classe passaram a ser privados para garantir o encapsulamento
  • Os nomes dos atributos foram padronizados, utilizando letras minúsculas e com um underscore no início por serem atributos privados
  • Foram adicionados dois construtores, um vazio e o cabeçalho de um com parâmetros
  • Foi removido um comentário redundante que havia no código

Encomenda.hpp

  • Os atributos da classe passaram a ser protegidos para garantir o encapsulamento e permitir o acesso das classes filhas
  • Os nomes dos atributos foram padronizados, utilizando letras minúsculas e com um underscore no início por serem atributos protegidos
  • Dois nomes de atributos foram alterados para serem mais descritivos
  • O atributo taxa foi definido com 0 por padrão
  • Foi adicionado um novo método, chamado custo_total para aumentar a reutilização de código, pois ele era utilizado pelas classes filhas da mesma maneira

EncomendaNormal.hpp

  • O método calcula foi removido, pois ele passou a ser implementado na superclasse
  • Foi adicionado um construtor para a classe EncomendaNormal
  • A chamada de std::cout no método print foi quebrado em várias chamadas menores para ajudar na legibilidade do código
  • Os nomes dos atributos foram atualizados no método print

EncomendaRelampago.hpp

  • O método calcula foi removido, pois ele passou a ser implementado na superclasse
  • Foi adicionado um construtor para a classe EncomendaRelampago, onde o atributo taxa foi definido como 0.25
  • A chamada de std::cout no método print foi quebrado em várias chamadas menores para ajudar na legibilidade do código
  • Os nomes dos atributos foram atualizados no método print

main.cpp

  • As declarações de clientes e encomendas foram atualizados para fazer uso dos construtores implementados
  • As variáveis de clientes e encomendas tiveram seus nomes alterados para serem mais descritivos
  • Foi incluída a biblioteca , que foi utilizada para criar dois vetores, encomendas_normais e encomendas_relampago, para armazenar as encomendas
  • Como foram utilizados vetores, as variáveis quantN e quantR se tornaram obsoletas e por isso foram removidas, assim como qualquer referência a elas
  • As variáveis custN e custR tiveram seus nomes alterados para descreverem melhor o que elas fazem
  • Foram criados dois loops for, uma para encomendas normais e um para encomendas relâmpago, que percorreram os seus respectivos vetores, para com isso evitar código repetido
  • A chamada de std::cout no final foi quebrada em várias chamadas menores para ajudar na legibilidade do código
  • Na última chamada de std::cout onde havia \n foi trocado para std::endl

@douglasmacharet douglasmacharet changed the title 2020006353 - Ana Flávia de Matos Souza Aluno #163 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