#17: Refactor sobre seccion de puntuacion, para tener un ControlDePuntaje, implementando Observer

Author: delucas

src/main/java/jwarrior/juego/Juego.java

@@ -5,7 +5,7 @@
 
 import jwarrior.comandos.Comando;
 import jwarrior.juego.niveles.Nivel;
-import jwarrior.juego.puntuacion.Puntaje;
+import jwarrior.juego.puntuacion.ControlDePuntaje;
 import jwarrior.piezas.unidades.Unidad;
 import jwarrior.ui.InterfazDeUsuario;
 
@@ -14,24 +14,16 @@ public class Juego {
 	private static final int CANTIDAD_MAXIMA_TURNOS = 100;
 	private final Mapa mapa;
 	private final Nivel nivel;
-	private Puntaje puntaje = new Puntaje();
 
-	private static Juego instancia;
+	private final ControlDePuntaje controlPuntaje = new ControlDePuntaje();
 
 	public Juego(final Nivel nivel) {
 		this.nivel = nivel;
-		this.mapa = new Mapa(nivel.obtenerPosiciones());
-		instancia = this;
+		this.mapa = new Mapa(nivel.obtenerPosiciones(controlPuntaje));
 	}
 
-	public static Juego obtenerInstancia() {
-		return instancia;
-	}
-
-	public void notificarMuerte(final Unidad unidad) {
-		if (unidad != this.mapa.obtenerGuerrero()) {
-			this.puntaje.notificarMuerteEnemigo(unidad);
-		}
+	public Unidad obtenerGuerrero() {
+		return this.mapa.obtenerGuerrero();
 	}
 
 	private void mensaje(final String mensaje) {
@@ -58,11 +50,11 @@ public void jugar() {
 			turno++;
 			if (turno > CANTIDAD_MAXIMA_TURNOS || this.mapa.estaTerminado()) {
 				juegoTerminado = true;
-				this.puntaje.notificarFinalizacionPartida(this.mapa.obtenerGuerrero());
+				this.controlPuntaje.notificarFinal(this);
 				mensaje("juego terminado");
-				mensaje("Se ha obtenido un puntaje de " + this.puntaje.obtenerPuntaje());
+				mensaje("Se ha obtenido un puntaje de " + this.controlPuntaje.obtenerPuntaje());
 			} else {
-				this.puntaje.notificarTurno();
+				this.controlPuntaje.notificarTurno(this);
 				mensaje("");
 			}
 		}

src/main/java/jwarrior/juego/niveles/ConstructorDeNiveles.java

@@ -3,6 +3,7 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import jwarrior.observadores.ObservadorDeUnidades;
 import jwarrior.piezas.Puerta;
 import jwarrior.piezas.Salida;
 import jwarrior.piezas.unidades.Guerrero;
@@ -11,11 +12,13 @@
 
 public class ConstructorDeNiveles {
 
-	public List<Posicion> construirPosiciones(final String plano, final Guerrero guerrero) {
+	public List<Posicion> construirPosiciones(final String plano,
+			final ObservadorDeUnidades observadorDeUnidades,
+			final Guerrero guerrero) {
 		List<Posicion> posiciones = new ArrayList<Posicion>();
 
 		for (char c : plano.toCharArray()) {
-			posiciones.add(new PosicionCodificada(c).construirPosicion(guerrero));
+			posiciones.add(new PosicionCodificada(c, observadorDeUnidades).construirPosicion(guerrero));
 		}
 		return posiciones;
 	}
@@ -24,9 +27,11 @@ public List<Posicion> construirPosiciones(final String plano, final Guerrero gue
 
 class PosicionCodificada {
 	private final char codigo;
+	private final ObservadorDeUnidades observadorDeUnidades;
 
-	public PosicionCodificada(final char codigo) {
+	public PosicionCodificada(final char codigo, final ObservadorDeUnidades observadorDeUnidades) {
 		this.codigo = codigo;
+		this.observadorDeUnidades = observadorDeUnidades;
 	}
 
 	public Posicion construirPosicion(final Guerrero guerrero) {
@@ -35,13 +40,16 @@ public Posicion construirPosicion(final Guerrero guerrero) {
 			retorno = new Puerta();
 		}
 		if (esGuerrero()) {
+			guerrero.agregarObservador(this.observadorDeUnidades);
 			retorno = new Posicion(guerrero);
 		}
 		if (esSalida()) {
 			retorno = new Salida();
 		}
 		if (esOrco()) {
-			retorno = new Posicion(new Orco());
+			Orco orco = new Orco();
+			orco.agregarObservador(this.observadorDeUnidades);
+			retorno = new Posicion(orco);
 		}
 		return retorno;
 	}

src/main/java/jwarrior/juego/niveles/Nivel.java

@@ -3,32 +3,34 @@
 import java.util.List;
 
 import jwarrior.juego.Jugador;
+import jwarrior.observadores.ObservadorDeUnidades;
 import jwarrior.piezas.unidades.Guerrero;
 import jwarrior.posiciones.Posicion;
 
 public abstract class Nivel {
 
 	private final String nombre;
 	private final String plano;
-	private Jugador jugador;
+	private final Jugador jugador;
 
 	public Nivel(final String nombre, final String plano, final Jugador jugador) {
 		this.nombre = nombre;
 		this.plano = plano;
 		this.jugador = jugador;
 	}
 
-	public final List<Posicion> obtenerPosiciones() {
+	public List<Posicion> obtenerPosiciones(final ObservadorDeUnidades observadorDeUnidades) {
 		ConstructorDeNiveles constructor = new ConstructorDeNiveles();
-		return constructor.construirPosiciones(this.plano, crearGuerrero());
+		return constructor.construirPosiciones(this.plano, observadorDeUnidades, crearGuerrero());
 	}
 
 	public final String obtenerNombre() {
 		return this.nombre;
 	}
 
 	private Guerrero crearGuerrero() {
-		return new Guerrero(this.jugador.obtenerNombreDelGuerrero(), this.jugador);
+		Guerrero resultado = new Guerrero(this.jugador.obtenerNombreDelGuerrero(), this.jugador);
+		return resultado;
 	}
 
 }

src/main/java/jwarrior/juego/puntuacion/ControlDePuntaje.java

@@ -0,0 +1,40 @@
+package jwarrior.juego.puntuacion;
+
+import jwarrior.juego.Juego;
+import jwarrior.observadores.ObservadorDePartida;
+import jwarrior.observadores.ObservadorDeUnidades;
+import jwarrior.piezas.unidades.Unidad;
+import jwarrior.referencias.Espacio;
+
+public class ControlDePuntaje implements ObservadorDePartida, ObservadorDeUnidades {
+
+	private Puntaje puntaje = new Puntaje();
+
+	@Override
+	public void notificarMuerte(final Unidad unidad) {
+		if (unidad.obtenerTipo() != Espacio.GUERRERO) {
+			Integer puntos = unidad.obtenerSaludMaxima();
+			this.puntaje.computar(puntos);
+		}
+	}
+
+	@Override
+	public void notificarTurno(final Juego partida) {
+		Integer puntos = -1;
+		this.puntaje.computar(puntos);
+	}
+
+	@Override
+	public void notificarFinal(final Juego partida) {
+		Integer puntos = partida.obtenerGuerrero().obtenerSaludActual() / 2;
+		this.puntaje.computar(puntos);
+	}
+
+	public Integer obtenerPuntaje() {
+		return this.puntaje.obtenerPuntaje();
+	}
+
+	public void establecerPuntaje(final Puntaje puntaje) {
+		this.puntaje = puntaje;
+	}
+}

src/main/java/jwarrior/juego/puntuacion/Puntaje.java

@@ -1,7 +1,5 @@
 package jwarrior.juego.puntuacion;
 
-import jwarrior.piezas.unidades.Unidad;
-import jwarrior.ui.InterfazDeUsuario;
 
 public class Puntaje {
 
@@ -13,28 +11,7 @@ public Integer obtenerPuntaje() {
 		return this.puntos;
 	}
 
-	// TODO: implementar una CalculadoraDePuntaje
-	public void notificarTurno() {
-		Integer puntos = 1;
-		mensaje("-" + puntos);
-		this.puntos -= puntos;
-	}
-
-	// TODO: implementar una CalculadoraDePuntaje
-	public void notificarMuerteEnemigo(final Unidad unidad) {
-		Integer puntos = unidad.obtenerSaludMaxima();
-		mensaje("+" + puntos);
+	public void computar(final Integer puntos) {
 		this.puntos += puntos;
 	}
-
-	// TODO: implementar una CalculadoraDePuntaje
-	public void notificarFinalizacionPartida(final Unidad guerrero) {
-		Integer puntos = guerrero.obtenerSaludActual() / 2;
-		mensaje("+" + puntos);
-		this.puntos += puntos;
-	}
-
-	protected void mensaje(final String mensaje) {
-		InterfazDeUsuario.obtenerInstancia().mensaje(mensaje);
-	}
 }

src/main/java/jwarrior/observadores/ObservadorDePartida.java

@@ -0,0 +1,11 @@
+package jwarrior.observadores;
+
+import jwarrior.juego.Juego;
+
+public interface ObservadorDePartida {
+
+	void notificarTurno(Juego partida);
+
+	void notificarFinal(Juego partida);
+
+}

src/main/java/jwarrior/observadores/ObservadorDeUnidades.java

@@ -0,0 +1,9 @@
+package jwarrior.observadores;
+
+import jwarrior.piezas.unidades.Unidad;
+
+public interface ObservadorDeUnidades {
+
+	void notificarMuerte(Unidad unidad);
+
+}

src/main/java/jwarrior/piezas/unidades/Guerrero.java

@@ -4,7 +4,6 @@
 import jwarrior.comandos.Comando;
 import jwarrior.juego.Jugador;
 
-
 public class Guerrero extends Unidad {
 
 	private static final int FUERZA_MAXIMA = 5;

src/main/java/jwarrior/piezas/unidades/Unidad.java

@@ -1,8 +1,11 @@
 package jwarrior.piezas.unidades;
 
+import java.util.LinkedList;
+import java.util.List;
+
 import jwarrior.comandos.Comando;
 import jwarrior.fabricas.LocalizadorDeFabricas;
-import jwarrior.juego.Juego;
+import jwarrior.observadores.ObservadorDeUnidades;
 import jwarrior.piezas.Pieza;
 import jwarrior.referencias.Direccion;
 import jwarrior.referencias.Espacio;
@@ -17,11 +20,17 @@ public abstract class Unidad extends Pieza {
 	private Integer saludActual = 0;
 	private Integer fuerzaMaxima = 0;
 
+	private List<ObservadorDeUnidades> observadores = new LinkedList<ObservadorDeUnidades>();
+
 	protected Unidad(final String nombre, final Espacio tipo) {
 		super(tipo);
 		this.nombre = nombre;
 	}
 
+	public void agregarObservador(ObservadorDeUnidades observadorDeUnidades) {
+		this.observadores .add(observadorDeUnidades);
+	}
+
 	// Puntos de extensión
 
 	public Comando proximoComando() {
@@ -84,10 +93,9 @@ public final void curar(final Integer cantidadARestaurar) {
 	protected void morir() {
 		this.decir("muere");
 		this.obtenerPosicion().liberar();
-		// FIXME: Deberia haber algo como una CentralDeNotificaciones, para
-		// evitar que se acople la unidad al juego.
-		if (Juego.obtenerInstancia() != null) { // FIXME: hack para testear.
-			Juego.obtenerInstancia().notificarMuerte(this);
+
+		for (ObservadorDeUnidades observador : this.observadores) {
+			observador.notificarMuerte(this);
 		}
 	}