JavaRush/Помощь по Java задачам/3 и 4 пункты не проходят, хотя при запуске игры все ячейк...

31 уровень

04.12.2019
596просмотров
19комментариев

3 и 4 пункты не проходят, хотя при запуске игры все ячейки правильно посчитаны

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

Решен

Где-то в сентябре начал решать эту задачу, не получилось, бросил. Сейчас вернулся, сбросил задачу и начал заново. Дошел до 6-ой задачи, заметил что метод getNeighbors(GameObject gameObject) сформировался автоматически. Реализовал метод countMineNeighbors(). При запуске игры видно, что все правильно посчиталось. В чем проблема не понимаю. Помогите, пожалуйста.

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

Для начала найдем всех соседей, а потом посчитаем, сколько из них "заминированы". Для получения списка соседей у тебя появился метод 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; } }

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

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

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

Ksenia Volkova Java Developer в DXC

Master

4 декабря 2019, 14:15

решение

setCellValue(x, y, ""+gameField[y][x].countMineNeighbors);

Зачем эта строка в countMineNeighbors ?

Руслан

Уровень 31

5 декабря 2019, 06:28

для просмотра/проверки работы программы, чтобы убедиться что все верно посчитано. Вы считаете что это влияет на результат проверки?

Руслан

Уровень 31

5 декабря 2019, 08:00

Да, дело было в ней. Забыл ее убрать. Спасибо, Ксения.

Yuriy

Уровень 41

Expert

4 декабря 2019, 13:59

gameField[y][x] == gameObject

Кто ж так объекты сравнивает? Ай-яй-яй... Да и если у Вас в трех if идущих подряд одно и то же внутри, не логичнее все условия в один if записать через || ?

Ksenia Volkova Java Developer в DXC

Master

4 декабря 2019, 14:10

В таком виде этот метод впиливается автоматически ))

Yuriy

Уровень 41

Expert

4 декабря 2019, 14:17

Да ну?)) Вроде ж до 1 апреля еще далеко...

Ksenia Volkova Java Developer в DXC

Master

4 декабря 2019, 14:21

Сравнение объектов в данном случае как раз корректно, кстати. А три if - ну, может это чтобы понятнее выглядело... )

Yuriy

Уровень 41

Expert

4 декабря 2019, 14:27

Не "корректно", а "сработает". В любом случае оно избыточно - достаточно исключить координаты.

Ksenia Volkova Java Developer в DXC

Master

4 декабря 2019, 14:32

Ну... я бы сказала, что исключение по координатам "сработает", но достаточно проверить равенство ссылки )

Yuriy

Уровень 41

Expert

4 декабря 2019, 14:35

Там в любом случае останется проверка координат, чтобы исключить выпавшие за границы поля.

Руслан

Уровень 31

5 декабря 2019, 06:26

Вы плохо, Yuriy, прочитали вопрос. В нем говорится - "метод getNeighbors(GameObject gameObject) сформировался автоматически".

Yuriy

Уровень 41

Expert

5 декабря 2019, 06:39

Если сформировался, это еще не значит что он корректный. Когда я эту задачу проходил его надо было самому реализовать, но видимо много народу спотыкалось и решили дать готовый, но при этом реализацию вставить забыли - буквально месяц назад были эти вопросы, но никто не удосужился в саппорт написать, пришлось самому. Сейчас добавили, и замечание вполне уместно. Не Вы писали - значить и претензия не к Вам.

Руслан

Уровень 31

5 декабря 2019, 06:50

Я тоже в первый раз сам писал, и вышло гораздо больше кода чем предлагает сейчас джавараш. А чем вам не нравится сравнение объектов (gameField[y][x] == gameObject)? По-моему, так вполне рабочее.

Yuriy

Уровень 41

Expert

5 декабря 2019, 07:01

Интересно что же Вы там такого понаписали? У меня реализация getNeighbors уложилась в 6 строчек, а countMineNeighbors в 4.

Руслан

Уровень 31

5 декабря 2019, 07:12

Не сохранилось и лучше не видеть.🙂

Yuriy

Уровень 41

Expert

5 декабря 2019, 07:43

В таком случае просто согласитесь с тем, что эта реализация getNeighbors, скажем так, не самая удачная. И удалите строку про которую Ksenia Volkova написала - она действительно не нужна.

Руслан

Уровень 31

5 декабря 2019, 07:58

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

Yuriy

Уровень 41

Expert

5 декабря 2019, 08:17

private ArrayList<GameObject> getNeighbors(GameObject gameObject) {
    ArrayList<GameObject> neighbors = new ArrayList<>();
    for (int dX = -1; dX <= 1; dX++)
        for (int dY = -1; dY <= 1; dY++)
            if (!(dX == 0 && dY == 0) && gameObject.x + dX >= 0 && gameObject.x + dX < SIDE && gameObject.y + dY >= 0 && gameObject.y + dY < SIDE)
                neighbors.add(gameField[gameObject.y + dY][gameObject.x + dX]);
    return neighbors;
}

Руслан

Уровень 31

5 декабря 2019, 08:31

Спасибо, Юрий! Да, действительно компактнее. К сожалению, мне легче понять логику метода, который предоставлен учебным курсом.