Неправильно реализуется метод

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

Игра 2048 (18/18)

Решен

Решил добавить возможность "отката" на один шаг при нажатии клавиши Escape, если тупонул / не та или не там плитка выпала / а что, если...

if (key == Key.ESCAPE) {
   oneStepBack();
   drawScene();
}

Для этого создал private int[][] gameFieldCopy = new int [SIDE][SIDE] и перед каждым ходом копирую матрицу игрового поля в неё. Метод создания копии:

private void makeBackUp() {
   gameFieldCopy = gameField;
   scoreCopy = score;
}

Чтобы вернуться на шаг назад присваиваю текущей матрице значение копии oneStepBack():

private void oneStepBack() {
   gameField = gameFieldCopy;
   score = scoreCopy;
   setScore(score);
}

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

if (key == Key.LEFT) {
   makeBackUp();
   moveLeft();
   drawScene();
}

Для других клавиш механизм тот же. Помогите разобраться, пожалуйста.

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

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

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

Vladimir Frontend Developer

17 марта 2021, 15:24

Причина в том что переменная массива имеет ссылочный тип, поэтому когда вы делаете gameFieldCopy = gameField; вы передаёте ссылку на массив gameField и дальнейшее изменение gameField повлечёт к изменению gameFieldCopy. Чтобы убрать эту ошибку нужно либо копировать массив в цикле for, либо использовать Arrays.copyOf(int[],length).

int [][] array = {{1,2},{3,4}};
        int [][] coppyArray=new int[array.length][];
        for(int i=0 ; i<array.length;i++){
            coppyArray[i]= Arrays.copyOf(array[i],array[i].length);
        }
        array[0][1]=10;
        for(int []i : coppyArray){
            System.out.println(Arrays.toString(i));
        }

Артем

Уровень 24

17 марта 2021, 15:47

Спасибо. Изначально пробовал gameFieldCopy = Arrays.copyOf(gameField, gameField.length), ситуация была та же. Попробую через цикл, отпишусь.

Артем

Уровень 24

17 марта 2021, 16:32

Попробовал циклом:

for (int i = 0; i < SIDE; i++) {
   for (int j = 0; j < SIDE; j++) {
      gameFieldCopy[i][j] = gameField[i][j];
   }
}

Результат тот же :(

Артем

Уровень 24

18 марта 2021, 11:27

Владимир, спасибо! Ваша подсказка на самом деле очень помогла. Всё дело действительно было в ссылках. Решил немного топорно, но всё же:

private void makeBackUp() {
        int[][] tempMatrix = new int[SIDE][SIDE];

        for (int i = 0; i < SIDE; i++) {
            for (int j = 0; j < SIDE; j++) {
                int cell = gameField[i][j];
                tempMatrix[i][j] = cell;
            }
        }
        gameFieldCopy = tempMatrix;

        scoreCopy = score;
    }

Благодаря введению новых объектов - ещё одной матрицы и переменной, устранил всякое пересечение игрового поля и его копии. В конце присваиваю копии ссылку на временную матрицу, чтобы не тратить ресурсы на циклы. Спасибо за помощь!

Vladimir Frontend Developer

18 марта 2021, 16:47

А просто использование в цикле copyOf не решило проблему? copyOf должно корректно работать, если в качестве параметра передавать одномерный массив. Если передавать ему многомерный массив, тогда он копирует только поверхностно, т.е. ссылки, а если передать одномерный массив он копирует значения

Артем

Уровень 24

19 марта 2021, 14:39

Без ввода дополнительного массива не сработало, только

int[][] tempMatrix = new int[gameField.length][];
for (int i = 0; i < gameField.length; i++) {
    tempMatrix[i] = Arrays.copyOf(gameField[i], gameField[i].length);
}
gameFieldCopy = tempMatrix;

Не знал о таких особенностях копирования двумерных массивов, так что спасибо ещё и за науку :)

Артем

Уровень 24

17 марта 2021, 12:47

Состояние матрицы до и после выполнения команд: [[0, 0, 0, 0], [0, 0, 4, 0], [0, 0, 0, 0], [0, 2, 0, 2]] - gameField [[0, 0, 0, 0], [0, 0, 0, 2], [0, 2, 0, 0], [0, 0, 0, 0]] - gameFieldCopy makeBackUp(); [[0, 0, 0, 0], [0, 0, 4, 0], [0, 0, 0, 0], [0, 2, 0, 2]] - gameField [[0, 0, 0, 0], [0, 0, 4, 0], [0, 0, 0, 0], [0, 2, 0, 2]] - gameFieldCopy moveLeft(); [[0, 0, 0, 0], [4, 0, 0, 0], [2, 0, 0, 0], [4, 0, 0, 0]] - gameField [[0, 0, 0, 0], [4, 0, 0, 0], [2, 0, 0, 0], [4, 0, 0, 0]] - gameFieldCopy Видно, что во время выполнения moveLeft() матрице gameFieldCopy неявно присваивается итоговое значение gameField. Неявно, потому что в реализции moveLeft() нигде явного присваивания/вызова makeBackUp() не прописано. Тем более странно, что методы для остальных клавиш также используют moveLeft(), но проблемы с ними нет...