Question

193

rated 0 times [ 197] [ 4] / answers: 1 / hits: 58844 / 10 Years ago, fri, june 13, 2014, 12:00:00

I have an array like this:

$scope.emails = [

  {key:Work,value:[email protected]},

  {key:,value:},

   {key:Work,value:[email protected]}

  {key:,value:}];

So, I want to remove empty emails but angular forEach method removing only one object that is last object why???.

js code

angular.forEach($scope.emails, function(email, index){

     if(email.value ===){

       $scope.emails.splice(index, 1);



     } 



    });

where I am doing wrong

Only authorized users can answer the question. Please sign in first, or register a free account.

Total Points: 462

Total Questions: 100

Total Answers: 102

Location: Jersey

Member since Mon, Jun 14, 2021

3 Years ago

lara questions

Thu, May 19, 22, 00:00, 2 Years ago

Tue, Jul 27, 21, 00:00, 3 Years ago

Thu, Apr 1, 21, 00:00, 3 Years ago

Mon, Jul 27, 20, 00:00, 4 Years ago

Tue, Mar 17, 20, 00:00, 4 Years ago

answered 10 Years ago irvinjovannix · Accepted Answer

The problem is that you remove elements from the array during the loop so later items are at different indices. You need to loop backwards instead:

for (var i = $scope.emails.length - 1; i >= 0; i--) {

    if (!$scope.emails[i].value) {

        $scope.emails.splice(i, 1);

    }

}