What is the problem with the draw method?

Question about the task

Under discussion

I am struggling to find the bugs inside the method draw.Can anybody help me? Is the logic behind the implementation wrong or is it just a little bug that i do not know how to find? Thanks in advance!

Now you need to combine the work of the createNewHead() and removeTail() methods. The move() method is a good place to do this.

Add a check to see whether the snake has gone beyond the boundaries of the playing field. If the new head is beyond the boundaries of the playing field, set the snake's state to dead.

After moving the snake, you need to clear from the playing field any snake segments that no longer exist. To do this, the drawScene() method should not only change the color of cells, but also assign an empty string to those cells.

Requirements:

The createNewHead() method must be called in the Snake class's move() method.
In the move() method, the result of the call to the createNewHead() method must be added to snakeParts at position 0.
In the Snake class's move() method, the removeTail() method must be called after the new snake head is created.
In the move() method, if the createNewHead() method returns an element whose coordinates are outside the playing field, you need to set the isAlive field to false and do nothing else.
In the SnakeGame class's drawScene() method, you must replace the call to the setCellColor(int, int, Color) method with a call to the setCellValueEx(int, int, Color, String) method with arguments (x, y, <color> (for example, Color.DARKSEAGREEN), "").

package com.codegym.games.snake; import com.codegym.engine.cell.*; public class Apple extends GameObject { int x; int y; public boolean isAlive = true; private static final String APPLE_SIGN = "\uD83C\uDF4E"; public Apple(int x,int y){ super(x,y); this.x = x; this.y = y; } public void draw(Game game){ game.setCellValueEx(x,y,Color.NONE,APPLE_SIGN, Color.GREEN,75); } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

26 December 2019, 14:41

When x or y are equal to 0 the snake head is still within the boundaries. You should be checking for this before adding the new piece to the array, and if it is outside of the boundaries, end the method. Also, while not technically wrong, you don't need the 'this' keyword when there is no conflict of variable names. It is not needed at lines 27 and 43 of the snake class.