Comentarios QMP5

[gastonprieto]

• Sugerencias, no hace falta que mantengas un diagrama grande para todo el sistema, lo podes separar por funcionalidades, o en este caso en entregas de QMP
• A las propuestas del guardarropa, en vez de poner afectarGuardarropas, podrias poner aceptar(), que seria el qué es lo que hace y no el cómo. Me imagino que el desafectar es el deshacer
• Fijate que cuando tenes un campo que es una lista tenes que ponerle el * a la flecha, eso te da la idea que tenes un conjunto. Cuando pones la flecha y el asterisco si queres podes evitar poner el atributo, porque ya es informacion que te da la flecha. Esta misma sugerencia aplica a relaciones unidireccionales, en general la flecha es redudante con el atributo. Para la unidireccional no hace falta que pongas el 1, si no tiene * se asume que es 1.
• Para cuando tenes 2 listas del mismo tipo, se hacen 2 flechas y le podes poner arriba de la flecha el nombre del atributo asi no tenes que gastar ese espacio en la clase.
• https://github.com/igonzalezmartin/QMP1/blob/83d91066edaf49f0cfe159e84a4ae505c463fee6/QMP5/pseudocodigo/guardarropas/PropuestaGuardarropa.txt#L2
  esto no es implicito, esto es explicito. Implicito es cuando no esta especificado en ningun lugar. Igual no esta bueno heredar solo para reutilizar campos, la herencia es una bala de plata, y conviene usarla para cuando tenemos comportamiento duplicado.
• https://github.com/igonzalezmartin/QMP1/blob/83d91066edaf49f0cfe159e84a4ae505c463fee6/QMP5/pseudocodigo/guardarropas/Guardarropa.txt#L24-L25
  Esto te conviene sacarlo y confiar en lo de adentro, pensa que seguramente el aceptar se haga sobre algo que nosotros le mostramos al usuario y el usuario decidio. Entonces si nosotros le vamos a presentar adecuadamente la información esto no deberia pasar.
• Para analizar si tal vez esto podria ser el comportamiento default en la clase abstracta propuesta
  https://github.com/igonzalezmartin/QMP1/blob/83d91066edaf49f0cfe159e84a4ae505c463fee6/QMP5/pseudocodigo/guardarropas/Guardarropa.txt#L26-L28
• me hace un poco de ruido el que la propuesta tenga el guardarropas, y el guardarropas la propuesta. Como aseguras de que siempre sea una creación consistente? creo que podrias tener una sola de las dos referencias y te ahorrarias esta redundancia que me parece que agrega mas complejidad sin un beneficio aparente
• https://github.com/igonzalezmartin/QMP1/blob/83d91066edaf49f0cfe159e84a4ae505c463fee6/QMP5/pseudocodigo/guardarropas/Guardarropa.txt#L33-L35
  Con esto ojo, habria que chequear con el usuario si deshacer significa volverla como pendiente, o solo es cancelar su efecto, pero siga estando como aprobada.