异常
其中二维数组中每位数字所表示的含义如下:
推箱子游戏,当点击鼠标右键后就回退到上一步。
单击鼠标右键后回退到上一步的地图数据如下,居然不是初始状态的数据,因为我们要求回退到上一步,而此次上一步的数据是初始状态。
也就是说我们并没有回退成功,并且还多绘制出一个小人,这是什么回事呢?
我们回退到上一步是通过将上一步的地图数据保存到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;
}
总结:在利用数组作为成员变量时,那么在方法中为数组初始化值时不要传递引用,而是传递值,否则当外界数组内容变化后,成员变量数组也会跟着变化。