JavaRush/Помощь по Java задачам/Ругается что я считаю мину у ячеек мин, но это не так... ...

33 уровень

16.09.2021
558просмотров
14комментариев

Ругается что я считаю мину у ячеек мин, но это не так... По крайней мере я этого не понимаю видимо...

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

Решен

Друзья, помогите советом что не так? Я для наглядности вывожу матрицу и игровую таблицу и все вроде совпадает... Так в чем же дело? P.S. никак не могу привыкнуть что игровое поле в графике (x,y) а матрица (y,x), зачем так сделано?

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

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

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

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

Требования:

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

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

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

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

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

Павел Безумный учёный

Expert

16 сентября 2021, 11:58

решение

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

Игорь

Уровень 33

16 сентября 2021, 12:03

Блин дело было только в этом) Благодарю за дельное замечание.

Ksenia Volkova Java Developer в DXC

Master

16 сентября 2021, 10:45

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

Игорь

Уровень 33

16 сентября 2021, 11:27

Убрал лишнее. Что дальше?

Михаил

Уровень 41

16 сентября 2021, 09:55

сам я тоже учусь, но главное отличие от других у тебя стрим. в основном форыч используют. for (GameObject go : nList) if (go.isMine) gameField[i][j].countMineNeighbors++; вот так например.

Игорь

Уровень 33

16 сентября 2021, 11:22

Ну мой стрим правильно работает, я же проверяю, поэтому мой код такой большой, тут все отображается в "двух" измерениях.

Игорь

Уровень 33

16 сентября 2021, 11:34

Я вот этого не понимаю... У меня совершенно другой подход, я подсчитываю в стриме количество соседей и сразу заношу в ячейку это количество. Хотя в целом идея таже... Судя по всему.

gameField[i][j].countMineNeighbors++;

Alexander Avduevsky

Уровень 25

17 сентября 2021, 14:23

Через стрим можно посчитать соседей и более коротким кодом )))

//не забываем импортировать в начале
import static java.util.stream.Collectors.toList;
//ну а потом в нужном месте
gameField[x][y].countMineNeighbors = getNeighbors(gameField[x][y]).stream()
.filter( cell -> cell.isMine)
.collect(toList()).size();

Alexander Avduevsky

Уровень 25

19 сентября 2021, 09:43

И конечно же, можно еще короче и без лишнего импорта!

gameField[x][y].countMineNeighbors = getNeighbors(gameField[x][y]).stream()
.map(s -> s.isMine ? 1:0).reduce(0, Integer::sum);

Михаил

Уровень 41

16 сентября 2021, 09:02

попробуй для всех посчитать соседей. флаг о том что тут мина, будет говорить что вместо цифры будет отображена мина. хотя врядли. подумаю еще :)

Михаил

Уровень 41

16 сентября 2021, 06:33

в матрице первым номером идет номер строки. а вторым номер символа в строке. это из-за того что начало у нас влевом верхнем углу. там ячейка (0,0). а в математике начало в нижнем левом углу. К этому надо поосто привыкнуть. это просто разные системы.

Игорь

Уровень 33

16 сентября 2021, 06:54

Имеется ввиду это?

Михаил

Уровень 41

16 сентября 2021, 07:49

да. исторически сложилось, из за текстового терминала в первых компах.

Игорь

Уровень 33

16 сентября 2021, 08:15

получается класс game следует математической логике для пользователя, но внутри всё наоборот. А создавая матрицу, мы сразу делаем правильно, как программисты? Спасибо за разъяснения, тема не сложная... Но почему тесты то не проходит, если ячейка Mine, я в ней не запупскаю поиск соседей. А запускаю поиск соседей только в ячейках не минах, вроде так требует условие, но проверку это не проходит.