Array.Push 覆盖之前的 Value Node Js

Question

美好的一天。 我有一个全局数组，它必须是全局的。

var comments= [];

我有一个套接字的回调，我正在迭代它并添加值。

代码更好：

    socket.on('commentAdded', function (data) {
    if (data !== null) {
        var stringify = JSON.stringify(data);
        var json = JSON.parse(stringify);

        Object.keys(users).forEach(function (key) {
            if (key !== "null") {
                data.eachUserId = key;
                console.log("data added with id " + key + " the size of comments is " + comments.size);
                comments.push(data);
            }
        });
        console.log("comment was added");
    }

    socket.broadcast.emit('onCommentAdded', data);
});

在这里，我的console.Log("data added with id)... 正在正确打印所有内容，理想情况下，我想向 现有数据（即 json 数据）添加一个新值，新值的名称是 eachUserId，哪个值必须完全不同，因为我在循环中这样做，如您所见。

这就是我之后获得物品的方式。

   for (var f = 0; f < Object.keys(comments).length; f++) {
        var comment = comments[f];
        var eachUserId = comment.eachUserId;
        console.log("checking with current user id" + userId + " each user id" + eachUserId + " or each user id in [] way " + comment['eachUserId']);
        if (eachUserId === userId) {
            socket.emit('onCommentAdded', comment);
        }
    }

这里 eachUserId 始终是循环中添加的最后一项...我做错了什么？为什么push() 方法会覆盖每个值？

Answer 1

问题：

您的问题是当您将值分配给eachUserId 到对象data 时。您只有 1 个名为 data 的对象，并且您一遍又一遍地将同一个对象添加到数组中。但请记住，它们都指向同一个对象，当您更改该对象中的任何内容时，它会在各处反映出来。

因此，当您在循环中更改 data.eachUserId = key; 时，它会更改数组中的所有项目。最后，它们都包含您分配给它的最后一个 eachUserId 值。

解决方案：

您需要克隆对象，然后将其推送到数组中。

---

我建议您使用lodash 库和cloneDeep 方法来创建对象的深层克隆。

var _ = require('lodash');  //require "lodash"

socket.on('commentAdded', function (data) {
    if (data !== null) {        
        Object.keys(users).forEach(function (key) {
            if (key !== "null") {
                var dataClone = _.cloneDeep(data);  //create deep clone
                dataClone.eachUserId = key;         //assign "key"
                comments.push(dataClone);           //push into the array
            }
        });
        console.log("comment was added");
    }    
    socket.broadcast.emit('onCommentAdded', data);
});

Answer 2

使用Spread Operator，已经是Stage 4 (ECMAScript)

const foo = {}

const bar = {...foo}

foo.a = 1;
bar.a = 10;

console.log(foo) // {a : 1}
console.log(bar) // {a : 10}

Answer 3

您确定您的comments 包含所需的数据吗？将其包裹在 closure 中，因此它会创建一个 new scope for each iteration 并传递正确的数据。

for (var f = 0; f < Object.keys(comments).length; f++) {
       (function(f){
          var comment = comments[f];
          var eachUserId = comment.eachUserId;
          console.log("checking with current user id" + userId + " each user id" + eachUserId + " or each user id in [] way " + comment['eachUserId']);
          if (eachUserId === userId) {
             socket.emit('onCommentAdded', comment);
          }
       )(f))
    }