foreach循环内的JavaScript拼接函数递减索引

Question

$scope.clearCompleted = function()
        {
            angular.forEach($scope.todos, function(todo, i)
            {
                if(todo.done)
                {
                    $scope.todos.splice(i, 1);
                }
            });

            if($scope.todos.length == 0)
            {
                $scope.isEmpty = true;
            }
        }

这是我从数组中删除'done'待办事项的代码，但是当两个todos相继删除后，它只删除第二个。 我认为这是因为splice函数重置并返回拼接数组。

Answer 1

您从迭代的数组中拼接元素，因此“todos”中的索引减少了。 对不起，我的英语不好。

var notDonedTodos = [];
angular.forEach($scope.todos, function(todo, i)
{
    if(!todo.done)
    {
       notDonedTodos.push(todo);
    }
});

$scope.todos = notDonedTodos;

Answer 2

发生这种情况是因为forEach只知道数组的初始状态，因此调用您的方法两次，即使第一次调用从数组中删除了一个项目。 只需做一个简单的while循环：

var i = $scope.todos.length;
while (i--){
    if ($scope.todos[i].done){
        $scope.todos.splice(i, 1);
    }
}

Answer 3

我发现自己的替代方案是使用array.filter方法。 这是基于对象键过滤数组的最简单方法。 如果你正在开发一个IE8项目（你很差），你需要为这个函数添加一个polyfill，因为它对JavaScript来说还算新。

关于javascript你需要知道的一切 。

答案代码：

$scope.clearCompleted = function() {
    $scope.todos = $scope.todos.filter(function(item) {
        return !item.done;
    });
}

Answer 4

each迭代的问题是它从数组中删除一个项，导致跳过迭代。 jQuery有一个很好的grep方法，它返回所有符合某个标准的元素，这个标准由提供的匿名函数决定。

var todos =[{id:1, done:false},{id:2, done:true},{id:3, done:true}];

function removeCompleted(todos){
    return $.grep(todos,function(todo){
        return todo.done == false;
    });
}

todos = removeCompleted(todos);
console.log(todos);

工作示例 http://jsfiddle.net/ktCEN/

文档

Answer 5

作为另一种选择，您可以在每次进行splice时减少索引。 例如：

$scope.clearCompleted = function() {
    angular.forEach($scope.todos, function(todo, i) {
        if(todo.done) {
            $scope.todos.splice(i, 1);
            i--;
        };
    });

    if($scope.todos.length == 0) {
        $scope.isEmpty = true;
    };
}

这会调整您的索引，以便在每次修改数组时保持其有效性。 您仍然可以使用angular.forEach ，并且最终不会得到阵列的两个副本。