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

223e03fc80cf1fe5785954a738902d1d #137

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

Conversation

marcus-laia
Copy link

Formatação e organização:
Alterar nomes das variáveis para o formato camelCase.
No método print(), padronizar o uso de “cout” em todas as linhas.
Adicionar comentários.
Alterar levemente o nome de algumas variáveis para que fique mais intuitivo.

Nomenclatura:
Substituir:
Q por quartos
B por banheiros
V por vagas
C por comissao

Modularização:
Separar arquivos em pastas
Criar makefile

Pull Up Method:
Mover os métodos valor(), comissao() e print() para a superclasse Imovel.

Imovel.hpp:
Deixar apenas a declaração das funções e criar arquivo Imovel.cpp.

Imovel.cpp:
Criar getters and setters.

double valor(): retorna sem criar uma variável desnecessária. Mudar para formato de um getter.

double comissao() -> reutilizar código de valor() e substituir “número mágico” por uma variável TaxaComissao (armazenando o valor correspondente à porcentagem) criada na superclasse. Mudar para formado de um getter.

Alterar os modificadores de acesso prezando pelo encapsulamento.

Apartamento.hpp, Cobertura.hpp, Casa.hpp:
Implementar a definição de variáveis referentes ao tipo do imóvel (nome do tipo e taxa de comissão) no construtor.

main.cpp:
Criar uma lista de imóveis criados e, no momento de gerar o relatório, iterar sobre este conjunto para evitar a repetição de código e aumentar o reúso.

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