Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

Message Chain #11

Open
PaulaPeraltaA opened this issue Jan 14, 2023 · 1 comment
Open

Message Chain #11

PaulaPeraltaA opened this issue Jan 14, 2023 · 1 comment

Comments

@PaulaPeraltaA
Copy link

Como se puede ver en el método cookRooms se encuentra una variable roomCount que requiere llamar a métodos de la clase Hotel y luego de la clase Room para obtener su valor, esto provocaría que la clase misma dependa de las demás y sea susceptible a cambios. Por este motivo se recomienda usar el refactoring Hide Delegate que elimina esas dependencias innecesarias y se logra que el código sea más legible.

@Coder-ACJHP
Copy link
Owner

Coder-ACJHP commented Jan 16, 2023

Thank you for your interest and feedback, this was a project I did years ago when I was still a newbie. If I remember correctly, the last commit was 5 years ago, and if we look again today, I am sure that at least 50% of the project should be updated, algorithms should be improved and dependency injection should be handled from the scratch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants