Javascript Array.Splice（）不会删除条目

Question

如果某个条件为真，我正试图从数组中删除一个条目，但是当我在console.log对象数组时，该对象尚未删除，我很困惑。我正确使用Splice（）函数吗？

    var itemsProcessed;
people.forEach(async function(peep, index, object){

    var d = new Date();
    var currenttime = d.getTime();
    if (peep.endtime < currenttime){
        var rolesub = guild.roles.find(r => r.name == roleNameSub);

        var user2 = await client.fetchUser(peep.id);
        var member = await guild.fetchMember(user2);

        member.removeRole(rolesub);

        object.splice(index, 1);  //This seems to go wrong...

        console.log(peep.id + " Lost subscription!");
        user2.send("Your subscription ended!");

    }
    itemsProcessed++;
    if (itemsProcessed === object.length){
        SaveJson(people, "users.json");
    }
});    

Answer 1

你的问题是你正在拼接你正在迭代的同一个数组，这就是为什么索引不正确。

您应该在迭代之前创建数组的副本，并通过检索要删除的元素的索引从原始数组中删除元素，请查看下面的内容。

arr.slice(0).forEach(function(item) {
    arr.splice(arr.indexOf(item), 1);
});

var arr = [{0:0},{i:1},{i:"test"},{i:"Something else"},{i:"Test"},5];
    
arr.slice(0).forEach(function(item) {

    if(item != 5)
        arr.splice(arr.indexOf(item), 1);
});
    
console.log(arr);

Answer 2

您可以考虑和改变的一件事是，当您迭代数组时，不要在迭代时从中删除它。只需将条目标记为删除即可。然后，完成后，过滤掉需要删除的那些。

例如：

people.forEach( (person,i) => {

   if( /* person needs to be removed from people */ ) {
       people[i] = null;  // use use array from 3rd parameter if you want to pass it in
   }
});

// then remove unwanted people

people = people.filter( person => person != null );

或者，如果您不想将此人设置为null，则可以在对象中设置一个标志，而不是将其标记为删除。

例：

people.forEach( (person,i) => {
   if( /* person needs to be removed from people */ ) {
       person.toBeDeleted = true;
   }
});

people = people.filter( person => person.toBeDeleted !== true );

可能更好或更清洁的方法是不使用forEach而只使用过滤器。

例：

    people = people.filter( p_person => {
         if( /* person stays */ ) {
             return true;
         } else {
             return false;
         }
    });