how to loop through angular.forEach and only show results once

Question

I want to loop through an array of numbers to see if it matches up with a variable that i want to check for. It's doing the check correctly but it's displaying the result every time it checks it. What i want it to do is....do the check and just spit out the result only once its finished. Is there a way to do that?

            var myNumber = 123;
            var sponsorNumber = [345, 234, 525];

            angular.forEach(sponsorNumber, function(value) {
              if (value !== myNumber) {              
                console.log('doesnt match!');
              }
            });

Suresh Atta · Accepted Answer · 2017-07-26 17:30:10Z

3

Do not print the message inside the loop. You need to maintain a flag that weather it found or not. And then print in the end.

However you don't need foreach here. Just try indexOf function.

 if (sponsorNumber.indexOf(myNumber) < 0 ) {              
       console.log('doesnt match!');
   }

answered Jul 26, 2017 at 17:30

Suresh Atta

122k38 gold badges206 silver badges315 bronze badges

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

Comments

kamyl · Accepted Answer · 2017-07-26 17:34:14Z

0

Why don't you use simple Array.prototype.indexOf() to do what you actually want? If it returns -1, then you know the array doesn't contain you number. There's no need to reinvent the wheel if there's already a perfect one.

answered Jul 26, 2017 at 17:34

kamyl

6,4564 gold badges26 silver badges29 bronze badges

Comments

xuso · Accepted Answer · 2017-07-26 17:41:31Z

0

If that's what you want, you can filter and then loop (less efficient):

sponsorNumber.filter(value => value !== myNumber).forEach(function(value) {
    console.log('doesnt match!');
});

But it does not make too much sense to me. I think you want something else, like the others commented, using indexOf.

answered Jul 26, 2017 at 17:41

xuso

7351 gold badge8 silver badges19 bronze badges

Collectives™ on Stack Overflow

how to loop through angular.forEach and only show results once

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related