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

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

Aluno #161 #161

wants to merge 1 commit into from

Conversation

andre-tavas
Copy link

Em primeiro lugar, foi aplicado o princípio do encapsulamento ao programa. Para isso, foi separada a definição da implementação das classes "Encomenda", "EncomendaNormal" e "EncomendaRelampago", isto por meio da criação dos arquivos .cpp destas classes. Ainda no princípio do encapsulamento, foram trocados os modificadores de acesso de todos os atributos de todas as classes que antes estavam públicos, para a classe "Cliente" foi utilizado o modificador "private" para os atributos e para a classe "Encomenda" modificador "protected" para que as classes filhas pudessem acessar. Por fim, ainda para cumprir com o princípio do encapsulamento foram criados os métodos getters e setters de todos os atributos de todas as classes.
No sentido das boas práticas de programação, foi removido o "using namespace std", além de inicializar os atributos e variáveis com os valores. Ademais, foram modificadas as linhas de código que utilizavam a taxa adicional como um número mágico, nestes lugares colocou-se o próprio nome da variável que também foi modificado para tornar o programa mais compreensível.
Para melhorar a legibilidade do código e seguir os princípios da POO, foram criados os métodos construtures das classes no arquivo main os objetos foram instanciados utilizando este método.

@douglasmacharet douglasmacharet changed the title 2020025897 - André Tavares Guerra Aluno #161 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