JavaRush/Помощь по Java задачам/Валідатор не приймає рішення

5 уровень

02.03.2021
421просмотров
9комментариев

Валідатор не приймає рішення

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

Решен

Не можу зрозуміти що Валідатору не подобаєтся. Ніякий код не приймаєтся валідатором.

public class MinesweeperGame extends Game {
    private static final int SIDE = 25;
    private GameObject[][] gameField = new GameObject[SIDE][SIDE];
    private int countMinesOnField = 0;

    public void initialize() {
        setScreenSize(SIDE, SIDE);
        createGame();
    }

    private void createGame() {
        boolean isMine;
        for (int y = 0; y < SIDE; y++) {
            for (int x = 0; x < SIDE; x++) {
                if (getRandomNumber(10) < 1) {
                    isMine = true;
                    countMinesOnField++;
                }
                else isMine = false;
                gameField[y][x] = new GameObject(x, y, isMine);
                setCellColor(x, y, Color.ORANGE);
            }
        }
    }
}

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

Давай введем в игру эти самые мины. Для этого в класс 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; public GameObject(int x, int y, boolean isMine){ this.x = x; this.y = y; this.isMine = isMine; } }

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

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

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

Yaroslaav

Уровень 5

8 марта 2021, 16:06

решение

Баг був у самій Idea , через веб інтерфейс валідатор прийняв рішення

boolean isMine = getRandomNumber(10) < 1;
if (isMine) {
    countMinesOnField++;
}
gameField[y][x] = new GameObject(x, y, isMine);
setCellColor(x, y, Color.ORANGE);

Евгений R&D pharmaceutical develo

2 марта 2021, 20:15

полезный

скорее всего валидатор хочет проверку поля объекта GameObject isMine. то есть сначала создать ячейку и только потом проверить является ли она миной и если да, то countMinesOnField++;

Yaroslaav

Уровень 5

8 марта 2021, 15:18

Якщо я правильно тебе зрозумів то конструкція коду буде така... Але вона теж не працює. Валідатор не хоче приймати рішення. Вже голову зламав собі(((

boolean isMine = getRandomNumber(10) / 1 == 1 ? true : false; //Створення та ініціалізація
                if (isMine) { //Перевірка чи буде створений обєкт з міною чи ні
                    countMinesOnField++;
                }
                gameField[y][x] = new GameObject(x, y, isMine); //Саме створення обєкта поля

Yaroslaav

Уровень 5

8 марта 2021, 15:37

Навіть так... Не приймає((((

for (int y = 0; y < SIDE; y++) {
    for (int x = 0; x < SIDE; x++) {
        gameField[y][x] = new GameObject(x, y, getRandomNumber(10)<1);
        if (gameField[y][x].isMine == true) {
            countMinesOnField++;
        }
        setCellColor(x, y, Color.ORANGE);
    }
}

Aleksei Ilin Software Developer в Axelot

2 марта 2021, 18:59

полезный

я так делал: ... boolean mine = getRandomNumber(10) / 1 == 1 ? true : false; ... надо ещё добавить там где цвет устанавливаете пустую строку в ячейку записать setCellValue(row, col, "");

Ksenia Volkova Java Developer в DXC

Master

2 марта 2021, 20:22

полезный

boolean mine = getRandomNumber(10) / 1 == 1 ? true : false;

это то же самое, что

boolean mine = getRandomNumber(10) / 1 == 1;

Для boolean тернарники не нужны. Ну и getRandomNumber(10) / 1 это то же самое, что просто getRandomNumber(10).

Евгений R&D pharmaceutical develo

2 марта 2021, 20:26

полезный

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

Yaroslaav

Уровень 5

8 марта 2021, 15:09

Тоді ж ми не зможемо підрахувати кількість мін на полі, замінна countMinesOnField. Не так?

Yaroslaav

Уровень 5

8 марта 2021, 15:23

boolean isMine = getRandomNumber(10) / 1 == 1 ? true : false; //Створення та ініціалізація
                if (isMine) { //Перевірка чи буде створений обєкт з міною чи ні
                    countMinesOnField++;
                }
                gameField[y][x] = new GameObject(x, y, isMine); //Саме створення обєкта поля

Попередній код робить те ж саме. Результат один і той же. Але всерівно не працює(