Skip to content

Comentarios QMP2 #1

@ggallici

Description

@ggallici

Buenas, antes que nada perdón por la demora. Te voy marcando un par de cositas fuí viendo en tu solución:

  • Si implementaras un builder como vimos en clase, podrías tener todas las validaciones en los steps del builder, para desacoplar a la prenda de toda la lógica creacional (validaciones/creación por pasos/mantener inmutable la prenda).
  • aspectosDelMaterial() no parece un mensaje muy cohesivo. Que ganas con él vs tener varios setters?
  • Ojo con tener objetos globales como el guardarropas porque son bastante difíciles para testear. No profundizo demasiado en esto, porque ya vamos a ver en clase estos problemas y cómo solucionarlos.
  • Tener la Trama contenida en el Material no se si es una buena idea. Pensá que tendrías que describir a priori todas las posibles combinaciones.
  • No se entiende muy bien que queres representar con las sugerencias en el Uniforme

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions