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

Code Smells #22

Open
DylanEscala opened this issue Jan 15, 2021 · 0 comments
Open

Code Smells #22

DylanEscala opened this issue Jan 15, 2021 · 0 comments

Comments

@DylanEscala
Copy link

Switch Statement
I found a switch on the bookroom method wich is too large
image
You can solve it applying the Replace Type Code with State/Strategy refactoring
image
Dead Code
On the class Singleroom you have a parameter which is not used by the class
image
You can solve it applying Remove Parameter refactoring
image
Large Class
The class Hotel is too large and has too many responsabilities. More than 5 methods.
image
You can solve it creating new classes and group those methods
image
image
image
image
image

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

1 participant