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

15413d843f1b5b8d036920b5af57f943 #121

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

Conversation

lauragodinho16
Copy link

Análise das alterações feitas

Para melhorar a organização e conteúdos gerais dos arquivos:
Criei os respectivos arquivos .cpp, separando especificação e implementação.

Para melhorar a legibilidade do código:
Alterei o formato dos nomes das variáveis, de modo que todas sigam o padrão camel case;
Alterei os nomes dos métodos "print" para nomes mais específicos do que é mostrado na tela;
Eliminei quebras de linha desnecessárias;
Em cada construtor, criei uma constante TAXA para definir a taxa da comissão.

Para evitar erros e trechos de códigos repetidos:
Em cada construtor, criei uma constante TAXA para definir a taxa da comissão;
As variáveis que definem o valor e a comissão são calculadas no próprio construtor, com base no valor do metro quadrado, na taxa e na comissão;
Em cada construtor, demais variáveis (além de valor e comissão) são inicializadas com valor igual a 0.

Para aplicar os princípios de orientação a objetos:
Fiz a especificação e a implementação de métodos de acesso (getters) e de os métodos modificadores (setters) para cada atributo;
Fiz as devidas mudanças em main.cpp para permitir que cada atributo seja modificado e acessado apenas por meio de métodos da classe;
Fiz a divisão de membros públicos e privados, de modo que os membros privados sejam os atributos de cada classe e os membros públicos sejam: o construtor; os métodos de acesso (getters); os métodos modificadores (setters) e o método para imprimir os dados.

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