Same as everyone else. It seems to work but won't pass validation. The key bit of code is in openTile(int x, int y).

Question about the task

Resolved

private void openTile(int x, int y) { if (gameField[x][y].isMine) setCellValue(x,y,MINE); else setCellNumber(x,y,gameField[x][y].countMineNeighbors); gameField[x][y].isOpen = true; setCellColor(x,y,Color.GREEN); }

At this stage, we'll deal with the logic to reveal cells. Let's create a corresponding method and a variable that will store the cell state (revealed or hidden).

When revealing a cell, the method will:

- draw a mine in the cell if it contains a mine;
- display the number of mined neighbors if the cell does not contain a mine;
- mark the cell as revealed in the model;
- change the cell color.

For convenience, we suggest adding the mine symbol to a separate constant. When the a cell be revealed? Correct! When you click on it with the left mouse button. To handle this click event, the engine has a special method: onMouseLeftClick(int, int).

Let's override it in our class. It will call the method that reveals a cell.

And, of course, run the program and enjoy the results of your work.

Requirements:

The MinesweeperGame class must have a private static final String MINE field that is initialized when it is declared. For example, you can use the UTF-16 symbol for a bomb: "\uD83D\uDCA3".
The MinesweeperGame class must have a private void openTile(int x, int y) method.
The openTile(int, int) method must draw MINE if the gameObject at the passed coordinates is a mine. Use the setCellValue(int, int, String) method.
The openTile(int, int) method must draw the number of adjacent mines if the gameObject at the passed coordinates is not a mine. Use the setCellNumber(int, int, int) method.
The GameObject class must have a public boolean isOpen field.
The openTile(int, int) method should set the isOpen field on the gameField element equal to true and use the setCellColor(int, int, Color) method to draw the cell background. For example, using Color.GREEN.
The MinesweeperGame class must override the Game parent class's onMouseLeftClick(int, int) method.
The onMouseLeftClick(int, int) method must call the openTile(int, int) method.

package com.codegym.games.minesweeper; // package com.codegym.games.minesweeper; //import com.codegym.engine.cell.*; public class GameObject { public int x; public int y; public boolean isMine; public int countMineNeighbors; public boolean isOpen; public GameObject(int x, int y, boolean isMine) { this.x = x; this.y = y; this.isMine = isMine; } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

3 March 2020, 16:39

The code you shared above has the x, y coordinates reversed. Take for example a small grid: 0 1 2 0|__|__|XX| 1|__|__|__| 2|OO|__|__| To access the box 2,0 (where the XX is) you would pass to the method the values 2 and 0 for x and y. The problem is that to access the same data in a multidimensional array you would need to access it like this [0][2] (not 2,0). Your code would have one box accessing the "XX" while the array is pulling "OO", or visa versa. This is the cause of your error. So using the values 2,0 and my grid above:

//                                        ↓ this is "XX"
if (gameField[x][y].isMine) setCellValue(x,y,MINE);
//               ↑ this is "OO"

Tony Roy

Level 41 , Victorville, United States

3 March 2020, 16:45

Thanks!! At my level, I've only just encountered two-dimensional arrays, and had not grasped this point about access. Thanks so much for your clear explanation!

Guadalupe Gagnon

Level 37 , Tampa, United States

3 March 2020, 17:01

It takes some tinkering and doing tasks for these concepts to set in. Keep at it and you will do fine.

Dyrits

Level 22

30 June 2020, 08:22

I think the code has the coordinates right, but for some reason, the exercise is reverted them...