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.

Sign up for GitHub

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

Jump to bottom

5b52e39907bc5bac9285526f490748f3 #193

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

5b52e39907bc5bac9285526f490748f3 #193

wants to merge 9 commits into from

Conversation

isabelassuncao
Copy link

As seguintes modificiações foram feitas para melhorar o código:

  1. Mudança no nome das variáveis:
    • respeitando o mesmo padrão camelCase
    • constantes em CAPSLOCK
  2. Retirada de código morto, que inclui:
    • comentários sem sentido
    • variáveis inutilizadas
    • constantes inutilizadas
    • texto duplicado
  3. Criação de construtores para facilitar a instanciação de objetos das classes
  4. Mudança na classe Gerente para ser herdada de Funcionário, evitando a repetição de código

@isabelassuncao
Mudando nomes
4291409 
ajustando todos para o mesmo tipo de escrita: camelCase
@isabelassuncao
Tirando código morto
0c0885b 
removendo variáveis que não são utilizadas e comentários inúteis
@isabelassuncao
Tirando código morto pt 2
27ee80c
@isabelassuncao
Tirando código morto pt 3
4fd67cd 
variável especialidade não era usada e coisas duplicadas
@isabelassuncao
Corrigindo erro
7854570 
Faltou um public
@isabelassuncao
Herança
f7325c2 
Gerente agora é herdado de Funcionário
@isabelassuncao
Identação e organização
168170a
@isabelassuncao
Criando construtores
ff4ab3d 
Criei construtores para facilitar a instanciação dos objetos.
@isabelassuncao
erro
cb692cc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@isabelassuncao