shefalidigikar / hotel-management-system-java Goto Github PK

Dentro de la clase room.java existe un dead code, un código que fue comentado, pero no se ha eliminado. La solución de manera sencilla es retirar ese código para mantener pulcra la clase.

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.