循环行为不端的条件

时间:2015-12-17 14:53:17

标签: javascript for-loop conditional

我认为它应该很简单,但不知怎的,我不知道我做错了什么。

我有这段代码:

var currentPosition = {
    x:10000,
    y:10000
};

var directions = "v>v<";
var housesVisited = [{x:10000, y:10000}];

function createHouseCoordinates(data){
    for(var x = 0; x<4; x++){
        if(data[x]=="^"){
           currentPosition.x += 1;  
        } else if(data[x]=="v"){
           currentPosition.x -= 1;
        } else if(data[x]==">"){
           currentPosition.y += 1;
        } else if(data[x]=="<"){
           currentPosition.y -= 1;  
        }
    housesVisited.push(currentPosition);
    }
}

createHouseCoordinates(directions);
console.log(housesVisited);

应该以符号的形式获取方向,修改当前位置,并创建一个列出所有访问过的位置的对象数组。上面的结果,当我运行它时,给我这个(基本上是结束位置):

[ { x: 10000, y: 10000 },
{ x: 9998, y: 10000 },
{ x: 9998, y: 10000 },
{ x: 9998, y: 10000 },
{ x: 9998, y: 10000 } ]

我在期待:

[ { x: 10000, y: 10000 },
{ x: 9999, y: 10000 },
{ x: 9999, y: 10001 },
{ x: 9998, y: 10001 },
{ x: 9998, y: 10000 } ]

我做错了什么?我应该怎么读才能更好地理解这一点......? 提前谢谢!

1 个答案:

答案 0 :(得分:2)

currentPosition是一个对象。当您对其进行更改时(例如currentPosition.x += 1),您始终会更改同一个对象。

然后将它推入一个数组,但你实际推送的是该对象的引用,因此数组的所有元素都指向相同的底层对象

要修复代码,您需要在将对象信息推入数组时将对象信息克隆到新对象中:

housesVisited.push({x: currentPosition.x, y: currentPosition.y});