could anybody help please

Question about the task

Under discussion

where is the bug in my code ?

Moving the snake is very simple: in a cell next to the head, a new head is created, and the last element (tail) of the snake is removed.

In the Snake class, implement a createNewHead() method that creates and returns a new GameObject. The new GameObject must be created next to the element that currently contains the snake head (element with index 0 in snakeParts).

The new GameObject is created in the direction of the snake's movement (direction field).

For example:

- if the snake is moving to the left, new GameObject(headX-1, headY);
- if the snake is moving down, new GameObject(headX, headY + 1)

Here, headX is the x coordinate of the snake's head, and headY is the y coordinate of the snake's head.

Note: coordinates can be negative and/or be beyond the boundaries of the playing field. This is okay for now. We'll fix it later.

Create and implement a removeTail() method in the Snake class.

Requirements:

The Snake class must have a public GameObject createNewHead() method.
The createNewHead() method must be implemented as outlined in the task conditions.
The Snake class must have a public void removeTail() method.
The removeTail() method must remove the last item in the snakeParts list.

// organize the classes in the Games of the codegym package com.codegym.games.snake; // import the engine class import com.codegym.engine.cell.*; /* * this exercise contian the code of the Snake Game in codegym *SnakeGame Calss inherit from Gam calss *@Autor Mohamad Alkasem */ public class SnakeGame extends Game { /* *@param width: width of the screen *@ param height: hieght of the screen */ public static final int WIDTH = 15; public static final int HEIGHT = 15; // private filed private Snake snake; private int turnDelay; /* * this Override Methode from super class Game * WIDTH and HEIGHT will be initialized * @param WIDTH * @param HEIGHT * @param return: would return nothing. void Methode return nothing */ @Override public void initialize(){ //Game gameSet = new Game(); // call and set setScreenSize with WIDTH and HEIGHT size of the screen setScreenSize(WIDTH, HEIGHT); this.createGame(); } /* * createGame: method for action * drawScene: for drawing the secreen */ private void createGame(){ Snake snake = new Snake(WIDTH / 2, HEIGHT / 2); this.snake = snake; snake.draw(this); this.drawScene(); turnDelay = 300; setTurnTimer(turnDelay); //Apple apple = new Apple(7, 7); // apple.draw(this); } private void drawScene() { for (int x = 0; x < WIDTH; x++){ for(int y = 0; y < HEIGHT; y++){ setCellColor(x, y, Color.PURPLE); } } snake.draw(this); //setCellColor(x, y, Color.DARKSEAGREEN); } @Override public void onTurn(int t){ snake.move(); drawScene(); } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Alejandro

Level 5 , Madrid

13 June 2021, 17:22

1) line 104 of the snake class should be inside the createNewHead method at the beginning. 2) the headx and heady variables must be initialized with the coordinates of the snake's head. for example headx = snakeParts.get (0) .x; 3) line 121 must be on line 119.

Thomas

Level 41 , Bayreuth, Germany

13 June 2021, 16:57

The values for headX and headY don't come out of nowhere. You should think about where they are saved and pull the x/ y values of the head from there. Just declaring new fields is wishful thinking (and these are initialized with zero anyways). So that's the first thing I'd do and, as said, the second is trying to remember where the snake parts are stored and from there take the first element, the head ;) btw, the last elseif in your createNewHead method is empty so you can remove it entirely (or you make and else clause out of it and move the last return statement inside of it's block).