32

我将一个类似于“John.Doe.100.Newbie-David.Miller.250.Veteran-”的字符串传递给 SplitDatabase 函数，该函数适当地拆分字符串并将值分配给 UserDataEntry 对象。然后将 UserDataEntry 对象推入应该存储所有用户数据的全局 UserData 数组。

但出于某种原因， UserData.push(UserDataEntry) 部分似乎正在覆盖 UserData 数组中的先前数据。第一个循环中的警报在循环时显示正确的数据，但底部的第二个循环中的警报只是一遍又一遍地显示最后一条记录。

我不确定这是为什么？

var UserData = [];


function SplitDatabase(result) {
    var RawUsers = result.split('-');
    var UserDataEntry = {};


    for (var i = 0; i < (RawUsers.length - 1); i++) {
        var tempUserData = RawUsers[i].split('.');
        for (var x = 0; x < (tempUserData.length); x++) {

            switch (x) {
            case 0:
                UserDataEntry.firstname = tempUserData[x];
                break;
            case 1:
                UserDataEntry.lastname = tempUserData[x];
                break;
            case 2:
                UserDataEntry.points = tempUserData[x];
                break;
            case 3:
                UserDataEntry.rank = tempUserData[x];
                UserData.push(UserDataEntry);
                alert(UserData[i].firstname);
                break;
            }
        }

    }

    for (var i = 0; i < (UserData.length); i++) {  
        alert(UserData[i].firstname);
    }

}
4

2 回答 2

77

调用push不会复制您的对象，因为 JavaScriptObject是通过引用传递的：您推送的内容与Object每个数组条目相同。

您可以通过移动循环主体var UserDataEntry = {}; 内部轻松解决此问题，以便在每次循环迭代时创建一个新对象：

    for (var x = 0; x < (tempUserData.length); x++) {
         var UserDataEntry = {};
于 2013-09-27T15:53:56.297 回答
10

将您的行var UserDataEntry = {}放在 for 循环中。

现在，您只有一个对象，并且您将数组的每一部分设置为该对象。您覆盖循环中的成员。

如果您在循环内创建一个新对象，您会将所有新成员添加到数组中。

于 2013-09-27T15:54:07.913 回答