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

Refactoring #1

Closed
sebasurto opened this issue Aug 22, 2022 · 1 comment
Closed

Refactoring #1

sebasurto opened this issue Aug 22, 2022 · 1 comment

Comments

@sebasurto
Copy link

Hola. realizaba una tarea de diseño y encontré tu repositorio. Debía hallar algún code smell y refactorizar.
Se halló lo siguiente.
Lazy Class: Existe una clase para validar si se debe realizar un sorteo, esta clase es innecesaria y se puede refactorizar eliminándola y moviendo el método a otra .

Inappropriate Intimacy En la clase Alumno tenemos una clase para calcular nota, cuyo retorno no es un atributo de alumno y se usa en otra clase. Longer Class

En la clase clases tenemos una clase con demasiados métodos, muchos de estos dedicados al ordenamiento de un atributo de una lista de alumnos cuando, cuando estos pueden simplificarse o usar otra interface dedicada a ese funcionamiento que es más coherente a su naturaleza.

@joseluisgs
Copy link
Owner

Muchas gracias, pero este es un ejercicio para el alumnado oficial.
En cualquier caso lo que propones es correcto y te felicito por tu trabajo

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

No branches or pull requests

2 participants