#19: Se realizan ajustes en normas de checkstyle para obtener informes representativos

Author: delucas

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

@@ -29,8 +29,7 @@ public final String obtenerNombre() {
 	}
 
 	private Guerrero crearGuerrero() {
-		Guerrero resultado = new Guerrero(this.jugador.obtenerNombreDelGuerrero(), this.jugador);
-		return resultado;
+		return new Guerrero(this.jugador.obtenerNombreDelGuerrero(), this.jugador);
 	}
 
 }

src/main/java/jwarrior/piezas/Pieza.java

@@ -5,7 +5,7 @@
 
 public abstract class Pieza {
 
-	protected Espacio tipo;
+	private final Espacio tipo;
 	private Posicion posicion;
 
 	protected Pieza(final Espacio tipo) {

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

@@ -27,8 +27,8 @@ protected Unidad(final String nombre, final Espacio tipo) {
 		this.nombre = nombre;
 	}
 
-	public void agregarObservador(ObservadorDeUnidades observadorDeUnidades) {
-		this.observadores .add(observadorDeUnidades);
+	public void agregarObservador(final ObservadorDeUnidades observadorDeUnidades) {
+		this.observadores.add(observadorDeUnidades);
 	}
 
 	// Puntos de extensión

src/main/java/jwarrior/posiciones/Posicion.java

@@ -11,7 +11,7 @@
 public class Posicion {
 
 	private Mapa mapa;
-	protected Pieza pieza;
+	private Pieza pieza;
 
 	public Posicion() {
 		this(new PiezaNula());

src/test/resources/sun_checks.xml

@@ -150,10 +150,10 @@
 <!--        <module name="DoubleCheckedLocking"/> -->
         <module name="EmptyStatement"/>
         <module name="EqualsHashCode"/>
-		<module name="HiddenField">
-			<property name="ignoreSetter" value="true"/>
-			<property name="ignoreConstructorParameter" value="true"/>
-		</module>
+<!-- 		<module name="HiddenField"> -->
+<!-- 			<property name="ignoreSetter" value="true"/> -->
+<!-- 			<property name="ignoreConstructorParameter" value="true"/> -->
+<!-- 		</module> -->
         <module name="IllegalInstantiation"/>
         <module name="InnerAssignment"/>
         <module name="MagicNumber"/>