Skip to content

Commit 7f9fe24

Browse files
committed
Issue #13: Refactor sobre la creacion de Niveles. Ahora todo está en la superclase Nivel
1 parent f946c08 commit 7f9fe24

4 files changed

Lines changed: 12 additions & 36 deletions

File tree

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

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,20 +9,25 @@
99
public abstract class Nivel {
1010

1111
private final String nombre;
12+
private final String plano;
1213
private Jugador jugador;
1314

14-
public Nivel(final String nombre, final Jugador jugador) {
15+
public Nivel(final String nombre, final String plano, final Jugador jugador) {
1516
this.nombre = nombre;
17+
this.plano = plano;
1618
this.jugador = jugador;
1719
}
1820

19-
public abstract List<Posicion> obtenerPosiciones();
21+
public final List<Posicion> obtenerPosiciones() {
22+
ConstructorDeNiveles constructor = new ConstructorDeNiveles();
23+
return constructor.construirPosiciones(this.plano, crearGuerrero());
24+
}
2025

21-
public String obtenerNombre() {
26+
public final String obtenerNombre() {
2227
return this.nombre;
2328
}
2429

25-
protected Guerrero crearGuerrero() {
30+
private Guerrero crearGuerrero() {
2631
return new Guerrero(this.jugador.obtenerNombreDelGuerrero(), this.jugador);
2732
}
2833

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,10 @@
11
package jwarrior.juego.niveles;
22

3-
import java.util.List;
4-
53
import jwarrior.juego.Jugador;
6-
import jwarrior.posiciones.Posicion;
74

85
public class Nivel01 extends Nivel {
96

107
public Nivel01(final Jugador jugador) {
11-
super("Nivel 01", jugador);
12-
}
13-
14-
@Override
15-
public List<Posicion> obtenerPosiciones() {
16-
ConstructorDeNiveles constructor = new ConstructorDeNiveles();
17-
return constructor.construirPosiciones("PG____S", crearGuerrero());
8+
super("Nivel 01", "PG____S", jugador);
189
}
1910
}
Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,10 @@
11
package jwarrior.juego.niveles;
22

3-
import java.util.List;
4-
53
import jwarrior.juego.Jugador;
6-
import jwarrior.posiciones.Posicion;
74

85
public class Nivel02 extends Nivel {
96

107
public Nivel02(final Jugador jugador) {
11-
super("Nivel 02", jugador);
12-
}
13-
14-
@Override
15-
public List<Posicion> obtenerPosiciones() {
16-
ConstructorDeNiveles constructor = new ConstructorDeNiveles();
17-
return constructor.construirPosiciones("PG_oooS", crearGuerrero());
8+
super("Nivel 02", "PG_oooS", jugador);
189
}
19-
2010
}
Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,10 @@
11
package jwarrior.juego.niveles;
22

3-
import java.util.List;
4-
53
import jwarrior.juego.Jugador;
6-
import jwarrior.posiciones.Posicion;
74

85
public class Nivel03 extends Nivel {
96

107
public Nivel03(final Jugador jugador) {
11-
super("Nivel 03", jugador);
12-
}
13-
14-
@Override
15-
public List<Posicion> obtenerPosiciones() {
16-
ConstructorDeNiveles constructor = new ConstructorDeNiveles();
17-
return constructor.construirPosiciones("PG_o_o_oo_oS", crearGuerrero());
8+
super("Nivel 03", "PG_o_o_oo_oS", jugador);
189
}
19-
2010
}

0 commit comments

Comments
 (0)