JavaRush/Помощь по Java задачам/Зачем из метода openTile(int,int) нужно вызывать метод ge...

39 уровень

03.12.2019
803просмотров
8комментариев

Зачем из метода openTile(int,int) нужно вызывать метод getNeighbors(GameObject)?

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

Решен

На 9-м шаге в требованиях 2-м пунктом значится: "Метод openTile(int, int) должен вызывать метод getNeighbors(GameObject), если элемент не является миной и количество соседей-мин равно нулю." Вопрос: зачем здесь (в методе openTile) в момент открытия ячейки понадобилось вызывать метод getNeighbors(GameObject)? Ведь метод getNeighbors(GameObject) подсчитывает количество заминированных соседей и этот подсчёт уже сделан ещё раньше, при построении игры и инициализации ячеек игрового поля. В тот момент, когда из countMineNeighbors() вызывается метод getNeighbors(GameObject), устанавливается и присваивается каждой ячейке (каждому элементу игрового поля) соответствующее значение - .countMineNeighbors - количество заминированных соседей. Разве это значение может измениться с течением игры?

К стати, валидатор пропустил, несмотря на то, что в коде из метода openTile(int, int) не вызывается метод getNeighbors(GameObject), как то требуется во втором пункте.

Зачем в требованиях этот пункт с ненужным условием?

Согласно правилам игры, при открытии ячейки, количество "заминированных" соседей которой равно нулю, открывается некоторая "не заминированная" область до ячеек с "заминированными" соседями. Давай это реализуем... Предлагаю использовать рекурсию. Ее применяют тогда, когда алгоритм решения задачи совпадает с алгоритмом решения подзадачи (части). У нас как раз такой случай.

При открытии ячейки с нулевым количеством "заминированных" соседей нам нужно сделать полный перебор всех закрытых соседних ячеек, вызвать у них метод openTile(int, int) и повторить процесс.

Также в методе openTile(int, int) реализуем показ в ячейке количества "заминированных" соседей.

Запусти игру, чтобы проверить корректность реализации.

Подсказка: если получил java.lang.StackOverflowError, значит ты поздно устанавливаешь флаг, что ячейка уже открыта.

Требования:

В методе openTile(int, int), если элемент не является миной и количество соседей-мин равно нулю, для каждого не открытого соседа должен рекурсивно вызываться метод openTile(int, int).
Метод openTile(int, int) должен вызывать метод getNeighbors(GameObject), если элемент не является миной и количество соседей-мин равно нулю.
В методе openTile(int, int), если элемент не является миной и количество соседей мин не равняется нулю, на игровое поле должно выводиться количество заминированных соседей. Используй метод setCellNumber(int, int, int).
Метод openTile(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; public GameObject(int x, int y, boolean isMine) { this.x = x; this.y = y; this.isMine = isMine; } }

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

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

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

ZevoFF

Уровень 11

Expert

21 ноября 2021, 07:32

Очень намучался с последним пунктом... они просят в ячейке где нет мины и количество соседей 0 , установить значение "" setCellValue(gameField[y][x].y,gameField[y][x].x,"");

Soros

Уровень 39

4 декабря 2019, 09:43

Спасибо, разобрался. Поражает скорость реакции на вопрос и возможность быстро находить проблему в коде. Спасибо.

Ksenia Volkova Java Developer в DXC

Master

3 декабря 2019, 15:16

К стати, валидатор пропустил, несмотря на то, что в коде из метода openTile(int, int) не вызывается метод getNeighbors(GameObject), как то требуется во втором пункте. Потому что ты в openTile() повторил уже написанную логику. Анти-DRY

Soros

Уровень 39

3 декабря 2019, 15:17

Повтор логики увидел (логика при переборе соседей), но ведь повтор логики - не вызов всего метода.

Ksenia Volkova Java Developer в DXC

Master

3 декабря 2019, 15:02

getNeighbors() ничего не подсчитывает, а возвращает список соседей. Он нужен для этого: При открытии ячейки с нулевым количеством "заминированных" соседей нам нужно сделать полный перебор всех закрытых соседних ячеек, вызвать у них метод openTile(int, int) и повторить процесс. Ты же фактически повторил то, что уже написано. А вот это неверно, так ты количество мин не получишь

getNeighbors(gameField[x][y]).size();

Soros

Уровень 39

3 декабря 2019, 15:15

getNeighbors(gameField[x][y]).size();

так получаем количество заминированных соседей.

Ksenia Volkova Java Developer в DXC

Master

3 декабря 2019, 15:16

решение

Так получаем количество всех соседей. Ты просто метод поменял, а не надо было

Ksenia Volkova Java Developer в DXC

Master

3 декабря 2019, 15:21

У метода же название не с потолка взялось. Оно отражает то, что метод делает. Никакой проверки на мины там не нужно. Зато этот метод можно использовать и для подсчета соседей-мин (в countMineNeighbors), и для рекурсии (в openTile). Это и позволяет избежать повтора логики.