关于为成员变量是二维数组的赋值而导致推箱子游戏回退上一步失败的问题

Posted 2021-06-27 二木成林

异常

其中二维数组中每位数字所表示的含义如下：

推箱子游戏，当点击鼠标右键后就回退到上一步。

单击鼠标右键后回退到上一步的地图数据如下，居然不是初始状态的数据，因为我们要求回退到上一步，而此次上一步的数据是初始状态。

也就是说我们并没有回退成功，并且还多绘制出一个小人，这是什么回事呢？

我们回退到上一步是通过将上一步的地图数据保存到List集合中，然后每次取List集合中的最后一个元素来获取上一步的数据，其实用栈更好理解点，先进后出，但这不是重点。

而每次存入数据是发生在移动之前的，代码如下：

其中Map是自定义的地图类，里面用来存储地图数据的

public class Map {
    // 当前角色的x和y坐标
    int manX = 0, manY = 0;
    // 当前的地图数据
    byte[][] map;
    // 当前的关卡数
    int grade;

    /**
     * 此构造方法用于撤销操作，撤销操作只需要人的位置和地图的当前状态
     *
     * @param manX
     * @param manY
     * @param map
     */
    public Map(int manX, int manY, byte[][] map) {
        this.manX = manX;
        this.manY = manY;
        this.map = map;
    }

    /**
     * 此构造方法用于保存操作，恢复地图时需要人的位置、地图的当前状态和关卡数
     *
     * @param manX
     * @param manY
     * @param map
     * @param grade
     */
    public Map(int manX, int manY, byte[][] map, int grade) {
        this(manX, manY, map);
        this.grade = grade;
    }

    public int getManX() {
        return manX;
    }

    public void setManX(int manX) {
        this.manX = manX;
    }

    public int getManY() {
        return manY;
    }

    public void setManY(int manY) {
        this.manY = manY;
    }

    public byte[][] getMap() {
        return map;
    }

    public void setMap(byte[][] map) {
        this.map = map;
    }

    public int getGrade() {
        return grade;
    }

    public void setGrade(int grade) {
        this.grade = grade;
    }
}

在本次移动中，变化的只有上面红色框内的数据，所以我们着重关注它们，从"9 2 5"变成了"2 6 9"，我们通过debug模式来查看数据的变化：

在移动之前查看map[][]的地图数据内容

在移动之前查看list集合中map[][]的内容

在向左移动一步后，查看map[][]数组的内容

在向左移动一步后，查看list集合中map[][]的内容：

注：list集合是用来保存上一步的地图数据的；map[][]二维数组是当前的地图数据，会随着移动不断变化。

为什么会这样呢？罪魁祸首就是Map.java类中的Map(int, int, byte[][])这个构造，我们知道this.map这个成员变量是一个二维数组，而传入的形参map也是一个二维数组，this.map=map相当于直接将引用传入了，那么随着形参map的内容变化，this.map当然也会变化，因为this.map的引用指向了传入的形参map。这是一个很容易忽视的问题。

解决方法就是遍历形参map的数据内容，重新创建一个临时的temp二维数组，将形参map中的数据全部复制到temp中，再将temp赋给this.map，这样就是值传递，而非引用传递，自然也就不会发生回退到上一步失败的问题了。

    /**
     * 此构造方法用于撤销操作，撤销操作只需要人的位置和地图的当前状态
     *
     * @param manX
     * @param manY
     * @param map
     */
    public Map(int manX, int manY, byte[][] map) {
        this.manX = manX;
        this.manY = manY;
//        this.map = map;// bug代码，不要使用
        // 将map数据直接复制到temp中，然后赋值给this.map
        int row = map.length;
        int column = map[0].length;
        byte[][] temp = new byte[row][column];
        for (int i = 0; i < row; i++) {
            for (int j = 0; j < column; j++) {
                temp[i][j] = map[i][j];
            }
        }
        this.map = temp;
    }

总结：在利用数组作为成员变量时，那么在方法中为数组初始化值时不要传递引用，而是传递值，否则当外界数组内容变化后，成员变量数组也会跟着变化。