How can i convert for loop to AngularJs forEach?

Question

I am new to JavaScript world just want to clarify this because i am using AngularJs client side.Below code is working as expected but question is,Is there a way to achieve below result with Angular loop forEach ?

ctrl.js

for (var i = 0; i < selectedOwners.length; i++) {
    if (selectedOwners[i].workerKey === obj.workerKey) {
        selectedOwners.splice(i, 1);
        break;
    }
}

just so you know, that for loop is more performant. also, theres a native js forEach that probably is as well. — Rooster
– Rooster, Commented Mar 9, 2016 at 18:53

Austin · Accepted Answer · 2016-03-09 18:54:16Z

1

This should work:

angular.forEach(selectedOwners, function(owner, $index) {
    if (owner.workerKey === obj.workerKey) {
        selectedOwners.splice($index, 1);
        break;
    }
});

answered Mar 9, 2016 at 18:54

Austin

1,29110 silver badges19 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Rajaprabhu Aravindasamy Over a year ago

break will not work as you are expecting inside forEach. Use return instead.

hussain Over a year ago

Perfect thanks alot i have same result with angularJs.

Bernd Artmüller · Accepted Answer · 2016-03-09 19:05:44Z

0

The easiest way to filter an array is to use the native javascript .filter() function.

var filteredOwners = selectedOwners.filter(function(owner) { 
    return owner.workerKey !== obj.workerKey; 
});

answered Mar 9, 2016 at 19:05

Bernd Artmüller

112 silver badges2 bronze badges

2 Comments

hussain Over a year ago

its not working what we doing with filteredOwners once we return ?

Bernd Artmüller Over a year ago

filteredOwners is an array containing a subset of your original selectedOwners array items that fulfill the return condition. What is not working? Is filteredOwners empty?

DerekMT12 · Accepted Answer · 2016-03-09 19:29:29Z

0

You could do something like this without using angular.forEach

var workerIndex = selectedOwners.map(function(owner) { return owner.workerKey; })
      .indexOf(obj.workerKey);

selectedOwners.splice(workerIndex, 1);

answered Mar 9, 2016 at 19:29

DerekMT12

1,35911 silver badges15 bronze badges

Collectives™ on Stack Overflow

How can i convert for loop to AngularJs forEach?

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related