Alexandre, you're not alone! I got the same problem!

Question about the task

Resolved

The codes work perfectly with the requiremnet but failed the validator. It's always painful to guess what the validator wants, the comments from it are totally not useful.

The time has come to implement the model's move() method, which is responsible for movement. But first, let's implement some helper methods. Add the following methods to the Model class:
15.1. boolean checkWallCollision(CollisionObject gameObject, Direction direction). This method checks for collisions with a wall. It should return true if the gameObject will collide with a wall by moving in the passed direction; otherwise, it should return false. To determine if there is a collision, use the game object's isCollision() method.
15.2. boolean checkBoxCollisionAndMoveIfAvaliable(Direction direction). This method checks for collisions with boxes. The method should:
15.2.1. Return true if the player can't move in the specified direction (i.e. the cell is occupied by a box with a wall behind it or by a box with another box behind it).
15.2.2. Return false if the player can move in the specified direction (i.e. the cell is occupied by an empty cell or by a storage location or by a box with an empty cell or a storage location behind it). In this case, if a movable box is in the way, then you need to move it to new coordinates. Note that all the objects move by a fixed distance equal to BOARD_CELL_SIZE regardless of the dimensions used for drawing.

Hint: check for collisions using the game objects' isCollision() methods and the model's checkWallCollision() method.

15.3. void checkCompletion(). This method must check whether the level has been completed (whether all the storage locations have boxes on them, i.e. their coordinates match). If this condition is true, then let the event listener know that the current level is complete.
15.4. void move(Direction direction). The method should:
15.4.1. Check for a wall collision (use the checkWallCollision() method). If there is a collision, then exit the method.
15.4.2. Check for a box collision (use the checkBoxCollisionAndMoveIfAvaliable() method). If there is a collision, then exit the method.
15.4.3. Move the player in the passed direction.
15.4.4. Check whether the level is completed.

Run the program and be sure that the player can move itself and boxes, that walls prevent the player from moving, and that when you move all the boxes onto storage locations that a message about completing the level is displayed.

Requirements:

Implement the boolean checkWallCollision(CollisionObject gameObject, Direction direction) method in the Model class.
Implement the boolean checkBoxCollisionAndMoveIfAvailable(Direction direction) method in the Model class.
Implement the void checkCompletion() method in the Model class.
Implement the void move(Direction direction) method in the Model class.

package com.codegym.task.task34.task3410.controller; import com.codegym.task.task34.task3410.model.Direction; import com.codegym.task.task34.task3410.model.GameObjects; import com.codegym.task.task34.task3410.model.Model; import com.codegym.task.task34.task3410.view.View; public class Controller implements EventListener { private View view; private Model model; public Controller() { model = new Model(); model.restart(); model.setEventListener(this); view = new View(this); view.init(); view.setEventListener(this); } public static void main(String[] args) { Controller controller = new Controller(); } @Override public void move(Direction direction) { model.move(direction); view.update(); } @Override public void restart() { model.restart(); view.update(); } @Override public void startNextLevel() { model.startNextLevel(); view.update(); } @Override public void levelCompleted(int level) { view.completed(level); } public GameObjects getGameObjects() { return model.getGameObjects(); } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Elvis Lee

Level 41 , Aurora, United States

3 March 2020, 15:35

Hi Seb, you're great! It's "Avaliable", haha!

Seb

Level 41 , Crefeld, Germany

3 March 2020, 17:33

:-)

Seb

Level 41 , Crefeld, Germany

3 March 2020, 09:34

solution

I haven't had a deeper look into your code yet, but one thing that may create the issue is the spelling mistake in the method name given in the requirements. It says "checkBoxCollisionAndMoveIfAvaliable". The last part is spelled incorrectly Avaliable instead of Available. The i and l are twisted. Maybe try to rename your method from checkBoxCollisionAndMoveIfAvailable to checkBoxCollisionAndMoveIfAvaliable...