what is wrong

Question about the task

Under discussion

I have received 3 mistakes and I can't figure out what is wrong: ---> The drawScene() method must be called in the createGame() method. ---> The createGame() method must be called in the initialize() method. ---> n the drawScene() method, call the setCellColor(int, int, Color) method on each cell of the playing field with the following arguments: x and y coordinates and any color.

Let's color the playing field. You can do this and every other game action in the initialize method. But cramming everything into one method is bad practice.

So, we'll initially create a couple of methods, each of which will be responsible for a specific job: the drawScene method is for drawing the screen, and the createGame method is for the actions required to create the game. Accordingly, in the initialize() method, we need to call the createGame() method, which will in turn call the drawScene method.

To paint the cells of the playing field a specific color, you need to go through all the cells (using loops) and call the setCellColor method on each of them.

Hint:
for (int x = 0; x < WIDTH; x++) { for (int y = 0; y < HEIGHT; y++) { setCellColor(x, y, Color); } }

Requirements:

The SnakeGame class must have a private void createGame() method.
The SnakeGame class must have a private void drawScene() method.
The drawScene() method must be called in the createGame() method.
The createGame() method must be called in the initialize() method.
In the initialize() method, the createGame() method is called after the setScreenSize(int, int) method.
In the drawScene() method, call the setCellColor(int, int, Color) method on each cell of the playing field with the following arguments: x and y coordinates and any color. For example, Color.DARKSEAGREEN.

package com.codegym.games.snake; import com.codegym.engine.cell.*; public class SnakeGame extends Game { public static final int WIDTH = 15; public static final int HEIGHT = 15; public void setScreenSize(int width, int height) { } private void drawScene(){ for (int x = 0; x < WIDTH; x++) { for (int y = 0; y < HEIGHT; y++) { setCellColor(x, y, Color.DARKSEAGREEN); } } } private void createGame(){ drawScene(); } public void initialize() { setScreenSize(WIDTH, HEIGHT); createGame(); } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

13 October 2019, 04:12

You need to fix your loops. The outer loop is the loop that defines the height, while the inner loop defines the width.

for (int y = 0; y < HEIGHT; y++) {
   for (int x = 0; x < WIDTH; x++) {
      setCellColor(x, y, Color.DARKSEAGREEN);
   }
}

. I know that this is level "5", but if you don't understand how multidimensional arrays work in regards to accessing the indices contained within, you are going to have a very hard time doing these games.

Mike

Level 9 , Lublin, Poland

13 October 2019, 08:48

Thank you for your answer. I have changed the loop like in your example but still there are the same errors, e.g. "The drawScene() method must be called in the createGame() method." Can you advice, please?

private void createGame(){
     drawScene();
  }

Guadalupe Gagnon

Level 37 , Tampa, United States

13 October 2019, 21:05

remove lines 10-12. The setScreenSize method is part of the Game class and over riding it in your class removes its functionality.