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

6edca9c772179893841bb1c4cda74bfc #42

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

Conversation

heittpr
Copy link

@heittpr heittpr commented Nov 24, 2023

Este é um PR de refatoração. As principais mudanças são:

  • Separação da implementação dos headers em arquivos .cpp próprios
  • Padronização do nome dos métodos e classes e da indentação do código
  • Modificação do escopo de acesso das variáveis das classes
  • Criação de métodos getters para os atributos necessários
  • Criação de um destrutor para a classe Banco, desalocando a memória alocada no heap
  • Criação de construtores para a classe ContaBancaria e suas classes derivadas
  • Remoção do uso desnecessário de using namespace std

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