mnigliazzo / qmp-grupo10

0 stars 0 forks source link

QMP Iteración 2 - Grupo 10

Diagrama de clases

Explicacion

Se procede a explicar los requerimientos del dominio, citando la forma de resolucion del ejercicio.

Este punto se resolvio agregando un Enumerable Trama el cual contiene las distintas tramas (limitadas) que puede tener una prenda.

En este caso nosotros creamos una clase estatica dentro de la clase Prenda (aka Nested Class) la cual utilizamos como builder de nuestra prenda. Si querriamos crear una prenda primero especificando su tipo lo que hariamos seria:

       Prenda chomba = new Prenda.PrendaBuilder(TipoPrenda.CHOMBA).build(); 

Lo que no gusta tanto de esta solucion es que luego para seguir cargando la prenda (por ejemplo agregarle su material) tenemos que agregarle setters a la prenda lo que la hara mutable.

Para resolver este punto y relacionado al punto anterior, tuvimos que hacer a la prenda mutable (agregarle setters). Esto tampoco nos gusto mucho porque en realidad en la iteracion 1 el tipo, color principal y categoria eran el punto de partida de una prenda. Esto de tener que ir construyendo la prenda poco a poco podriamos resolverlo de 2 formas. La primera es agregar setters y hacer mutable a la prenda o la otra es ir creando una instancia nueva de la prenda y destruyendo la anterior a medida que queremos agregarle propiedades. La desventaja de la primer opcion es que podriamos llegar a perder consistencia de las prendas y ademas romper reglas de negocio (puesto que la validacion de la prenda se hace en su creacion actualmente) lo que nos llevaria a tener que crear mas validaciones en cada setter de prenda para ver si estamos en un estado consistente.

        Prenda zapatos = new Prenda.PrendaBuilder(TipoPrenda.ZAPATOS)
                .material(Material.CUERO)
                .colorPrimario(new Color(0,0, 0))
                .build();
  un tiempo mas tarde...
      zapatos.setColorSecundario(new Color(255,255,255)); 

Claramente se ve lo feo de usar setters (? En mi opinion seria mejor destruir la prenda y volver a construirla con la ayuda de nuestro builder.

        public Prenda build() {
            Prenda prenda =  new Prenda(this);
            validarPrenda(prenda);
            return prenda;
        }
        public void validarPrenda(Prenda prenda) {
            if(prenda.getTipoPrenda() == null) {
                throw new excepciones.PrendaInvalidaException("La prenda no es valida porque no se cargo el tipo.");
            } else if(prenda.getMaterial() == null) {
                throw new excepciones.PrendaInvalidaException("La prenda no es valida porque no se cargo el material.");
            } else if(prenda.getColorPrincipal() == null) {
                throw new excepciones.PrendaInvalidaException("La prenda no es valida porque no se cargo el color principal.");
            } else if (!prenda.getCategoria().esTipoValido(prenda.getTipoPrenda())){
                throw new excepciones.PrendaInvalidaException("El tipo no pertenece a la categoria seleccionada");
            }
        }

Por otro lado, el equipo de producto está analizando potenciales futuras funcionalidades para la aplicación y, a fin de tener una estimación de su complejidad, nos pidió que esbocemos una solución a los siguientes requerimientos, orientados a integrar el software con colegios e instituciones privadas:

UniformeFactorySanJuan factorySanJuan = new UniformeFactorySanJuan()
factorySanJuan.crearUniforme(); //Tenemos un uniforme listo