Etapa 3

[lihuencarranza]

Buenas,

Espero que se encuentren bien. Entrego la etapa 3.

Buen finde!

[dessaya]

En general está muy bien. Igualmente te pido que hagas una reentrega corrigiendo los siguientes items. Solo los primeros (marcados con [R]) son requisito para que el TP pueda estar aprobado.

---

[R] Fallan algunos tests unitarios en BoxTest.

---

[R] En el diagrama de clases que está en el README las flechas que llegan a BoxFactory son incorrectas, ya que la relación no es de herencia.

---

[R] En la línea 152 de Board.Java hay una invocación a new Box(). Eso no es correcto ya que viola el uso del patrón Flyweight. En cualquier lugar que sea necesario una nueva instancia de Box se debería resolver mediante BoxFactory.

---

La llamada BoxFactory.createBox(BoxFactory.Type.BOX, 0) es sospechosa porque ese 0 no tiene significado en ese contexto.

Propongo reemplazar la función createBox() por 4 funciones específicas createBomb(), createBox(n), createClickedBox(n) y createFlag(n). Con ese cambio probablemente se puede eliminar el tipo BoxFactory.Type.

---

Un detalle: puedo seguir jugando luego de que el juego terminó. Debería quedar inhabilitado hasta que inicio un juego nuevo.

---

Salen bastates advertencias. Idealmente no debería haber ninguna.

Description Resource    Path    Location    Type
The enum constant BACK needs a corresponding case label in this enum switch on MouseButton  View.java   /untitled/src/main/java/org/example/juego   line 137    Java Problem
The enum constant FORWARD needs a corresponding case label in this enum switch on MouseButton   View.java   /untitled/src/main/java/org/example/juego   line 137    Java Problem
The enum constant MIDDLE needs a corresponding case label in this enum switch on MouseButton    View.java   /untitled/src/main/java/org/example/juego   line 137    Java Problem
The enum constant NONE needs a corresponding case label in this enum switch on MouseButton  View.java   /untitled/src/main/java/org/example/juego   line 137    Java Problem
The import java.lang.constant.Constable is never used   Controller.java /untitled/src/main/java/org/example/juego   line 3  Java Problem
The import java.util.List is never used Board.java  /untitled/src/main/java/org/example/juego   line 3  Java Problem
The import org.example.juego.Boxes.Box is never used    GameTest.java   /untitled/src/test/java/org/example line 8  Java Problem
The type Box is not exported from this module   Board.java  /untitled/src/main/java/org/example/juego   line 9  Java Problem
The type Box is not exported from this module   Board.java  /untitled/src/main/java/org/example/juego   line 164    Java Problem
The type Box is not exported from this module   BoxFactory.java /untitled/src/main/java/org/example/juego   line 14 Java Problem
The value of the field App.game is not used App.java    /untitled/src/main/java/org/example/juego   line 8  Java Problem
The value of the local variable view is not used    App.java    /untitled/src/main/java/org/example/juego   line 12 Java Problem

---

Hay un poco de código duplicado en las funciones numberImage, askImage, bloodImage, etc.

[lihuencarranza]

Hola, estuve eliminando todos los warnings pero los primeros 4 no me aparecen. Igual seguí la lista y modifiqué todo lo solicitado. Pensando lógicamente, es realmente necesario poner las acciones de los clicks de las acciones BACK, FORWARD... ? Es decir, si no tiene definida ninguna acción, no hace nada y listo, no aparece error.

Reentrega terminada

[dessaya]

Muy bien, está aprobado con 9. Felicitaciones!

No llega al 10 porque:

• Ahora en la segunda revisada noté que la clase BombBox hereda de Box, y por lo tanto hereda un atributo number que no tiene ningún significado en el contexto de BombBox (porque nunca debe mostrarse un número cuando hay una bomba). En realidad veo que se usa el número 10 para indicar que es una bomba, pero eso es un "número mágico" que como mínimo debería estar especificado en una constante. No son cosas graves pero son indicadores de que la herencia no está correctamente usada.
• Todavía pasa que me permite seguir clickeando después de que el juego terminó.
• Todavía hay código duplicado en la carga de imágenes. Por ejemplo la cadena "file:src/main/java/org/example/juego/resources" se repite muchas veces.

[lihuencarranza]

Muchas Gracias. En estos días lo voy a actualizar, además tengo ganas de agregarle mas niveles. Buen fin de año!