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

8fdfdac9e8dc17daf99818aebd34d1c0 #187

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

Conversation

EricSoBo
Copy link

Modificações na Classe Cliente:

  • Encapsulamento dos atributos (private);
  • Criação de um construtor que recebe os parâmetros do cliente diretamente;
  • Retirada do atributo AlturaDosPais que não realizava nada;
  • Em Cliente.cpp havia include desnecessário de ;
  • Método void print() agora é marcado como const.

Modificações na Classe Funcionario

  • Encapsulamento dos atributos (protected);
  • Retirada do método void print_oi() que não era chamado em momento algum;
  • não era usado;
  • Funcionario agora é Classe Abstrata e agora possui métodos que são virtuais;
  • Método void print() agora é marcado como const e não possui corpo;
  • Destrutor é virtual;
  • Criação do método std::string GetNome().

Modificações na Classe Gerente:

  • Gerente agora herda de Funcionario;
  • Criação de um construtor que recebe os parâmetros do Gerente diretamente;
  • Método print conta com a palavra "override";
  • Criação do método double GetSalarioTotal().

Modificações na Classe Especialista:

  • Método print conta com a palavra "override";
  • Incluia "Cliente.hpp" sem necessidade;
  • Variável "double percWanda" não usada;
  • Criação do método void AtendeuCliente();
  • Criação dos atributos int numAtendimentos e int comiss;
  • Criação dos métodos que retornam número de atendimentos e o salário total;
  • Remoção de duplicidade.

Modificaçoes na Classe Venda:

  • Encapsulamento dos atributos (private);
  • Incluia "Funcionario.hpp" sem necessidade;
  • Criação de um construtor que recebe os parâmetros da venda diretamente;
  • Agora recebe um ponteiro para um especialista ao invés de receber a cópia de um;
  • Novo atributo estático que contabiliza o número de vendas;
  • Criação de métodos que retornam a descrição e o número de vendas;
  • Remoção de código morto ou repetido.

Modificações no Main:

  • Adaptação do código aos novos métodos, atributos e construtores.

Modificações no geral:

  • Eu retirei o uso do espaço de nomes std:: em todos .hpp e .cpp com exceção do main (uma questão de gosto mesmo);
  • Tentei uniformizar o nome das funções e das variáveis a fim de padroniza-las.

- Encapsulamento dos atributos (private);
- Criação de um construtor que recebe os parâmetros do cliente diretamente;
- Retirada do atributo AlturaDosPais que não realizava nada;
- Em Cliente.cpp havia include desnecessário de <string>;
- Método void print() agora é marcado como const.
- Método print conta com a palavra "override";
- Incluia "Cliente.hpp" sem necessidade;
- Variável "double percWanda" não usada;
- Criação do método void AtendeuCliente();
- Criação dos atributos int numAtendimentos e int comiss;
- Criação dos métodos que retornam número de atendimentos e o salário total;
- Remoção de duplicidade.
- Encapsulamento dos atributos (protected);
- Retirada do método void print_oi() que não era chamado em momento algum;
- <iomanip> não era usado;
- Funcionario agora é Classe Abstrata e agora possui métodos que são virtuais;
- Método void print() agora é marcado como const e não possui corpo;
- Destrutor é virtual;
- Criação do método std::string GetNome().
- Gerente agora herda de Funcionario;
- Criação de um construtor que recebe os parâmetros do Gerente diretamente;
- Método print conta com a palavra "override";
- Criação do método double GetSalarioTotal().
- Encapsulamento dos atributos (private);
- Incluia "Funcionario.hpp" sem necessidade;
- Criação de um construtor que recebe os parâmetros da venda diretamente;
- Agora recebe um ponteiro para um especialista ao invés de receber a cópia de um;
- Novo atributo estático que contabiliza o número de vendas;
- Criação de métodos que retornam a descrição e o número de vendas;
- Remoção de código morto ou repetido.
- Adaptação do código aos novos métodos, atributos e construtores.

Modificações no geral:

- Eu retirei o uso do espaço de nomes std:: em todos .hpp e .cpp com exceção do main (uma questão de gosto mesmo);
- Tentei uniformizar o nome das funções e das variáveis a fim de padroniza-las.
@EricSoBo EricSoBo changed the title Revisão de código e Rafatoração 8fdfdac9e8dc17daf99818aebd34d1c0 Nov 26, 2023
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