I copied and pasted but do not get it to work. I have some specific questions about how certain code works

Question about the task

2048 (Part 6/18)

Under discussion

There are a couple of things I do not understand 1) How to add a color to the specific field? 2) should you use if statements or a a switch statement 3) why does it not accept the empty string "" in the setCellValueEx ? 4) I do not understand this part of the code: private void drawScene(){ for (int i = 0;i < SIDE;i++){ for (int y = 0;y < SIDE;y++){ setCellColoredNumber(i,y,gameField[i][y]);// what exactly are we doing on this line of code? } } } Can anybody please explain the above to me. It will be very much appreciated!!

It's already much better. But I think we can do more. Let's paint cells with the same values the same colors.

To do this, write a setCellColoredNumber method that will take as arguments a cell's coordinates and value, and:

- calculate the cell color (using the getColorByValue method, which is described below),
- display the cell value and color on the game board.

To calculate the color, we implement the getColorByValue helper method, which will return a color based on the cell value passed to method. For example, if we pass 2 to the method, then it returns Color.BLUE, etc. Cells with different values should have different colors; cells with the same values should have the same color. The numbers that can exist in the gameField matrix are: 0, 2, 4, 8, 16, 32, 64, 128, 256, 512, 1024, 2048.

To display the cell value and color with coordinates (x, y) on the game board, the Game class's special setCellValueEx(int x, int y, Color cellColor, String value) method comes to our aid.
Use it to change the cell color and value at the same time.

Don't display the value 0 on the game board: instead, we simply see an empty cell. Therefore, for cells with the value 0, simply pass an empty string to the setCellValueEx() method.

And the final touch: in the drawScene() method, replace the call to the setCellColor() method with the setCellColoredNumber() method.

Notes:
1. For clarity, empty cells will be denoted by zeros in the examples.
2. The matrix coordinates must match the displayed coordinates.

Example:

Matrix:  {

{  2,    4,    8,  16},

{ 32,   64,  128, 256},

{512, 1024, 2048,   0},

{  2,    4,    8,  16}

}

Corresponding representation:

To do this, when setting a value on the game board, invert the matrix coordinates. For example, to set the value of the cell with coordinates (x, y), use the matrix value gameField[y][x].

To better understand what we're saying, draw the matrix — first using gameField[y][x], and then gameField[x][y].

Requirements:

A private Color getColorByValue(int value) method must be created in the Game2048 class. It must return a cell color based on its value.
A private void setCellColoredNumber(int x, int y, int value) method must be created in the Game2048 class.
The setCellColoredNumber(int, int, int) method must call the getColorByValue(int) method.
The setCellColoredNumber(int, int, int) method must call the setCellValueEx(int, int, Color, String) method.
The setCellColoredNumber(int, int, int) method must change the color of a tile depending on its value.
The setCellColoredNumber(int, int, int) method must not display a tile's value if it is 0.
In the drawScene() method, the call to the setCellColor(int, int, Color) method must be replaced by setCellColoredNumber(int, int, int), so that the values and colors of the tiles on the game board correspond to the values in the gameField matrix.

package com.codegym.games.game2048; import com.codegym.engine.cell.*; public class Game2048 extends Game { private static final int SIDE = 4; private int[][] gameField = new int [SIDE][SIDE]; int value; @Override public void initialize(){ setScreenSize(SIDE, SIDE); createGame(); drawScene(); } private void createGame(){ createNewNumber(); createNewNumber(); } private void drawScene(){ for( int i = 0; i < SIDE; i++) for(int j = 0; j < SIDE; j++) setCellColoredNumber(i, j, gameField[i][j]);//copied this from someone lese but I do not understand this bit } private void createNewNumber(){ int x = getRandomNumber(SIDE);//get a random number and assign its value to x int y = getRandomNumber(SIDE);// same but for y if(getRandomNumber(10) == 9) value =4;//get a random number and if is equal to nine assign value to 4 else value = 2;//otherwise you assign 2 to the value if (gameField[x][y] == 2 || gameField[x][y] == 4) createNewNumber();//how do we assign values to one of the fields? else gameField[x][y] = value;//if the cell has no value yet assign it value 2 or 4 depending on outcome of randomNumber(10) //as this method createNewNumber is only called twice only two cells will get a value } //private Color getColorByValue(int value){ //returns a cell with a color based on its value/*private Color getColorByValue(int value){ //return a cell color based on value /*switch(value){ case 0: return Color.WHITE; case 2: return Color.BLUE; case 4: return Color.RED; case 8: return Color.GREEN; case 16: return Color.CYAN; case 32: return Color.GRAY; case 64: return Color.MAGENTA; case 128: return Color.ORANGE; case 256: return Color.PINK; case 512: return Color.YELLOW; case 1024: return Color.PURPLE; case 2048: return Color.BROWN; default: return Color.WHITE; }*/ private Color getColorByValue(int value){//copied this also from someone else if (value == 0){ return Color.WHITE; } else if (value == 2){ return Color.ALICEBLUE; } else if (value == 4){ return Color.ANTIQUEWHITE; } else if (value == 8){ return Color.AQUA; } else if (value == 16){ return Color.AQUAMARINE; } else if (value == 32){ return Color.AZURE; } else if (value == 64){ return Color.BEIGE; } else if (value == 128){ return Color.BISQUE; } else if (value == 256){ return Color.BLACK; } else if (value == 512){ return Color.BLANCHEDALMOND; } else if (value == 1024){ return Color.BLUE; } else if (value == 2048) { return Color.BLUEVIOLET; } else return Color.WHITE; } private void setCellColoredNumber(int x, int y, int value){ if (value != 0) { setCellValueEx(y, x, getColorByValue(value), Integer.toString(value)); } else { setCellValueEx(y, x, getColorByValue(value), "");//why does this not work here? copied it from someone else } } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Pawel

Level 22 , Austin, TX, USA

4 March 2020, 21:31

(2) doesn't meter, it's up to you (3) The problem is not empty string but third argument, method setCellValueEx(int x, int y, Color cellColor, String value) as a third argument accept object, and you have there method getColorByValue(value) which return just color. try Color color = getColorByValue(value) and pass it there (1)(4) Double loop go though all cells on the gameFiled and call on them method setCellColoredNumber(i,y,gameField[i][y]) --> i equals first coordinate in matrix, y equals second coordinate in matrix and gameField[i][y] is value of this cell. How to fix setCellColoredNumber check point (3)