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

87536870439e964df4db05213800a2b9 #83

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

Conversation

matheusathan
Copy link

Removi a instrução "using namespace std;" dos cabeçalhos para, pois não é recomendado.

Todos os atributos foram tornados privados.

Todas as classes passaram a ter construtores para evitar a configuração manual dos atributos.

Atributos modificados e acessados externamente passaram a ter getters e setters.

Alguns tipos de dados precisaram ser trocados para outros mais convencionais, como é o caso das idades que eram string e passaram a ser int.

Algumas variáveis com o nome todo em maiúsculas foram renomeadas.

As constantes em camelCase passaram a ser snake_case com todas as letras maiúsculas.

Grupos de variáveis semelhantes sucessivamente declaradas passaram a ser Arrays.

Variáveis necessariamente globais passaram a ser constantes.

matheusathan and others added 2 commits July 3, 2022 18:03
Correção do modificador de acesso
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