xpbootcamp / refactoring-code-smell Goto Github PK

Problema: Podemos encontrar que el objeto Empleado contiene unos tipos definidos de forma incorrecta, o en teoría al interactuar con otras clases podría dar problemas en los condicionales activando mas de uno, por lo tanto, debemos quitar esa forma de definir los tipos, el code smell encontrado es Repeated Switch.

Beneficios: A futuro esta técnica de refactoring facilita el hecho de crear mas tipos del mismo objeto trabajando con la herencia, esto ayuda a que no se toque el código original de empleado para agregar mas tipos con números, y nos permite directamente crear nuevas clases que hereden de Empleado.

Solución

Problema: Existen 2 clases con los mismos campos por lo tanto hacen en teoría lo mismo, resulta incómodo que tengan los mismos campos, code smell encontrado Data Clumps.

Beneficios: Este método de refactorización nos ayudará a mantener la adherencia al principio de responsabilidad única, por lo tanto, el código será más obvio y comprensible.

Solución

Problema: Debido al crecimiento del programa se podría volver confuso el hecho de entender que hace la clase y el porqué de los datos primitivos que existen ahí, así mismo estos datos y comportamientos podrían dispersarse a más clases creando código duplicado u otros malos olores en el código que generarían un problema conforme aumenta el código del sistema, por lo tanto, es mejor mover tanto los campos y sus datos relacionados a una nueva clase que se ocupe de ellos.

Beneficios: Los datos y sus comportamientos relevantes ahora se encuentran en una misma clase, lo cual nos asegura que existe encapsulamiento y nos permite manejar estos datos de una manera más segura para evitar fallos en el sistema en caso de que se agreguen nuevas funcionalidades relacionadas con la clase que contiene estos datos.

Solución