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

Duda Buen Uso de Controladores #202

Open
Martinrsts opened this issue Oct 20, 2023 · 1 comment
Open

Duda Buen Uso de Controladores #202

Martinrsts opened this issue Oct 20, 2023 · 1 comment

Comments

@Martinrsts
Copy link

Hola,
Implementé el proyecto de la forma en que hay varios controladores que manejan los datos del juego y jugadores. Tengo una duda para que sea limpio. Algunos controladores se instancian pasándole estructuras de datos y después simplemente se ejecuta con una función del controlador sin argumentos. Otros controladores no se instancian con estructuras de datos, pero su función necesita argumentos (servirían como controladores estáticos). ¿Es necesario que todos sean de alguna forma? ¿Alguna de las dos formas es más correcta? ¿O está bien que haya distintos tipos de controladores, unos que se ejecutan con argumentos y otros que se instancian con argumentos?

@RodrigoToroIcarte
Copy link
Owner

Lamentablemente, la respuesta es depende.

Siempre ayuda que exista cierto grado de consistencia en el código. Hace que alguien externo se acostumbre a ver que los controladores funcionan de cierta forma. Y si encuentra uno que funciona distinto, lo sorprenda. Un código realmente limpio no sorprende a quien lo lee.

Pero los controladores pueden ser muy distintos entre si. Algunos requieren parámetros que son "fijos" para cumplir su responsabilidad, como el view. Normalmente ese tipo de parámetros se da en el constructor. Otros parámetros dependen de cuando se quiera llamar al método (como el jugador sobre el que se aplica el efecto). Normalmente esos parámetros los damos en la función.

Entonces la respuesta es depende. Si puedes hacer que exista cierta consistencia entre los controladores, mejor.

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

2 participants