ueb02.1

src/fr/enssat/BoulderDash/controllers/BoulderAndDiamondController.java

@@ -7,7 +7,7 @@ import fr.enssat.BoulderDash.helpers.AudioLoadHelper;
 
 /**
  * ElementPositionUpdateHelper
- *
+ * <p>
  * Updates position of all elements displayed on the map, according to their
  * next potential position. Each object has a weight, which is used to compare
  * their power to destroy in the food chain. Sorry for that Darwinism.
@@ -79,6 +79,25 @@ public class BoulderAndDiamondController implements Runnable {
         }
     }
 
+    private String getSpriteNameLeft(int x, int y) {
+        DisplayableElementModel elementLeft = this.levelModel.getGroundLevelModel()[x - 1][y];
+        return elementLeft.getSpriteName();
+    }
+
+    private String getSpriteNameRight(int x, int y) {
+        DisplayableElementModel elementRight = this.levelModel.getGroundLevelModel()[x + 1][y];
+        return elementRight.getSpriteName();
+    }
+
+    private String getSpriteNameBelow(int x, int y) {
+        DisplayableElementModel elementBelow = getElementBelow(x, y);
+        return elementBelow.getSpriteName();
+    }
+
+    private DisplayableElementModel getElementBelow(int x, int y) {
+        return this.levelModel.getGroundLevelModel()[x][y + 1];
+    }
+
     /**
      * Expand the wall at left & right
      *
@@ -86,17 +105,12 @@ public class BoulderAndDiamondController implements Runnable {
      * @param y Vertical position
      */
     private String expandWall(int x, int y) {
-		DisplayableElementModel elementLeft  = this.levelModel.getGroundLevelModel()[x - 1][y];
-        DisplayableElementModel elementRight = this.levelModel.getGroundLevelModel()[x + 1][y];
-        String spriteNameLeft  = elementLeft.getSpriteName();
-		String spriteNameRight = elementRight.getSpriteName();
-		
         String way = "";
-		if(spriteNameLeft == "black"){
+        if (getSpriteNameLeft(x, y) == "black") {
             this.levelModel.expandThisWallToLeft(x, y);
             way = "left";
         }
-		if(spriteNameRight == "black"){
+        if (getSpriteNameRight(x, y) == "black") {
             this.levelModel.expandThisWallToRight(x, y);
             way = "right";
         }
@@ -110,16 +124,9 @@ public class BoulderAndDiamondController implements Runnable {
      * @param y Vertical position
      */
     private void manageFall(int x, int y) {
-		// Get informed about Rockford surroundings
-        DisplayableElementModel elementBelow = this.levelModel.getGroundLevelModel()[x][y + 1];
-        DisplayableElementModel elementLeft  = this.levelModel.getGroundLevelModel()[x - 1][y];
-        DisplayableElementModel elementRight = this.levelModel.getGroundLevelModel()[x + 1][y];
-
-        String spriteNameBelow = elementBelow.getSpriteName();
-		String spriteNameLeft  = elementLeft.getSpriteName();
-		String spriteNameRight = elementRight.getSpriteName();
 
         // Then, process in case of the surrounding
+        String spriteNameBelow = getSpriteNameBelow(x, y);
         if (spriteNameBelow == "black") {
             this.levelModel.makeThisDisplayableElementFall(x, y);
         } else if (spriteNameBelow == "boulder") {
@@ -151,11 +158,11 @@ public class BoulderAndDiamondController implements Runnable {
                     this.levelModel.deleteThisBoulder(x, y);
                 }
             }
-		} else if (elementBelow.isDestructible() && spriteNameBelow != "dirt" && this.levelModel.getGroundLevelModel()[x][y].isFalling()) {
+        } else if (getElementBelow(x, y).isDestructible() && spriteNameBelow != "dirt" && this.levelModel.getGroundLevelModel()[x][y].isFalling()) {
             this.levelModel.exploseThisBrickWall(x, y);
-		} else if (spriteNameLeft == "rockford" && this.levelModel.getRockford().isRunningRight() && this.levelModel.getGroundLevelModel()[x + 1][y].getSpriteName() == "black") {
+        } else if (getSpriteNameLeft(x, y) == "rockford" && this.levelModel.getRockford().isRunningRight() && this.levelModel.getGroundLevelModel()[x + 1][y].getSpriteName() == "black") {
             this.levelModel.moveThisBoulderToRight(x, y);
-		} else if (spriteNameRight == "rockford" && this.levelModel.getRockford().isRunningLeft() && this.levelModel.getGroundLevelModel()[x - 1][y].getSpriteName() == "black") {
+        } else if (getSpriteNameRight(x, y) == "rockford" && this.levelModel.getRockford().isRunningLeft() && this.levelModel.getGroundLevelModel()[x - 1][y].getSpriteName() == "black") {
             this.levelModel.moveThisBoulderToLeft(x, y);
         } else {
             this.levelModel.getGroundLevelModel()[x][y].setFalling(false);

src/fr/enssat/BoulderDash/models/RockfordModel.java

@@ -114,16 +114,22 @@ public class RockfordModel extends DisplayableElementModel {
 		}
 	}
 
-	/**
+	private void resetMovement(){
-	 * Stops the Rockford movement
-	 */
-	public void startStaying() {
 		isCollisionDone = false;
-		isStaying = true;
+		isStaying = false;
 		isRunningLeft = false;
 		isRunningRight = false;
 		isRunningUp = false;
 		isRunningDown = false;
+	}
+
+	/**
+	 * Stops the Rockford movement
+	 */
+	public void startStaying() {
+		resetMovement();
+		isStaying = true;
+
 		previousTime = 0;
 		currentFrame = 0;
 	}
@@ -132,12 +138,9 @@ public class RockfordModel extends DisplayableElementModel {
 	 * Starts moving Rockford to the left
 	 */
 	public void startRunningLeft() {
-		isCollisionDone = false;
+		resetMovement();
-		isStaying = false;
 		isRunningLeft = true;
-		isRunningRight = false;
+
-		isRunningUp = false;
-		isRunningDown = false;
 		previousTime = 0;
 	}
 
@@ -145,12 +148,9 @@ public class RockfordModel extends DisplayableElementModel {
 	 * Starts moving Rockford to the right
 	 */
 	public void startRunningRight() {
-		isCollisionDone = false;
+		resetMovement();
-		isStaying = false;
-		isRunningLeft = false;
 		isRunningRight = true;
-		isRunningUp = false;
+
-		isRunningDown = false;
 		previousTime = 0;
 	}
 
@@ -158,12 +158,9 @@ public class RockfordModel extends DisplayableElementModel {
 	 * Rockford running up
 	 */
 	public void startRunningUp() {
-		isCollisionDone = false;
+		resetMovement();
-		isStaying = false;
-		isRunningLeft = false;
-		isRunningRight = false;
 		isRunningUp = true;
-		isRunningDown = false;
+
 		previousTime = 0;
 	}
 
@@ -171,12 +168,9 @@ public class RockfordModel extends DisplayableElementModel {
 	 * Rockford running down
 	 */
 	public void startRunningDown() {
-		isCollisionDone = false;
+		resetMovement();
-		isStaying = false;
-		isRunningLeft = false;
-		isRunningRight = false;
-		isRunningUp = false;
 		isRunningDown = true;
+
 		previousTime = 0;
 	}
 

src/fr/enssat/BoulderDash/views/GroundView.java

@@ -43,13 +43,12 @@ public abstract class GroundView extends JPanel implements Observer {
 	 */
 	public void drawTerrain(int width, int height, Graphics g) {
 		// Draw items
-		if (this.levelModel.getMode() == "game") {
 		for (int x = 0; x < width; x++) {
 			for (int y = 0; y < height; y++) {
 				g.drawImage(this.levelModel.getImage(x, y), (x * 16), (y * 16), this);
 			}
 		}
-
+		if (this.levelModel.getMode() == "game") {
 			if(!this.levelModel.isGameRunning()) {
 				if(!this.levelModel.getRockford().getHasExplosed()) {
 					this.displayWin();
@@ -58,11 +57,6 @@ public abstract class GroundView extends JPanel implements Observer {
 				}
 			}
 		} else {
-			for (int x = 0; x < width; x++) {
-				for (int y = 0; y < height; y++) {
-					g.drawImage(this.levelModel.getImage(x, y), (x * 16), (y * 16), this);
-				}
-			}
 			if (this.levelModel.getShowCursor()) {
 	            g.drawImage(
 	                    this.levelModel.getCursorImage(),