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

a1a790557b8195629f1fcf62681f8b22 #142

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

a1a790557b8195629f1fcf62681f8b22 #142

wants to merge 1 commit into from

Conversation

analuizakan
Copy link

@analuizakan analuizakan commented Aug 18, 2021

DESCRIÇÃO MUDANÇAS

  • Primeiramente, mudei os nomes dos atributos utilizando a padronização Came Case;
  • Também foi realizado o encapsulamento das classes, uma vez que seus atributos e métodos estavam todos públicos. Foi criado métodos auxiliares para manusear os atributos, gets e setters, e os atributos foram mudados para private;
  • Todas as classes herdeiras compartilhavam de um mesmo método cuja implementação era igual para todos. Tal método foi excluído das classes herdeiras e criado e implementado na classe mãe;
  • Também foi feita a indentação do código;
  • Os nomes dos atributos das classes e de seus métodos foram dados de forma que eles fossem autoexplicativos, anteriormente estavam com nomes arbitrários e não intuitivos;
  • No arquivo main, foram feitas todas as mudanças com o novo encapsulamento. Foram substituídos os acessos diretos aos atributos por seus métodos setters;
  • Em alguns métodos eram utilizadas variáveis auxiliares como retorno, tais variáveis foram excluídas e foi feito o retorno direto

DESCRIÇÃO MUDANÇAS
	Primeiramente, mudei os nomes dos atributos utilizando a padronização Came Case;
	Também foi realizado o encapsulamento das classes, uma vez que seus atributos e métodos estavam todos públicos. Foi criado métodos auxiliares para manusear os atributos, gets e setters, e os atributos foram mudados para private;
	Todas as classes herdeiras compartilhavam de um mesmo método cuja implementação era igual para todos. Tal método foi excluído das classes herdeiras e criado e implementado na classe mãe;
	Também foi feita a indentação do código;
	Os nomes dos atributos das classes e de seus métodos foram dados de forma que eles fossem autoexplicativos, anteriormente estavam com nomes arbitrários e não intuitivos; 
	 No arquivo main, foram feitas todas as mudanças com o novo encapsulamento. Foram substituídos os acessos diretos aos atributos por seus métodos setters;
	Em alguns métodos eram utilizadas variáveis auxiliares como retorno, tais variáveis foram excluídas e foi feito o retorno direto
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