Simple async code with angular.forEach

Question

flag is always false. how can I do?

code here:

var list = ['NOVALUE','VALUE','NOVALUE'];
var flag = false;
angular.forEach(list, function(value){
    if('VALUE' === value){
        flag = true;
    }
});
console.log(flag);

sorry, i missed something

because last element making it false again 'VALUE' === 'NOVALUE' — Pankaj Parkar
– Pankaj Parkar, Commented May 28, 2015 at 10:51
if you're trying to check if an array has a value, you can use indexOf — yoavmatchulsky
– yoavmatchulsky, Commented May 28, 2015 at 10:52
You'd need to break out of that loop when you found a match. Unfortunately there's no breaking mechanism in angular.forEach: stackoverflow.com/questions/13843972/angular-js-break-foreach — Sergiu Paraschiv
– Sergiu Paraschiv, Commented May 28, 2015 at 10:54

JLRishe · Accepted Answer · 2015-05-28 11:30:55Z

1

You should either use .indexOf(), as Alexander demonstrates:

var list = ['NOVALUE','VALUE','NOVALUE'];

var flag = list.indexOf('VALUE') !== -1;

console.log(flag);

or .some() in the more general case:

var list = ['NOVALUE','VALUE','NOVALUE'];

var flag = list.some(function (value) {
    return value === 'VALUE';
});

console.log(flag);

edited May 28, 2015 at 11:30

answered May 28, 2015 at 11:10

JLRishe

102k19 gold badges138 silver badges171 bronze badges

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

Comments

adamjld · Accepted Answer · 2015-05-28 10:55:27Z

0

var list = ['NOVALUE','VALUE','NOVALUE'];
var flag = false;
angular.forEach(list, function(value){
    if (value === 'VALUE'){
        flag = true;
    }
    else{
        flag = false;
    }
    console.log(flag);
});

//false, true, false

answered May 28, 2015 at 10:55

adamjld

3101 silver badge9 bronze badges

1 Comment

NathanOliver Over a year ago

Answers are better if you provide an explanation of why\how they work

Shubham Nigam · Accepted Answer · 2015-05-28 10:56:56Z

0

Use break; so that whentrue statement comes it will go out of the loop

var list = ['NOVALUE','VALUE','NOVALUE'];
        var flag = false;
        for(var i=0;i<list.length;i++){
        if('VALUE' === value)
         {
          flag=true;                
          break;
          }else
          flag=false;
        }
        console.log(flag);

edited May 28, 2015 at 10:56

answered May 28, 2015 at 10:55

Shubham Nigam

3,96821 silver badges33 bronze badges

Comments

Alexander · Accepted Answer · 2015-05-28 11:05:02Z

0

Because the last value in list is "NOVALUE" so it is always false. You can do

flag = list.indexOf('VALUE') > -1

edited May 28, 2015 at 11:05

answered May 28, 2015 at 10:53

Alexander

3,2651 gold badge24 silver badges27 bronze badges

Collectives™ on Stack Overflow

Simple async code with angular.forEach

4 Answers 4

Comments

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related