Can anyone plz help me out with this..where I went wrong?

Question about the task

Minesweeper (Part 6/16)

Games, Level 0, Lesson 0

Under discussion

Where is it wrong?

Now let's count the mines in adjacent cells. To do this, we'll add a field to each cell of the matrix, which will be responsible for the number of mined neighbors. First, we find all the neighbors, and then we count how many of them are mined.

To get the list of neighbors, you now have the getNeighbors() method, which has one GameObject (cell) parameter.

Your task is to, in a separate method, count the number of mined neighbors for each cell in gameField and assign this value to the appropriate field of the cell.

Although your work is not visible (you started program to see what happens, right?), it is very important to future steps.

Requirements:

The GameObject class must have a public int countMineNeighbors field.
The MinesweeperGame class must have a private void countMineNeighbors() method.
The countMineNeighbors() method should, for each non-mined cell in the gameField matrix, count the number of adjacent mined cells and assign this value to the countMineNeighbors field.
The countMineNeighbors() method should use the getNeighbors(GameObject gameObject) method.
In the createGame() method, after all the elements of the gameField matrix are created, the countMineNeighbors() method must be called.

package com.codegym.games.minesweeper; import com.codegym.engine.cell.*; import java.util.*; public class MinesweeperGame extends Game { private static final int SIDE = 9; private int countMinesOnField = 0; private GameObject gameField[][] = new GameObject[SIDE][SIDE]; public void initialize() { setScreenSize(SIDE,SIDE); createGame(); } private void createGame() { for(int x =0;x<SIDE;x++) { for(int y=0;y<SIDE;y++) { gameField[y][x] = new GameObject(x,y,false); setCellColor(x,y,Color.ORANGE); int random = getRandomNumber(10); if(random==1)//any one number between 0-9 to get 10% probability { gameField[y][x] = new GameObject(x,y,true); countMinesOnField++; } } } countMineNeighbors(); } private void countMineNeighbors() { for(int x=0;x<SIDE;x++) { for(int y=0;y<SIDE;y++) { if(!gameField[y][x].isMine) { for(GameObject g:getNeighbors(gameField[y][x])) { if(g.isMine) gameField[y][x].countMineNeighbors++; } } } } } private ArrayList<GameObject> getNeighbors(GameObject gameObject) { ArrayList<GameObject> neighbors = new ArrayList<>(); int m = gameObject.y; int n = gameObject.x; for(int x=m-1;x<m+1;x++) { for(int y=n-1;y<n+1;y++) { if(x<0 || x>SIDE) continue; if(y<0 || y>SIDE) continue; if(gameField[y][x]==gameObject) continue; neighbors.add(gameField[y][x]); } } return neighbors; } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Roman

Level 41

17 October 2019, 08:08

Why do you change getNeighbors method? Change it back

private List getNeighbors(GameObject gameObject) {
        List result = new ArrayList<>();
        for (int y = gameObject.y - 1; y <= gameObject.y + 1 ; y++) {
            for (int x = gameObject.x - 1; x <= gameObject.x + 1 ; x++) {
                if (y < 0 || y >= SIDE) {
                    continue;
                }
                if (x < 0 || x >= SIDE) {
                    continue;
                }
                if (x == gameObject.x && y == gameObject.y) {
                    continue;
                }
                result.add(gameField[y][x]);
            }
        }
        return result;
    }