Hotel-Management-System-Java icon indicating copy to clipboard operation
Hotel-Management-System-Java copied to clipboard

code smells and refactoring 1

Open BryanEstrada003 opened this issue 2 years ago • 0 comments

Se encontraron dentro de la clase LoginPane.java como code smells Data Clumps debido a que los campos tfusername, tfpwd, Ustar y Pstar se usan juntos en varios lugares, lo que indica que podrían pertenecer a una clase separada y también Feature Envy debido a que el método actionPerformed en el botón de inicio de sesión se preocupa principalmente por los campos tfusername y tfpwd, lo que sugiere que podrían pertenecer a una clase diferente. Para estos casos se aplicaron las técnicas de refactoring como Extract class para crear una clase separada para almacenar los campos tfusername, tfpwd, Ustar y Pstar, y usar esa clase en lugar de los campos individuales. Al crear una clase separada para almacenar los campos tfusername, tfpwd, Ustar y Pstar, se mejora la cohesión de la clase loginPage. Esto significa que la clase loginPage se preocupa solo por la lógica de la ventana de inicio de sesión, mientras que la clase LoginData se preocupa por los detalles específicos del inicio de sesión. También hace que sea más fácil de entender y mantener el código. Move method para mover el método actionPerformed a la nueva clase creada para almacenar los campos tfusername y tfpwd. Al mover el método actionPerformed a la nueva clase LoginData, se asegura que la clase loginPage solo se preocupa por la lógica de la ventana de inicio de sesión, mientras que la clase LoginData se preocupa por la lógica específica del inicio de sesión. Esto también mejora la cohesión de la clase loginPage y hace que sea más fácil de entender y mantener el código. image image

BryanEstrada003 avatar Jan 13 '23 23:01 BryanEstrada003