Question

我是节点和javascript的新手，并且一直在敲打以下内容。我创建了一个对象如下：

var Subscriber = {
'userID': String,
'email': String,
'name': String,
'stage': String,
'poster': Boolean,
'canEmail': Boolean,
'stage': String, }

我有一个函数，我查询mongodb，并循环结果，尝试加载一组订阅者，我已声明：

var s = Subscriber;
var subscribers = [];

循环如下：

//load array of users that are subscribed to the group
        async.forEach(g.subscribers, function(item, callback) {     
            //load user document for this user
            User.findOne({ _id: item}, function(err, u) {
                if(!err && u) {                 
                    //var s = new Subscriber();
                    console.log('Sub load, found user %s, building array item', u.email);
                    console.log('Subs @ loop start');
                    console.log(util.inspect(subscribers));

                    console.log('Heres foo: ' + util.inspect(foo));


                    s.userID = u._id;
                    s.email = u.email;
                    s.name = u.firstName + ' ' + u.lastName;
                    s.stage = u.stage;
                    s.poster = false; //we're just loading subscribers at this point'
                    if(s.stage != 'new') s.canEmail = true;

                    //push new subscriber onto the array
                    console.log('Pushing ' + util.inspect(s));
                    subscribers.push(s);

                    console.log('At end ' + util.inspect(subscribers));

                    foo.push(s.email);
                    console.log('Heres foo now: ' + util.inspect(foo));

                    callback(null, item);
                }

每次调用subscriber.push（s）之后，数组都有正确数量的元素，但是所有元素都匹配s的最后一个值，就像这样（从数据库中提取两个不同的用户）：

[ { userID: 4fc53a71163006ed0f000002,
email: 'test@test.com',
name: 'undefined undefined',
stage: 'new',
poster: false,
canEmail: true },
  { userID: 4fc53a71163006ed0f000002,
email: 'test@test.com',
name: 'undefined undefined',
stage: 'new',
poster: false,
canEmail: true } ]

推送单个元素而不是整个对象似乎没问题。我添加了“foo”数组作为测试，它工作正常：

Heres foo now: [ 'email1@foo.com', 'test@test.com' ]

这里发生了什么？！？！??！

Answer 1

问题不在于push的{{1}}方法，而在于您的绑定。您正在Array.prototype块中的每个迭代中修改相同的s对象，该对象实际上是与先前定义的async.foreach相同的对象。

首先，您应该将Subscriber变量的声明移动到foreach块。

如果你想创建一个具有默认值的对象，它应该是s，它返回一个新对象：

function

然后你可以像这样实例化一个function Subscriber() { return { 'userID': '', 'email': '', 'name': '', 'stage': '', 'poster': false, 'canEmail': false, 'stage': '' }; };对象：

Subscriber

有关详细说明，请参阅this answer或Closures on MDN。

Answer 2

在推入数组之前克隆对象，也解决了问题。

temp = clone(s);
subscribers.push(temp);

获取https://www.npmjs.com/package/clone

在推送对象时，Array.push（）使所有元素都相同

2 个答案: