JavaRush/Помощь по Java задачам/Вроде как правильно работает, но не проходят проверки вал...

39 уровень

11.11.2019
597просмотров
8комментариев

Вроде как правильно работает, но не проходят проверки валидатора

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

Решен

Кучу вариантов написания OpenTile() перепробовала, фейлятся вторая и последняя проверки валидатора. Тестирую, вроде как корректно все работает. Вот один из вариантов. Что не так?

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

        gameField[y][x].isOpen = true;
        countClosedTiles--;

        if (gameField[y][x].isMine) {
            setCellValueEx(x, y, Color.LIGHTCORAL, MINE);
            gameOver();
            return;
        }
        setCellColor(x, y, Color.LIGHTSEAGREEN);
        if (countClosedTiles == countMinesOnField) {
            win();
            return;
        }

        if (gameField[y][x].countMineNeighbors == 0) {
            GameObject[] neigh = getNeighbors(gameField[y][x]);

            for (int i = 0; i < neigh.length; i++ ) {
                if (neigh[i] == null) continue;
                openTile(neigh[i].x, neigh[i].y);
            }
            setCellValue(x, y, "");
        }
        else {
            setCellNumber(x,y,gameField[y][x].countMineNeighbors);
        }

    }

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

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

Давай напишем метод 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; public class GameObject { public int x; public int y; public boolean isMine; public int countMineNeighbors; public boolean isOpen; public boolean isFlag; public GameObject(int x, int y, boolean isMine) { this.x = x; this.y = y; this.isMine = isMine; } }

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

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

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

Roman Pryshchepa Java Developer

Expert

12 ноября 2019, 12:14

решение

Нужно исправить сигнатуру метода getNeighbors

private List<GameObject> getNeighbors(GameObject gameObject)

Юлия

Уровень 39

12 ноября 2019, 17:30

Помогло, спасибо. Но, про это раньше не было речи, что getNeighbours должен возвращать лист. Согласна, что лист удобнее, но я на тот момент проходила массивы, поэтому их и использовала. Поправьте пожалуйста шаг 6/16, чтоб он не пускал дальше если getNeighbors возвращает массив.

Юлия

Уровень 39

11 ноября 2019, 17:29

Получается пройти вторую проверку ("В методе openTile(int, int) значение поля countClosedTiles должно уменьшаться на 1, если элемент матрицы gameField отмечается флагом isOpen"), если поменять:

if (countClosedTiles == countMinesOnField) {
    win();
    return;
}

на

if (countClosedTiles == countMinesOnField || !isGameStopped) {
    win();
    return;
}

Но это решение неправильное, т.к. win() будет вызван уже на первом вызове openTile!

Yuriy

Уровень 2

Expert

11 ноября 2019, 08:45

Тоже с подобной причудой валидатора сталкивался, помог вынос этих действий внутрь if. Т.е. вместо Вашей 87 строки вот такой if в котором все остальное тело метода:

if (!gameField[y][x].isFlag && !isGameStopped && !gameField[y][x].isOpen) {
...
}

Юлия

Уровень 39

11 ноября 2019, 12:12

Не помогло :( Такое впечатление, что у каждого свой рецепт, как обойти валидатор в этой задаче.

Yuriy

Уровень 2

Expert

11 ноября 2019, 12:19

Возможно и остальные return стоит убрать, не очень удачное решение использовать его в void методе. У меня была примята вот такая реализация:

private void openTile(int x, int y) {
    if (!gameField[y][x].isFlag && !isGameStopped && !gameField[y][x].isOpen) {
        gameField[y][x].isOpen = true;
        countClosedTiles--;
        if (gameField[y][x].isMine) {
            setCellValueEx(x, y, Color.RED, MINE);
            gameOver();
        }
        else {
            setCellColor(x, y, Color.GREEN);
            if (gameField[y][x].countMineNeighbors > 0)
                setCellNumber(x, y, gameField[y][x].countMineNeighbors);
            else setCellValue(x, y, "");
            if (gameField[y][x].countMineNeighbors == 0)
                for (GameObject near : getNeighbors(gameField[y][x]))
                    if (!near.isOpen) openTile(near.x, near.y);
            if (countClosedTiles == countMinesOnField)
                win();
        }
    }
}

Юлия

Уровень 39

11 ноября 2019, 13:44

Скопипастила, не прошла проверка. Может дело не в этой функции.

Yuriy

Уровень 2

Expert

11 ноября 2019, 13:48

Вполне возможно.