JavaRush/Помощь по Java задачам/Спустя месяц проблема не обнаружена! )) Может валидатор п...

25 уровень

20.07.2019
778просмотров
13комментариев

Спустя месяц проблема не обнаружена! )) Может валидатор поехал крышей

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

Решен

Все работает и считает, в том числе учитывается и мина. В случае выигрыша отрабатывает win(). В случае проигрыша gameOver(). В том числе, если из последних двух клеток я выбираю мину.

Игра считается выигранной, когда количество оставшихся закрытых ячеек равно количеству мин. Для подсчета числа закрытых ячеек введем дополнительную переменную, значение которой изначально равно общему количеству ячеек на игровом поле. Значение этой переменной будет уменьшаться по мере открытия ячеек. Когда ее значение станет равно количеству мин, игра будет выиграна.

На данный момент, в нашей игре в случае выигрыша ничего не происходит, а должно бы. Пора это исправить.

Давай напишем метод win(), в котором будем останавливать игру и поздравлять победителя красивым сообщением.

Запусти игру и проверь, что при выигрыше отработал метод win().
Обрати внимание, что метод win() вызывается, если количество закрытых ячеек равно количеству мин на поле. При этом, если последняя открытая ячейка оказалась миной, то метод не должен быть вызван.

Требования:

В классе MinesweeperGame должно существовать приватное поле countClosedTiles типа int, инициализированное при объявлении значением SIDE * SIDE.
В методе openTile(int, int) значение поля countClosedTiles должно уменьшаться на 1, если элемент матрицы gameField отмечается флагом isOpen.
В классе MinesweeperGame должен существовать приватный метод void win().
Метод win() должен устанавливать значение переменной isGameStopped в true.
Метод win() должен вызывать метод showMessageDialog(Color, String, Color, int).
Метод openTile(int, int) должен вызывать метод win(), если количество не открытых ячеек равно количеству мин на поле, и последняя открытая ячейка не является миной.

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

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

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

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

Roman Java Developer

Expert

13 августа 2019, 09:52

решение

Изначально метод getNeighbors имел такую сигнатуру:

private List<GameObject> getNeighbors(GameObject gameObject)

не стоило ее менять...

МаксимJava Developer в Freightender

13 августа 2019, 10:36

Да я и не менял его... Метод таким был с момента написания и принят системой. Насколько я помню, в условии было сказано именно про массив. Это видно и по решениям других амиго.

МаксимJava Developer в Freightender

13 августа 2019, 10:38

Спасибо за реакцию )

Алексей

Уровень 17

22 июля 2019, 18:15

private void openTile(int x, int y) {
        if (gameField[y][x].isOpen || gameField[y][x].isFlag) return;

        gameField[y][x].isOpen = true;
        countClosedTiles--;
        setCellColor(x, y, CELL_OPEN);
        if (gameField[y][x].isMine) {
            setCellValueEx(x, y, CELL_MINE, MINE);
            gameOver();
        } else {
            score += 5;
            setScore(score);
            if (gameField[y][x].countMineNeighbors == 0) {
                setCellValue(x, y, "");
                ArrayList<GameObject> gameObjects = getNeighbors(gameField[y][x]);
                for (int i = 0; i < gameObjects.size(); i++) {
                    openTile(gameObjects.get(i).x, gameObjects.get(i).y);
                }
            } else {setCellNumber(x, y, gameField[y][x].countMineNeighbors);
            }
            if (countClosedTiles == countMinesOnField) {
                win();
                return;
            }

        }

    }

Алексей

Уровень 17

22 июля 2019, 18:17

Я тоже с этим долго сидел. Потом сообразил, он по кругу одни и те же клетки открывает, поскольку они подпадают под соседей к разным ячейкам.

МаксимJava Developer в Freightender

23 июля 2019, 04:58

Не совсем понял, что ты имеешь ввиду... ) Чтобы одни и те же клетки при рекурсии не открывались, для этого и стоит условие:

for (GameObject neigh : getNeighbors(array[y][x]))
    if (!neigh.isOpen)                                                  <<<----  это условие.
        openTile(neigh.x, neigh.y);

Уровень 6

21 июля 2019, 15:30

private void openTile(int x, int y) {
if (!isGameStopped && !array[y][x].isFlag && !array[y][x].isOpen ) {
                if (array[y][x].isMine) {
                    setValueCell(x, y,MINE, Color.RED);
                    gameOver();
                }
                if (!array[y][x].isMine && array[y][x].countMineNeighbors == 0) {
                    array[y][x].isOpen = true;
                    countClosedTiles--;
                    setStringValue(x, y, "");
                    setColor(x, y, Color.GREEN);

                        for (GameObject neigh : getNeighbors(array[y][x]))
                            if (!neigh.isOpen)
                                openTile(neigh.x, neigh.y);
                    } else {
                    array[y][x].isOpen = true;
                    setNumberValue(x, y, array[y][x].countMineNeighbors);
                    setColor(x, y, Color.GREEN);
                        countClosedTiles--;
                    }
                }
        if (countClosedTiles == countMinesOnField) win();
    }

попробуй должно работать, поменяй только названия методов на свои

МаксимJava Developer в Freightender

21 июля 2019, 16:07

Ну просто один в один, как мой первый вариант решения. Не работало! ) Ради интереса повторю) Я уже в поддержку написал, быть может валидатор подглючил...

МаксимJava Developer в Freightender

21 июля 2019, 16:08

))) Даже переменные neigh как у меня )))

МаксимJava Developer в Freightender

21 июля 2019, 16:13

Проверил. Те же грабли ))

Уровень 6

21 июля 2019, 19:41

это твой код, просто переделаный немного

Уровень 6

21 июля 2019, 19:51

твой выложеный код немного не правильно работал он открывал даже те ячеки которые не были равны 0. у меня в приложении код который я тебе отправил работает нрмально. но не могу точно сказать что на rush этот код пройдет валидацию

МаксимJava Developer в Freightender

21 июля 2019, 20:11

У меня финальных версий метода openTile() три: 1. Точно такой код, как ты скинул 2. Код, который приложен здесь 3. Слегка перекрученый код 2 ), но меньшего размера. Все три работают одинаково. При клике на ячейку с цифрами должна открываться цифра, при клике на ячейку нулевую, открываются соседи и ячейки с цифрами. Через консоль проверял как счетчики работают и остановка игры. Все четко! Все три варианта непроходят )