Ругается на 4-й и 5-й пункты.

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

Сапер(part10)

Games, 0 уровень, 0 лекция

Решен

Уже все перебрал, игра работает, флажки ставятся и удаляются.

Теперь давай займемся реализацией метода, который отвечает за маркировку ячейки флажком. Метод назовем markTile .

Он должен:

- отмечать ячейку на игровом поле флагом или снимать флаг;
- следить за количеством флагов;
- заниматься отрисовкой и стиранием флагов на игровом поле;
- менять цвет ячейки поля, если в ней устанавливается флаг и возвращать цвет обратно, если флаг снимается.

Поставить или снять флаг в игре можно с помощью клика правой кнопки мыши. Поэтому в классе MinesweeperGame нужно переопределить метод onMouseRightClick(int, int) класса Game и в нем вызвать метод markTile(int x, int y).

Проверь, что все работает — запусти игру.

Требования:

В классе MinesweeperGame должен существовать приватный метод void markTile(int, int).
Метод markTile(int x, int y) не должен ничего делать, если элемент уже открыт (isOpen == true).
Метод markTile(int, int) не должен ничего делать, если количество неиспользованных флагов countFlags равно нулю, и текущий элемент — не флаг (isFlag = false).
Метод markTile(int, int) должен устанавливать значение поля isFlag в true, уменьшать количество неиспользованных флагов на единицу, отрисовывать на поле знак FLAG, если текущий элемент — не флаг (используй метод setCellValue(int, int, String)) и менять фон ячейки на поле, используя метод setCellColor(int, int, Color). Например, в Color.YELLOW.
Метод markTile(int, int) должен устанавливать значение поля isFlag в false, увеличивать количество неиспользованных флагов на единицу, отрисовывать на поле пустую ячейку, если текущий элемент — флаг (используй метод setCellValue(int, int, String)) и возвращать исходный цвет ячейки (используй метод setCellColor(int, int, Color)).
В классе MinesweeperGame должен быть переопределен метод onMouseRightClick(int, int) родительского класса Game.
Метод onMouseRightClick(int, int) должен вызывать метод markTile(int, int).

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; GameObject (int x1, int y1, boolean mine) { x = x1; y = y1; isMine = mine; } }

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

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

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

Antonio

Уровень 13

7 августа 2019, 16:15

Зачем-же так издеваться над людьми? Ну ведь время то не резиновое!!! Я уже 3 года подписки покупаю!

Antonio

Уровень 13

7 августа 2019, 16:14

решение

Ааааааа! Я просто негодую! Валик не принимал решения только из-за дополнительной (лишней) строки, которую я ввел для проверки количества флагов!

setCellNumber(0, 0, countFlags);

Кирилл ОрловJava Developer в АО ЦПЛ

7 августа 2019, 10:15

Вы пробовали ставить флаги на пустые ячейки? Или открывать и закрывать уже открытые ячейки кликами мышью? Попробуйте дебажить прямо в интерфейсе, без дебагера в ide. Найдете баги, найдете ответы.

Antonio

Уровень 13

7 августа 2019, 15:56

Кирилл. Спасибо за ответ. Да, ячейки маркируются и флаги снимаются без проблем. Что подразумеваете под открывать и закрывать? При открытии ячейки она уже не закрывается. Или имеет в виду ставить и снимать флаги? Я работаю через браузер.

Кирилл ОрловJava Developer в АО ЦПЛ

9 августа 2019, 13:55

Лучше б через IDE. У меня была бага, что открытую ячейку можно было флагать, даже закрыть, тыкая в соседнюю. По таким ситуациям удобно понимать, где накосячил. Я пока игру писал, на каждом шаге проверял, тыкая во все подряд поля. И не пойму, какой ваш код смотреть, что в шапке задачи, или который ниже?

Antonio

Уровень 13

10 августа 2019, 07:15

Я не совсем корректно выложил код, который ниже. Дело в том, что в нем нет ошибок, а ошибка была в том, что для проверки я выводил в верхней левой ячейке (для себя) количество флагов, которое уменьшалось каждый раз при установке флага и увеличивалось. Это видно, если посмотреть весь код в задаче. В комменте ниже этого нет, поэтому некорректно запросил помощь. Сейчас уже все сделал и сапер работает. Можно двигаться дальше.

Antonio

Уровень 13

5 августа 2019, 14:45

if (!gameField[y][x].isOpen) { // проверяем что ячейка закрыта
            if (countFlags == 0 && !gameField[y][x].isFlag) return;
            else if (countFlags > 0 && !gameField[y][x].isFlag) { // проверяем что есть флаги и ячейка еще не помечена
                setCellValue(x, y, FLAG); // рисуем на ячейке флаг
                gameField[y][x].isFlag = true; //помечаем флагом на ячеку
                setCellColor(x, y, Color.YELLOW);
                countFlags = countFlags - 1; // уменьшаем количество флагов на единицу

            } else if (gameField[y][x].isFlag) { // если же поле уже помечено флагом то делаем все наоборот

                    gameField[y][x].isFlag = false;//говорим что поле не помечено флагом
                    setCellValue(x, y, "");//стираем флаг и устанавливаем пустую строку
                    setCellColor(x, y, Color.ORANGE);//окрашиваем в исходный цвет
                    countFlags++; //снимаем флаг и увеличиваем количество флагов
                }

       }