JavaRush/Помощь по Java задачам/Убедись, что в методе createGame() вызывается метод getRa...

8 уровень

08.01.2019
1497просмотров
12комментариев

Убедись, что в методе createGame() вызывается метод getRandomNumber() с параметром 10.

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

Игра сапер (5/16)

Игры, 0 уровень, 0 лекция

Решен

Перепробованы куча вариантов. Хочется застрелиться.

Игра "Сапер" не зря так называется. Игроку нужно будет иметь дело с минами, хотя лучше с ними дела не иметь :)

Давай введем в игру эти самые мины. Для этого в класс GameObject внесем изменения, которые учитывают, что ячейка может быть миной — это флаг isMine.

Новый конструктор создавать не нужно: достаточно отредактировать существующий.

Кроме того, при создании ячеек матрицы добавь генерацию мин с вероятностью 10%. Самый простой способ это сделать — использовать метод getRandomNumber(int n) класса Game, который возвращает случайное число от 0 до n-1 включительно. Следовательно, вероятность генерации определенного числа равна 1/n.

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

Если все сделал, запусти программу и проверь, что ничего не поломалось :)

Требования:

В классе GameObject должно существовать публичное поле isMine типа boolean.
В классе GameObject должен существовать один конструктор с тремя параметрами типа int, int, boolean, который устанавливает соответствующие значения полям x, y и isMine в указанном порядке.
В классе MinesweeperGame должно существовать приватное поле countMinesOnField типа int.
В методе createGame() при создании ячеек должно случайным образом задаваться, будет ли эта ячейка с миной. Используй метод getRandomNumber(int) класса Game с параметром 10.
После выполнения метода createGame() полю countMinesOnField должно быть присвоено значение количества мин на поле.

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

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

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

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

Артем Divertitto Senior Android-разработчи в United Tech

Master

10 января 2019, 16:24

1. Вы обращаетесь к несуществуещему объекту

if (getRandomNumber(10) < 1) {
         gameField[x][y].isMine = true;
         countMinesOnField++;
     }

2. Конструктор класса принимает сначала x. new GameObject(y, x, gameField[x][y].isMine); 3. В матрице нужно обращатся наоборот gameField[y][x], сначала y, потом x

Станислав

Уровень 0

10 января 2019, 14:33

У меня просто в конструкторе при создании объекта используется метод с рандомом 29 строка класса майн свип gameField[x][y] = new GameObject(y, x, getRandomNumber(10) < 1);

Alexander Java Developer в ИТ компания

8 января 2019, 18:37

Стреляться не надо))) у меня вот так: /**Выкладывать решение запрещено*/ но тут уже ранее ответили - почти тоже самое

Денис

Уровень 20

8 января 2019, 17:01

Ирина, сначала нужно создать переменную типа boolean (назвать как угодно) и инициализировать ее как false~~(думаю внутри второго for, так как, выносить ее выше за циклы пока не вижу смысла)~~ . Дальше у вас все верно, только замените gameField[x][y].isMine на созданную вами переменную boolean. P.s. сам очень долго тупил (:

Ирина

Уровень 8

11 января 2019, 19:07

Пробовала. На результат никак не влияет, по-моему.

IceBerg

Уровень 41

Master

8 января 2019, 01:30

К getRandomNumber() вопросов вроде нет, но есть к следующей строке: gameField[x][y].isMine = true; У вас же ещё нет объекта в этой ячейке, а вы уже обращаетесь к переменной экземпляра, должно быть NullPointerException.

Ирина

Уровень 8

8 января 2019, 11:43

Не понимаю, что именно должно быть "NullPointerException"?

IceBerg

Уровень 41

Master

8 января 2019, 12:53

Я имел ввиду, что будет вылетать ошибка под названием NullPointerException

Дмитрий Соснин

Уровень 40

Expert

8 января 2019, 13:00

обрати внимание сюда

for (int x = 0; x < gameField.length; x++) {
    for (int y = 0; y < gameField[x].length; y++) {
        if (getRandomNumber(10) < 1) {
            gameField[x][y].isMine = true;
            countMinesOnField++;
        }

        gameField[x][y] = new GameObject(y, x, gameField[x][y].isMine);
        setCellColor(y, x, Color.OLIVE);

    }
}

gameField[x][y] - это массив, в каждой ячейке которого храниться GameObject. однако твой код пытается сделать GameObject в этой ячейке массива миной еще до того, как этот GameObject в ней создан. Точнее даже так тут всё происходит сначала делаешь еще не созданный объект миной, а потом создаёшь сам этот объект, и в качестве одного из параметров передаёшь ему одно из его же полей

Alexander Java Developer в ИТ компания

8 января 2019, 18:38

лучше сделать отдельную булин-переменную, тогда и NPE не будет

Ирина

Уровень 8

11 января 2019, 19:07

То есть, если

if (getRandomNumber(10) < 1) {
            gameField[x][y].isMine = true;
            countMinesOnField++;
        }

gameField[x][y] = new GameObject(y, x, gameField[x][y].isMine);
        setCellColor(y, x, Color.OLIVE);

поменять местами, должно заработать? Или что? Я попробовала, результат тот же.

Дмитрий Соснин

Уровень 40

Expert

11 января 2019, 19:23

решение

не, так тоже не совсем верно, потому что вот тут

gameField[x][y] = new GameObject(y, x, gameField[x][y].isMine);

ты пытаешься обратиться к полю объекта, который ещё только пытаешься создать но ты уже близко, осталось чуть чуть