Он не согласен

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

2048(10)

Решен

не понимаю, что не так

Какие методы вызывать по нажатию клавиш мы уже знаем. Поэтому приступим к их реализации. Начнем с метода moveLeft().

В методе moveLeft() мы сдвигаем элементы каждой строки матрицы gameField методом compressRow(int[]), после соединяем их значения с помощью метода mergeRow(int[]), а затем снова сдвигаем (так как после соединения могут образоваться пустые промежутки).
В эти методы в качестве параметра должна поочередно передаваться каждая строка матрицы gameField.

Пример:

Начальная строка: {2, 2, 0, 2}
После compressRow(int[]): {2, 2, 2, 0}
После mergeRow(int[]): {4, 0, 2, 0}
После compressRow(int[]): {4, 2, 0, 0}

Если был хотя бы один сдвиг или соединение элементов, ход считается совершенным, и мы добавляем в массив новое число — вызываем метод createNewNumber().

Чтобы это было видно на экране, в конце нужно вызвать наш метод рисования drawScene().

Требования:

Метод moveLeft() должен перемещать и соединять элементы матрицы gameField влево в соответствии с правилами игры.
Метод moveLeft() должен вызывать метод compressRow(int[]) как минимум по одному разу для каждой строки матрицы gameField.
Метод moveLeft() должен вызывать метод mergeRow(int[]) ровно по одному разу для каждой строки матрицы gameField.
Метод moveLeft() должен добавлять плитку с помощью метода createNewNumber(), если перемещение или соединение произошло.
Метод moveLeft() не должен изменять содержимое матрицы gameField, если ни перемещение, ни соединение влево невозможны.
Метод onKeyPress(Key) должен вызывать метод drawScene(), если была нажата клавиша движения (Key.UP, Key.RIGHT, Key.LEFT, Key.DOWN).

package com.javarush.games.game2048; import com.javarush.engine.cell.Game; import com.javarush.engine.cell.*; public class Game2048 extends Game { private static final int SIDE = 4; private int[][] gameField = new int[SIDE][SIDE];//приватная матрица @Override public void initialize() { setScreenSize(SIDE, SIDE); createGame(); drawScene(); } private void createGame() { createNewNumber(); createNewNumber(); } private void drawScene() { for (int x = 0; x < SIDE; x++) { for (int y = 0; y < SIDE; y++) { int v = gameField[y][x]; setCellColoredNumber(x, y, v); } } } private void createNewNumber() { int x = getRandomNumber(SIDE); int y = getRandomNumber(SIDE); int number = getRandomNumber(10); if (gameField[y][x] == 0) { if (number < 9) gameField[y][x] = 2; else gameField[y][x] = 4; } else createNewNumber(); } private Color getColorByValue(int value) { Color c = null; if (value == 2) c = Color.AQUA; if (value == 2) c = Color.GRAY; if (value == 4) c = Color.RED; if (value == 8) c = Color.FIREBRICK; if (value == 16) c = Color.LAVENDER; if (value == 32) c = Color.GREENYELLOW; if (value == 64) c = Color.TEAL; if (value == 128) c = Color.NAVAJOWHITE; if (value == 256) c = Color.MAGENTA; if (value == 512) c = Color.CRIMSON; if (value == 1024) c = Color.LIME; if (value == 2048) c = Color.TOMATO; return c; } private void setCellColoredNumber(int x, int y, int value) { Color color = getColorByValue(gameField[y][x]); if (value == 0) { setCellValueEx(x, y, color, ""); } else setCellValueEx(x, y, color, String.valueOf(value)); } private boolean compressRow(int[] row) { boolean result = false; for (int j = 0; j < SIDE; j++) { for (int i = 1; i < SIDE; i++) { //int temp = gameField[i-1]; if (row[i - 1] == 0 && row[i] != 0) { int temp = row[i - 1]; row[i - 1] = row[i]; row[i] = temp; result = true; } } } return result; } private boolean mergeRow(int[] row) { boolean result = false; for (int i = 1; i < SIDE; i++) { if (row[i - 1] == row[i] && row[i - 1] != 0) { row[i - 1] *= 2; row[i] = 0; i++; result = true; } } return result; } @Override public void onKeyPress(Key key) { switch (key) { case LEFT:moveLeft(); break; case RIGHT: moveRight(); break; case UP: moveUp(); break; case DOWN: moveDown(); } } private void moveLeft() { boolean compressRow = false; boolean mergeRow=false; for (int i = 0; i < SIDE; i++) { if (compressRow(gameField[i])){ compressRow=true;} if (mergeRow(gameField[i]) || compressRow == true){ mergeRow=true;} if (compressRow(gameField[i])||mergeRow==true) { compressRow = true; } if (compressRow == true || mergeRow == true) { createNewNumber(); drawScene(); } } } private void moveRight() { } private void moveUp() { } private void moveDown() { } }

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

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

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

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

Expert

7 января 2021, 18:34

решение

Проверки внутри цикла в методе moveLeft() можно несколько упростить:

if (compressRow(gameField[i])) {
    compressRow = true;
}
if (mergeRow(gameField[i])) {
    mergeRow = true;
}
if (compressRow(gameField[i])) {
    compressRow = true;
}

Метод createNewNumber() должен быть вызван один раз (его вызов необходимо вынести за пределы тела цикла). Проверки для вызова метода также можно упростить:

if (compressRow || mergeRow) {
    createNewNumber();
}

Вызов метода drawScene() перенесите в конец тела метода onKeyPress().

hidden #2297125

Уровень 18

7 января 2021, 19:27

Спасибо большое!!! за внятно-понятное объяснение. Получилось,,, единственное, что метод drawScene() не срабатывал в конце метода, а сработало только вот так

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

Expert

7 января 2021, 19:44

решение

Всё верно ) Поскольку метод drawScene() должен вызываться только при нажатии клавиши движения, то тут есть два варианта. Один из них представлен Вами выше, другой выглядит так:

public void onKeyPress(Key key) {
    switch (key) {
        case LEFT  : moveLeft();
            break;
        case RIGHT : moveRight();
            break;
        case UP    : moveUp();
            break;
        case DOWN  : moveDown();
            break;
        default    : return;
    }
    drawScene();
}

В этом случае при нажатии других клавиш управление из метода будет возвращено без вызова drawScene().