Merge pull request #69 from NazaQuintero/refactorFinal

martinboca · web-flow · commit 3e39d3369003 · 2021-08-12T17:13:49.000-03:00
refactorFinal
diff --git a/src/main/java/edu/fiuba/algo3/controladores/ActivarTarjetasEventHandler.java b/src/main/java/edu/fiuba/algo3/controladores/ActivarTarjetasEventHandler.java
@@ -2,7 +2,6 @@
 
 import edu.fiuba.algo3.modelo.Jugador;
 import edu.fiuba.algo3.modelo.excepciones.*;
-import edu.fiuba.algo3.modelo.paises.Pais;
 import edu.fiuba.algo3.modelo.tarjetas.Tarjeta;
 import edu.fiuba.algo3.vista.VentanaTarjetas;
 import javafx.event.EventHandler;
@@ -27,12 +26,10 @@ public void handle(MouseEvent event) {
     }
 
     void activarTarjetas(ArrayList<Tarjeta> tarjetas) {
-        // Usado para no repetir el reproducir sonido en todos los catchs
         boolean error = true;
 
         if (tarjetas.size() == 0) ventanaTarjetas.mostrarError("Se debe seleccionar al menos una tarjeta");
 
-
         for (Tarjeta tarjeta: tarjetas) {
             try {
                 jugador.activarTarjeta(tarjeta);
@@ -42,25 +39,27 @@ void activarTarjetas(ArrayList<Tarjeta> tarjetas) {
             } catch (ElJugadorNoTieneTurnoException | TarjetaNoEncontradaException ignored) {}
 
             catch (LaTarjetaYaFueActivadaException e) {
-                ventanaTarjetas.mostrarError("La tarjeta de " + tarjeta.obtenerPais().getNombre() +" ya fue activada");
+                ventanaTarjetas.mostrarError("La tarjeta de " + tarjeta.getPais().getNombre() + " ya fue activada");
             }
             catch (ActivacionTarjetaEnRondaEquivocadaException e) {
                 ventanaTarjetas.mostrarError("No se pueden activar tarjetas en ronda de ataque");
             }
             catch (JugadorNoPoseePaisDeLaTarjetaException e) {
-                ventanaTarjetas.mostrarError("El jugador no posee el pais " + tarjeta.obtenerPais().getNombre());
+                ventanaTarjetas.mostrarError("El jugador no posee el pais " + tarjeta.getPais().getNombre());
             }
 
             ventanaTarjetas.deseleccionarVistaTarjeta(tarjeta);
         }
+
         if (error) ReproductorDeSonido.getInstance().playError();
         else ReproductorDeSonido.getInstance().playClick();
+
     }
 
     private String crearMensajeExito(ArrayList<Tarjeta> tarjetas){
         String mensaje = "Tarjetas activadas correctamente";
         if (tarjetas.size() == 1)
-            mensaje = "Tarjeta " + tarjetas.get(0).obtenerPais().getNombre() +" activada correctamente";
+            mensaje = "Tarjeta " + tarjetas.get(0).getPais().getNombre() +" activada correctamente";
         return mensaje;
     }
 
diff --git a/src/main/java/edu/fiuba/algo3/controladores/AtaqueEventHandler.java b/src/main/java/edu/fiuba/algo3/controladores/AtaqueEventHandler.java
@@ -46,7 +46,6 @@ public void handle(Event event) {
                 int cantidadDeEjercitos = Integer.parseInt(inputText.getText());
                 Resultado resultado = atacante.atacarA(defensor, cantidadDeEjercitos);
 
-
                 menuLateralDerecho.update();
                 mostrarAtaque();
                 ProcesadorResultado.obtenerInstancia().procesar(resultado);
diff --git a/src/main/java/edu/fiuba/algo3/controladores/PaisEventHandler.java b/src/main/java/edu/fiuba/algo3/controladores/PaisEventHandler.java
@@ -30,8 +30,6 @@ public PaisEventHandler(VistaPais vistaPais, CampoDeJuego campoDeJuego) {
     @Override
     public void handle(MouseEvent mouseEvent) {
 
-
-
         Ronda ronda = this.campoDeJuego.getJuego().getTurno().obtenerRondaActual();
         MenuLateralDerecho menuLateral = (MenuLateralDerecho) campoDeJuego.getRight();
         VBox formularioDeAccion = (VBox) menuLateral.getChildren().get(1);
@@ -45,8 +43,9 @@ public void handle(MouseEvent mouseEvent) {
                 ronda.colocarEjercitos(pais,1);
                 ReproductorDeSonido.getInstance().playClick();
                 menuLateral.update();
-            } catch (NoEsRondaDeColocacionException | NoQuedanMasEjercitosPorColocarException e) {
+            } catch (NoEsRondaDeColocacionException ignored) {}
 
+            catch (NoQuedanMasEjercitosPorColocarException e) {
                 ReproductorDeSonido.getInstance().playError();
                 menuLateral.mostrarErrorColocacion();
                 Label labelDeError = (Label)(formularioDeAccion.getChildren().get(4));
diff --git a/src/main/java/edu/fiuba/algo3/modelo/CargarJuego.java b/src/main/java/edu/fiuba/algo3/modelo/CargarJuego.java
@@ -17,12 +17,16 @@
 
 public class CargarJuego {
 
-    public static void cargarContinentesYPaises(String archivoContinentesYPaises) throws ArchivoDeContinentesYPaisesNoEncontradoException {
+    private static final String ARCHIVO_CONTINENTES_Y_PAISES = "continentesYPaises.json";
+    private static final String ARCHIVO_PAISES_LIMITROFES = "paisesLimitrofes.json";
+    private static final String ARCHIVO_TARJETAS = "tarjetas.json";
+
+    public static void cargarContinentesYPaises() throws ArchivoDeContinentesYPaisesNoEncontradoException {
         String json;
         Gson gson = new Gson();
 
         try {
-            InputStream is = App.class.getClassLoader().getResourceAsStream(archivoContinentesYPaises);
+            InputStream is = App.class.getClassLoader().getResourceAsStream(ARCHIVO_CONTINENTES_Y_PAISES);
             json = new String(Objects.requireNonNull(is).readAllBytes(), StandardCharsets.UTF_8);
         }
         catch (IOException | NullPointerException e) {
@@ -39,11 +43,11 @@ public static void cargarContinentesYPaises(String archivoContinentesYPaises) th
         MultitonContinentes.cargarContinentes(continentes);
     }
 
-    public static void cargarPaisesLimitrofes(String archivoPaisesLimitrofes) throws ArchivoDePaisesLimitrofesNoEncontradoException {
+    public static void cargarPaisesLimitrofes() throws ArchivoDePaisesLimitrofesNoEncontradoException {
         String json;
 
         try {
-            InputStream is = App.class.getClassLoader().getResourceAsStream(archivoPaisesLimitrofes);
+            InputStream is = App.class.getClassLoader().getResourceAsStream(ARCHIVO_PAISES_LIMITROFES);
             json = new String(Objects.requireNonNull(is).readAllBytes(), StandardCharsets.UTF_8);
         } catch (IOException | NullPointerException e) {
             throw new ArchivoDePaisesLimitrofesNoEncontradoException();
@@ -74,11 +78,11 @@ public static void cargarPaisesLimitrofes(String archivoPaisesLimitrofes) throws
 
     }
 
-    public static void cargarTarjetas(String archivoTarjetas) throws ArchivoDeTarjetasNoEncontradoException {
+    public static void cargarTarjetas() throws ArchivoDeTarjetasNoEncontradoException {
         String json;
 
         try {
-            InputStream is = CargarJuego.class.getClassLoader().getResourceAsStream(archivoTarjetas);
+            InputStream is = CargarJuego.class.getClassLoader().getResourceAsStream(ARCHIVO_TARJETAS);
             json = new String(Objects.requireNonNull(is).readAllBytes(), StandardCharsets.UTF_8);
         }
         catch (IOException | NullPointerException e) {
diff --git a/src/main/java/edu/fiuba/algo3/modelo/Juego.java b/src/main/java/edu/fiuba/algo3/modelo/Juego.java
@@ -9,19 +9,20 @@
 
 public class Juego {
 
-    private static final String ARCHIVO_CONTINENTES_Y_PAISES = "continentesYPaises.json";
-    private static final String ARCHIVO_PAISES_LIMITROFES = "paisesLimitrofes.json";
-    private static final String ARCHIVO_TARJETAS = "tarjetas.json";
-
     private final Jugadores jugadores;
     private Turno turno = new SinTurno();
 
     public Juego() throws ArchivoDeContinentesYPaisesNoEncontradoException, ArchivoDePaisesLimitrofesNoEncontradoException, ArchivoDeTarjetasNoEncontradoException {
         jugadores = new Jugadores();
 
-        CargarJuego.cargarContinentesYPaises(ARCHIVO_CONTINENTES_Y_PAISES);
-        CargarJuego.cargarPaisesLimitrofes(ARCHIVO_PAISES_LIMITROFES);
-        CargarJuego.cargarTarjetas(ARCHIVO_TARJETAS);
+        CargarJuego.cargarContinentesYPaises();
+        CargarJuego.cargarPaisesLimitrofes();
+        CargarJuego.cargarTarjetas();
+
+    }
+
+    public Turno getTurno() {
+        return turno;
     }
 
     public void agregarJugador(Jugador jugador){
@@ -31,15 +32,11 @@ public void agregarJugador(Jugador jugador){
     public void comenzar() throws CantidadDeJugadoresInsuficienteException {
         if (jugadores.obtenerCantidad() < 2) throw new CantidadDeJugadoresInsuficienteException();
 
-        RepartidorDePaises repartidorDePaises = new RepartidorDePaises();
-        repartidorDePaises.repartirPaises(jugadores);
+        RepartidorDePaises.repartirPaises(jugadores);
         jugadores.mezclar();
         Objetivos.asignarObjetivos(jugadores);
         turno = new ConTurno(jugadores);
-    }
 
-    public Turno getTurno() {
-        return turno;
     }
 
     public boolean estaTerminado() {
diff --git a/src/main/java/edu/fiuba/algo3/modelo/Jugador.java b/src/main/java/edu/fiuba/algo3/modelo/Jugador.java
@@ -85,21 +85,25 @@ public void recibirTarjeta(Tarjeta tarjeta){
     }
 
     public void activarTarjeta(Tarjeta tarjeta) throws JugadorNoPoseePaisDeLaTarjetaException, ActivacionTarjetaEnRondaEquivocadaException, ElJugadorNoTieneTurnoException, LaTarjetaYaFueActivadaException, TarjetaNoEncontradaException {
+
         if (!this.tarjetas.contains(tarjeta)) throw new TarjetaNoEncontradaException();
-        if (!this.paisesDominados.contains(tarjeta.obtenerPais())) throw new JugadorNoPoseePaisDeLaTarjetaException();
+        if (!this.paisesDominados.contains(tarjeta.getPais())) throw new JugadorNoPoseePaisDeLaTarjetaException();
+
         this.turno.activarTarjeta(tarjeta);
+
     }
 
     public void colocarEjercitos(Pais pais, int cantidadEjercitos) throws ElJugadorNoTieneTurnoException, NoEsRondaDeColocacionException, PaisOcupadoPorOtroJugadorException, NoQuedanMasEjercitosPorColocarException {
+
         if (paisesDominados.contains(pais)) {
             this.turno.colocarEjercitos(pais, cantidadEjercitos);
         }
-
-        else if (pais.estaLibre()){
+        else if (pais.estaLibre()) {
             pais.colocarEjercito(new Ejercito(this));
             pais.modificarCantidadEjercito(cantidadEjercitos-1);
         }
         else throw new PaisOcupadoPorOtroJugadorException();
+
     }
 
     public void finalizarRonda() throws ElJugadorNoTieneTurnoException{
@@ -135,9 +139,7 @@ public void canjearTarjetas(ArrayList<Tarjeta> tarjetas) throws JugadorNoTieneTo
     public void devolverTarjetas(ArrayList<Tarjeta> tarjetasADevolver) {
 
         tarjetasADevolver.forEach(tarjeta -> {
-            try {
-                tarjeta.desactivar();
-            } catch (LaTarjetaYaEstaDesactivadaException ignored) {}
+            tarjeta.desactivar();
             MultitonTarjetas.agregarTarjeta(tarjeta);
             tarjetas.remove(tarjeta);
         });
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/Canje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/Canje.java
@@ -16,7 +16,7 @@ public Canje() {
 
     public abstract Canje siguienteCanje();
 
-    public abstract int cantidadEjercitos(); // Setear canjeActivado como true
+    public abstract int cantidadEjercitos();
 
     public Canje habilitarCanje(ArrayList<Tarjeta> tarjetas) throws SinCanjeHabilitadoException {
 
@@ -26,4 +26,5 @@ public Canje habilitarCanje(ArrayList<Tarjeta> tarjetas) throws SinCanjeHabilita
         return nuevoCanje;
 
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/NesimoCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/NesimoCanje.java
@@ -21,4 +21,5 @@ public int cantidadEjercitos() {
         canjeActivado = true;
         return cantidadEjercitos;
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/PrimerCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/PrimerCanje.java
@@ -11,4 +11,5 @@ public int cantidadEjercitos() {
         canjeActivado = true;
         return 4;
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/SegundoCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/SegundoCanje.java
@@ -11,4 +11,5 @@ public int cantidadEjercitos() {
         canjeActivado = true;
         return 7;
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/SinCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/SinCanje.java
@@ -16,4 +16,5 @@ public TipoCanje compararTarjetas(ArrayList<Tarjeta> tarjetas) {
     public Canje canjear(Canje canje) throws SinCanjeHabilitadoException {
         throw new SinCanjeHabilitadoException();
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/TercerCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/TercerCanje.java
@@ -11,4 +11,5 @@ public int cantidadEjercitos() {
         canjeActivado = true;
         return 10;
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/canjes/TipoCanje.java b/src/main/java/edu/fiuba/algo3/modelo/canjes/TipoCanje.java
@@ -6,6 +6,7 @@
 import java.util.ArrayList;
 
 public interface TipoCanje {
+
     TipoCanje compararTarjetas(ArrayList<Tarjeta> tarjetas);
 
     Canje canjear(Canje canje) throws SinCanjeHabilitadoException;
diff --git a/src/main/java/edu/fiuba/algo3/modelo/excepciones/PaisOcupadoPorOtroJugadorException.java b/src/main/java/edu/fiuba/algo3/modelo/excepciones/PaisOcupadoPorOtroJugadorException.java
@@ -1,4 +1,4 @@
 package edu.fiuba.algo3.modelo.excepciones;
 
-public class PaisOcupadoPorOtroJugadorException extends Exception{
+public class PaisOcupadoPorOtroJugadorException extends Exception {
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/objetivos/Objetivo.java b/src/main/java/edu/fiuba/algo3/modelo/objetivos/Objetivo.java
@@ -4,6 +4,9 @@
 import edu.fiuba.algo3.modelo.excepciones.ContinenteInvalidoException;
 
 public interface Objetivo {
+
     boolean estaCumplido(Jugador jugador) throws ContinenteInvalidoException;
+
     String obtenerDescripcion();
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/paises/RepartidorDePaises.java b/src/main/java/edu/fiuba/algo3/modelo/paises/RepartidorDePaises.java
@@ -11,7 +11,8 @@
 
 public class RepartidorDePaises {
 
-    public void repartirPaises(Jugadores jugadores) {
+    public static void repartirPaises(Jugadores jugadores) {
+
         int cantidadJugadores = jugadores.obtenerCantidad();
         ArrayList<Pais> paises = new ArrayList<>(MultitonPaises.obtenerTodosLosPaises());
 
diff --git a/src/main/java/edu/fiuba/algo3/modelo/rondas/ColocacionInicial.java b/src/main/java/edu/fiuba/algo3/modelo/rondas/ColocacionInicial.java
@@ -4,6 +4,7 @@
 import edu.fiuba.algo3.modelo.turnos.Turno;
 
 public class ColocacionInicial extends RondaColocacion {
+
     public ColocacionInicial(Jugador unJugador){
         super(unJugador);
     }
@@ -21,7 +22,9 @@ public String obtenerDescripcion() {
     @Override
     public void finalizarRonda(Turno unTurno) {
         unTurno.finalizarTurnoActual();
+
         if (unTurno.leTocaALPrimerJugador()) unTurno.setRonda(new ColocacionSecundaria(unTurno.obtenerJugadorTurnoActual()));
         else calcularEjercitosColocables(unTurno.obtenerJugadorTurnoActual());
     }
+
 }
diff --git a/src/main/java/edu/fiuba/algo3/modelo/rondas/RondaColocacion.java b/src/main/java/edu/fiuba/algo3/modelo/rondas/RondaColocacion.java
@@ -38,13 +38,15 @@ public void reagrupar(Pais origen, Pais destino, int cantidad) throws NoEsRondaD
     }
 
     public void colocarEjercitos(Pais unPais, int cantidadEjercitos) throws NoQuedanMasEjercitosPorColocarException {
+
         if (this.cantidadEjercitosColocables > 0) {
             if (cantidadEjercitos <= this.cantidadEjercitosColocables) {
                 unPais.modificarCantidadEjercito(cantidadEjercitos);
                 this.cantidadEjercitosColocables -= cantidadEjercitos;
             }
         }
         else throw new NoQuedanMasEjercitosPorColocarException();
+
     }
 
     public void activarTarjeta(Tarjeta unaTarjeta) throws LaTarjetaYaFueActivadaException {
diff --git a/src/main/java/edu/fiuba/algo3/modelo/tarjetas/MultitonTarjetas.java b/src/main/java/edu/fiuba/algo3/modelo/tarjetas/MultitonTarjetas.java
@@ -14,7 +14,7 @@ public class MultitonTarjetas implements Subject {
     private MultitonTarjetas() {}
 
     public static void agregarTarjeta(Tarjeta unaTarjeta) {
-        tarjetas.put(unaTarjeta.obtenerPais(), unaTarjeta);
+        tarjetas.put(unaTarjeta.getPais(), unaTarjeta);
     }
 
     public static Tarjeta obtenerTarjeta(Pais unPais) {
@@ -24,7 +24,7 @@ public static Tarjeta obtenerTarjeta(Pais unPais) {
     public static Tarjeta obtenerTarjetaAleatoria() {
         ArrayList<Tarjeta> _tarjetas = new ArrayList<>(tarjetas.values());
         Collections.shuffle(_tarjetas);
-        return tarjetas.remove(_tarjetas.get(0).obtenerPais());
+        return tarjetas.remove(_tarjetas.get(0).getPais());
     }
 
     @Override
diff --git a/src/main/java/edu/fiuba/algo3/modelo/tarjetas/Tarjeta.java b/src/main/java/edu/fiuba/algo3/modelo/tarjetas/Tarjeta.java
diff --git a/src/main/java/edu/fiuba/algo3/modelo/tarjetas/TarjetaActivada.java b/src/main/java/edu/fiuba/algo3/modelo/tarjetas/TarjetaActivada.java
diff --git a/src/main/java/edu/fiuba/algo3/modelo/tarjetas/TarjetaDesactivada.java b/src/main/java/edu/fiuba/algo3/modelo/tarjetas/TarjetaDesactivada.java
diff --git a/src/main/java/edu/fiuba/algo3/vista/VistaTarjeta.java b/src/main/java/edu/fiuba/algo3/vista/VistaTarjeta.java
diff --git a/src/main/test/edu/fiuba/algo3/modelo/ExcepcionesTest.java b/src/main/test/edu/fiuba/algo3/modelo/ExcepcionesTest.java
diff --git a/src/main/test/edu/fiuba/algo3/modelo/MultitonTest.java b/src/main/test/edu/fiuba/algo3/modelo/MultitonTest.java
diff --git a/src/main/test/edu/fiuba/algo3/modelo/ObjetivosTest.java b/src/main/test/edu/fiuba/algo3/modelo/ObjetivosTest.java
diff --git a/src/main/test/edu/fiuba/algo3/modelo/TarjetaTest.java b/src/main/test/edu/fiuba/algo3/modelo/TarjetaTest.java

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ public Canje() {`
`16`	`16`
`17`	`17`	`public abstract Canje siguienteCanje();`
`18`	`18`
`19`		`- public abstract int cantidadEjercitos(); // Setear canjeActivado como true`
	`19`	`+ public abstract int cantidadEjercitos();`
`20`	`20`
`21`	`21`	`public Canje habilitarCanje(ArrayList<Tarjeta> tarjetas) throws SinCanjeHabilitadoException {`
`22`	`22`
`@@ -26,4 +26,5 @@ public Canje habilitarCanje(ArrayList<Tarjeta> tarjetas) throws SinCanjeHabilita`
`26`	`26`	`return nuevoCanje;`
`27`	`27`
`28`	`28`	`}`
	`29`	`+`
`29`	`30`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,4 +21,5 @@ public int cantidadEjercitos() {`
`21`	`21`	`canjeActivado = true;`
`22`	`22`	`return cantidadEjercitos;`
`23`	`23`	`}`
	`24`	`+`
`24`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,4 +11,5 @@ public int cantidadEjercitos() {`
`11`	`11`	`canjeActivado = true;`
`12`	`12`	`return 4;`
`13`	`13`	`}`
	`14`	`+`
`14`	`15`	`}`