не могу пройти 3, 4 и 5 пункты

Вопрос по задаче

Решен

Помогите пожалуйста, вроде все правильно написал

Теперь давай займемся подсчетом мин в соседних ячейках. Для этого каждой ячейке матрицы добавим параметр (поле класса), который отвечает за количество "заминированных" соседей.

Для начала найдем всех соседей, а потом посчитаем, сколько из них "заминированы". Для получения списка соседей у тебя появился метод getNeighbors, который принимает один параметр типа GameObject (ячейку).

И, наконец, в отдельном методе мы посчитаем количество "заминированных" соседей для каждой ячейки gameField и внесем это значение в соответствующее поле ячейки матрицы

Хотя визуально твоя работа не видна (ты же запустил программу, чтобы посмотреть что получилось?), она очень важна для будущих шагов..

Требования:

В классе GameObject должно существовать публичное поле countMineNeighbors типа int.
В классе MinesweeperGame должен существовать приватный void метод countMineNeighbors().
Метод countMineNeighbors() должен для каждой ячейки "не мины" из матрицы gameField подсчитать количество соседних ячеек "мин" и установить это значение в поле countMineNeighbors.
В методе countMineNeighbors() должен использоваться метод getNeighbors(GameObject gameObject).
В методе createGame() после создания всех элементов матрицы gameField должен быть вызван метод countMineNeighbors().

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

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Дмитрий Малигон QA Automation Engineer

22 января 2020, 18:31

Решил, спасибо ВСЕМ!

Михаил Евреинов Android Developer

22 января 2020, 17:18

Попробуй в циклах "Х" и "Y" поменять местами что бы было как в этом коде.

private void countMineNeighbors(){
        for (int y = 0; y < SIDE; y++) {
            for (int x = 0; x < SIDE; x++) {
                if (!gameField[y][x].isMine) {
                    List<GameObject> neighborsList = getNeighbors(gameField[y][x]);

                    for (GameObject gameObject : neighborsList){
                        if(gameObject.isMine){
                            gameField[y][x].countMineNeighbors++;
                        }
                    }
                }
            }
        }
    }

Ігор Малий

Уровень 6

21 января 2020, 07:54

Привет) во-первых зачем тут добавлена в конструктор переменная int countMineNeighbors? она там не нужна, так как мы создаем объекты не зная будет ли мина рядом. В

GameObject (int x, int y, boolean isMine, int countMineNeighbors) {
        this.x = x;
        this.y = y;
        this.isMine = isMine;

        }
    }

В методе createGame используете rund, а потом строгое равенство единице, в таком случае мин не так уж и много будет, на скок я понимаю, лучше взять значение меньше или больше, подумайте над этим. Ну и опять же лучше выпилить из конструктора countNeighbors и не присваивать ничего, если этого не нужно. Иначе предположительно могут повылезать нули на игровом поле в местах, где не нужно (не вспомню сейчас, это потом будете править по ходу разработки или нет). В методе countMineNeighbors, я бы не создавал List, он излишен, проще создать в одном из вложенных циклов простую int переменную и ей увеличивать значение при каждом проходе с getNeighbors(gameFIeld[y][x]), и потом значение этой переменной присвоить в переменную каждого объекта countMineNeighbors Там вроде не сильно сложно, так что код готовый скидывать не буду, сами додумайтесь) ну или тогда уже подскажу.

Евгений

Уровень 1

21 января 2020, 17:30

Эту переменную в конструктор не нужно добавлять

Дмитрий Малигон QA Automation Engineer

22 января 2020, 06:24

Поменл, но не работает private void countMineNeighbors () { int numberMines = 0; GameObject gameObj; List<GameObject> countMine = new ArrayList<>(); for (int x = 0; x <= SIDE; x++) { for (int y = 0; y <= SIDE; y++) { if (gameField[y][x].isMine) { setCellColor(x, y, Color.RED); continue; } else if (!gameField[y][x].isMine){ countMine = getNeighbors(gameField[y][x]); for (GameObject p : countMine) { if (p.isMine) { numberMines++; } } gameField[y][x].countMineNeighbors = numberMines; } } } } }

Ігор Малий

Уровень 6

22 января 2020, 07:58

int numberMines = 0;  //лучше объявлять и инициализировать её в том блоке кода, где она нужна. ну или её придётся обнулять в конце обработки очередной ячейки
        GameObject gameObj; // лишнее
        List<GameObject> countMine = new ArrayList<>(); // тоже лишнее

        for (int x = 0; x <= SIDE; x++) {
            for (int y = 0; y <= SIDE; y++) {
                /* следующий if-блок не нужен здесь, мы тут считаем кол-во мин, если тут помечать клетку красным, то клетка с миной будет становится красной при использовании этого метода)*/
                if (gameField[y][x].isMine) {
                    setCellColor(x, y, Color.RED);
                    continue;
                }
                else if (!gameField[y][x].isMine){ //вот тут бы я сделал первый if
                                                               //тут бы объявлял и инициализировал переменную int numberMines
                    countMine = getNeighbors(gameField[y][x]); // можно засунуть в условие цикла foreach
                    for (GameObject p : countMine) {
                        if (p.isMine) {
                            numberMines++;
                        }
                    }
                    gameField[y][x].countMineNeighbors = numberMines;

                }
            }
        }

    }
}

Должно заработать

Илья Ненашев

Уровень 14

22 января 2020, 17:52

я смотрю здесь собрались прям знатоки

Ігор Малий

Уровень 6

22 января 2020, 21:29

😂😂😂 прям уж там просто помню, как я намудохался именно с шестой подзадачей этого квеста по Сапёру. Плюс обычно, когда я захожу в вопросы, то там уже всё решили и без моих пяти копеек)