Correcciones y sugerencias QMP4

[SofiCortes]

Buenas! Te dejo algunos comentarios sobre la entrega QMP4.

1. Me perdí un poco con el diagrama de clases, el hecho de que esté en partes dificulta su lectura (cómo se relacionan entre si esas clases? están separadas del resto del sistema?) Te sugiero que por iteración hagas solo 1 diagrama con las clases relevantes a esa iteración (no sería muy grande porque lo limitamos por iteración).
2. Muy bueno que venis haciendo tests para las entregas!
   

   dds/01-lecture/quemepongo/src/test/java/quemepongo/service/ClimaTest.java

   Lines 16 to 20 in 76f3735

   	public void climaBaires() {
   	Assertions.assertDoesNotThrow(() -> {
   	new AccuWeatherAPI().getWeatherForBuenosAires();
   	});
   	}

   
   En este caso, el assert está teniendo poco sentido porque en realidad en todos los casos (excepto los de error) queremos que no arroje excepción un método. No estamos verificando que funcione bien. Una opción es testear lo que devuelve ese método contra lo que esperás que devuelva, de esa forma si tenemos algún error en la lógica nos vamos a enterar.
3. AccuWeatherAPI es parte de un SDK cuyo código no podemos modificar, por lo cual no podemos hacer que implemente ClimaService. Como vimos en clase, sí podemos envolverlo y adaptarlo para que devuelva lo que le sirva al dominio.
4. Ojo con el uso del nombre service. Como vimos en clase, un service puede ser literalmente cualquier cosa. Veo que estás usando la palabra service como nombres de packages en Java. Qué clases queremos agrupar bajo este concepto? Es muy vago y amplio y termina siendo una bolsa de gatos. Una opción es hacer packages que representen funcionalides de dominio.
5. No hagas código que no se vaya a utilizar o por las dudas. Por ejemplo, todo lo que está dentro de Clima... a qué funcionalidad responde del enunciado? Esto generó un sobrediseño, una complejidad accidental que no esperábamos para este ejercicio.
6. Qué funcionalidad cumple la clase Pronostico? No tiene ningún comportamiento. Para evitar esto, podemos tener como atributo ClimaService donde lo vayamos a utilizar.
7. La idea de fabricar las prendas excede al enunciado, y está complejizando mucho el diseño. Se podría hacer mucho más simple si las prendas ya existieran y se eligiera sobre esas. Además, no encontré donde efectivamente se recomienda un atuendo dependiendo del clima.

Saludos!