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

Validación relación user y chatId en Message #79

Open
c4ebt opened this issue Oct 2, 2024 · 1 comment
Open

Validación relación user y chatId en Message #79

c4ebt opened this issue Oct 2, 2024 · 1 comment
Assignees
Labels

Comments

@c4ebt
Copy link

c4ebt commented Oct 2, 2024

El enunciado dice que la entidad Message "contiene el texto del mensaje, la referencia al chat al que pertenece (chatId), y el usuario que lo envía (username). Las FK chatId y username establecen relaciones con las entidades Chat y User, respectivamente".

Si bien se dice que chatId y username son foreign keys que referencian entidades Chat y User, no se especifica nada sobre la rigurosidad con la que están relacionados estos atributos entre sí. Es claro que en términos de funcionamiento de la aplicación, uno esperaría que el username sea partícipe del chat con chatId, esto es, que (en pseudocódigo para facilitar comprensión) username = Chats[chatId].username1 OR username = Chats[chatId].username2. Sin embargo, el enunciado no especifica nada sobre esta validación.

En otras tareas he tenido problemas y me han dicho que es porque mi código "hace más de lo que el enunciado pide que haga", y que esto es incorrecto y no se debería hacer. En este contexto, lo correcto sería no validar la relación anteriormente descrita para hacer solo exactamente lo que pide el enunciado.

Sin embargo, surgen inconsistencias claras en el funcionamiento de la aplicación si esta relación no se valida. Un comportamiento que surgiría y que no tendría sentido está descrito en esta issue. Otro comportamiento absurdo que surgiría sería que en la ruta POST /messages/ para crear un mensaje, no habría ninguna restricción de que el username sea partícipe del chat con id chatId.

¿Hay que implementar la validación en cuestión? ¿O simplemente permitir los comportamientos absurdos que surgen al no implementarla?

@rodrigoogalde
Copy link
Member

rodrigoogalde commented Oct 2, 2024

Hola!
Las issues son para preguntas objetivas y no para emitir opiniones propias, si tienes algún problema con el equipo docente te recomiendo hablar con el profesor o responder la encuesta docente.
Con respecto a tu duda sobre el POST, no haría sentido que se agregara un mensaje a un chat que no corresponde. Puedes revisar el frontend, ya que este no tiene una funcionalidad "absurda" como la que describes.

@rodrigoogalde rodrigoogalde self-assigned this Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants