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

0f16c9aa2161ea16e181a8d1781bc048 #209

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

Conversation

Chr1s-J
Copy link

@Chr1s-J Chr1s-J commented Nov 29, 2023

  1. Remoção de using namespace std, fazendo devidas alteraçãoes para
    inserir o namespace std no código pertinente.

  2. Alteração de espaçamento e justificação de código para melhor organização.

  3. Remoção de código morto:

    @ Em Cliente.cpp :
    $ Deletado /* #include */ desnecessário, pois está incluso no arquivo Cliente.hpp

    @ Em Cliente.hpp :
    $ Remoção de comentário óbvio

    @ Em Especialista.hpp :
    $ Remoção de /* #include "Cliente.hpp" / desnecessário
    $ Remoção de variável desnecessário /
    double percWanda /
    $ Remoção de declaração /
    public: */ duplicado.

    @ Em Funcionario.hpp :
    $ Remoção de função /* void print_oi() */ desnecessário.

    @ Em Venda.hpp :
    Remoção de comentários desnecessários

    @ Em Gerente.cpp :
    $ Remoção de variável /* double x; / desnecessário na função
    /
    Gerente::calcula_BONIFICACAO_GERENTE() */

  4. Separação de implementação (.cpp) e declaração (.hpp) das classes,
    resultando em inclusão de arquivos:
    @ Especialista.cpp
    @ Gerente.cpp
    @ Funcionario.cpp
    @ Venda.cpp

  5. Criação de pastas para armazenar arquivos de implementação (src/),
    de declaração (include/) e de objetos (obj/)

  6. Criação de Makefile

  7. Aplicação de herança public em classe Gerente. Gerente agora é filho de Funcionario.

  8. Alteração de variáveis :

    @ Em Especialista.hpp :
    $ /* double perc / renomeado / double percentual_comissao */ e movido para arquivo
    Especialista.cpp

    @ Em Gerente.hpp :
    $ /* double ValorBONIFICACAO */ movido para arquivo Gerente.cpp como variável public
    em vez de variável global

  9. Emprego de camelCase para renomeação de variáveis e funções

  10. Criação de Construtor para classe Cliente

  11. Privatização de variáveis da classe Cliente e criação de funções getters para acesso dos dados

  12. Tentativas fracassadas de aplicação do Padrão de Projetos Strategy para função print()
    foram omitidos por motivo de permitir compilação do código

1) Deletado /* #include <string> */ desnecessário, pois está incluso no arquivo Cliente.hpp.

2) Remoção de /* using namespace std; */ e atribuição dos devidos inclusões do namespace /* std:: */ .

3) Alteração de espaçamento e justificação do código.
1) Remoção do /* using namespace std; */ e devidos inclusões de namespace /* std:: */ .

2) Alteração de espaçamento do código.
1) Depois da separação de implementação (.cpp) e declaração (.hpp) destas classes, Adição dos arquivos de implementação:
Especialista.cpp
Gerente.cpp
Funcionario.cpp
Venda.cpp
1) Remoção de /* using namespace std; */ e devidos inclusões de namespace /* std:: */ .

2) Remoção  de /* #include "Cliente.hpp" */ desnecessário.

3)  Remoção de variáveis:
double perc = 0.1; /* Renomeado double percentual_comissao e movido para arquivo Especialista.cpp */
double percWanda = 0.1;

4) Remoção de declaração /* public: */ duplicado.

5) Alteração de espaçamento e justificação do código.
1) Remoção de /* using namespace std */ e adição de devidos namespace /* std:: */ .

2) Remoção de função /* void print_oi() */ desnecessário.

3) Alteração de espaçamento e justificação de código.

4) Separação de implementação de funções para arquivo Funcionario.cpp
1) Remoção de /* using namespace std */ e adição de devidos namespace /* std:: */ .

2) Remoção de:
double ValorBONIFICACAO; /* Movido para arquivo Gerente.cpp */

3) Alteração de espaçamento e justificação de código.

4) Separação de implementação para arquivo Gerente.cpp
1) Remoção de /* using namespace std */ e adição de devidos namespace /* std:: */ .

2) Remoção de código morto (comentários desnecessários).

3) Alteração de espaçamento e justificação de código.

4) Separação de implementação para arquivo Venda.cpp
Criação de um Makefile para estrutura de arquivos pretendido.
1) Remoção de variável /* double x; */ desnecessário na função 
/* Gerente::calcula_BONIFICACAO_GERENTE() */ .
1) Remoção de comentário óbvio.
1) Aplicação de herança public em classe Gerente. Gerente é filho com superclasse em Funcionario.
Uso de construtor de classe Cliente
Criação de Construtor de de Getters
Uso de Getters
Emprego de camelCase
Emprego de camelCase
Emprego de camelCase
Emprego de camelCase
Renomeação rgFuncionario
Emprego de camelCase
Emprego de camelCase
Emprego de camelCase
Emprego de camelCase
Renomeação para NumTotalServicos
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