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);
});

在这里，我正确地打印了所有内容，理想情况下，我想向现有数据（即 json 数据）console.Log("data added with id)...添加一个新值，新值的名称必须与我在循环内执行的值完全不同，如您所见。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

问题：

您的问题是当您将值分配给eachUserIdobject时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);
});