How can I test if all values in an associative array are true? (Javascript)

Question

Associative array:

var signUpStates = {
    "CNPJ": false,
    "PASSWORDS": false,
    "ADDRESS": false,
    "GENERAL_FIELDS": false,
    "TERMS": false,
}

My attempt:

function updateButton() {
    var tempArray = []
    $.each(signUpStates, function(i, val) {
        tempArray.push(val);
    });
    if(tempArray.every(function(e, i, a) { return e == true; })) {
        $(".btn-cadastrar-fornecedor").attr('disabled', false);
    } else {
        $(".btn-cadastrar-fornecedor").attr('disabled', true);
    }
}

Iterating over it and checking individually does not work. What's the best approach to do such a test?

there should be a .all() function, depending on which lib you are using — epoch
– epoch, Commented Nov 25, 2015 at 18:25
You should consider stating your array objects by setting its position like signUpStates = [], cnpj[0] = false, passwords[1] = false. — Anna
– Anna, Commented Nov 25, 2015 at 18:31

dfsq · Accepted Answer · 2015-11-25 18:34:48Z

4

The most straighforward solution (if you are asking about logic) would be to use for loop:

var is = true;

for (var key in signUpStates) {
    if (!signUpStates[key]) {
        is = false;
        break;
    }
}

Strictly saying, correct approach would be also to check only own properties, this is important if the object you are testing might inherit properties from another objects:

var is = true;

for (var key in signUpStates) {
    if (signUpStates.hasOwnProperty(key) && !signUpStates[key]) {
        is = false;
        break;
    }
}

In simple cases like yours, this additional check is not needed.

edited Nov 25, 2015 at 18:34

answered Nov 25, 2015 at 18:27

dfsq

193k26 gold badges244 silver badges261 bronze badges

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

2 Comments

T.J. Crowder Over a year ago

May be worth a mention of hasOwnProperty. Not necessary given the OP's code, though. And +1, sometimes the old ways are best.

dfsq Over a year ago

Yes, I thought about hasOwnProperty but in OP's case it's not necessary. But yes, better to mention.

T.J. Crowder · Accepted Answer · 2015-11-25 18:32:36Z

1

You can get an array of the object's property names from Object.keys, and then use Array#every:

if (Object.keys(signUpStates).every(function(name) { return signUpStates[name]; })) {
    // All are true
} else {
    // Not all are true
}

With ES2015's (aka ES6) arrow functions, this becomes a bit shorter:

if (Object.keys(signUpStates).every(name => signUpStates[name])) {
    // All are true
} else {
    // Not all are true
}

Support for ES2015 is still relatively thin on the ground, particularly if you have to handle browsers that were released before the spec was finalized, so if this is for a general web page, you'd not use them or use a transpiler. If this is for, say, NodeJS, you can use them directly in v4 and higher.

edited Nov 25, 2015 at 18:32

answered Nov 25, 2015 at 18:27

T.J. Crowder

1.1m201 gold badges2k silver badges2k bronze badges

1 Comment

T.J. Crowder Over a year ago

@ZohaibIjaz: Far from it, I assume (but probably shouldn't) that people know that. I use transpilation.

Collectives™ on Stack Overflow

How can I test if all values in an associative array are true? (Javascript)

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related