Javascript multiple OR statement not working properly

Question

I have this simple code in Javascript (jQuery) and I need to acces inside a IF statement in case one condition became TRUE, but I always obtain TRUE, even the condition been FALSE, and I don't understand why. Here is the simple code:

for(var i = 0; i < vector.length; i++) {
    genere = vector[i].gender;
    pitch = synthProcess[i].pitch;
    mood = synthProcess[i].mood;
    speed = synthProcess[i].speed;
    identifier = synthProcess[i].ident;

    if (genere !== "auto" || pitch !== "0" || mood !== "0" || speed !== "0" || identifier !== "0"){
        console.log("EXECUTING COMMAND");
    }  
}

What I want is ONLY when genere is different from 'auto', pitch different from '0' etc, then, enter inside the IF but always is entering. I tried with "0" (string) and 0 (int) options, but still getting the same result, in every iteration I get the Log inside the IF

statement in `OR` condition gets executed if one them is `true` and latter condition won't be checked. — Jai, Dec 09 '15 at 10:17
Yes, of course, but in some cases I have: 'auto', '0', '0', '0', '0' and still entering inside... and don't know why — Sergi, Dec 09 '15 at 10:18
No @JamesThorpe , I only want enter inside IF, when just one of the statement become true, only one it's OK — Sergi, Dec 09 '15 at 10:20
@Sergi what if in this case `'auto', '0', '0', '0', '7a6sd6fasd6' ` should go in or not. — Jai, Dec 09 '15 at 10:21
And if any of your variables are actually numbers rather than strings, you want `==` or change `"0"` to `0` as per @Parth's comment. — James Thorpe, Dec 09 '15 at 10:22
@Jai, in this case 'auto', '0', '0', '0', '7a6sd6fasd6' should go in — Sergi, Dec 09 '15 at 10:24
@JamesThorpe yes, I tried to chane "0" to 0 but still the same result — Sergi, Dec 09 '15 at 10:25
From the sounds of things you'll need to make the logic more complex than just one `if` statement if it should only go in if any one of them matches but not more than one - may need a count of how many are true and only execute the insides if the count === 1 — James Thorpe, Dec 09 '15 at 10:27
@Sergi that i have mentioned in 3rd Comment that change it !== then != — Parth Trivedi, Dec 09 '15 at 10:39

score 1 · Accepted Answer · edited May 23 '17 at 12:23

You are doing a strict comparison, which checks the type of your variables as well. Like this:

var a = 'auto';
var b = 0;
var c = 0;

if (a !== 'auto' || b !== '0' || c !== '0') {
    // this will be called since (0 !== '0') = true
    console.log('called!');
}

if (a !== 'auto' || b != '0' || c != '0') {
    // this wont be called since (0 != '0') = false
    console.log('not called');
}

That is, if you compare a zero int with a 0 String using strict comparison, they will be considered different:

console.log(0 === '0') // prints false
console.log(0 !== '0') // prints true

If you compare then using a normal comparison, they will be considered the same:

console.log(0 == '0') // prints true
console.log(0 != '0') // prints false

Check out this explanation and my fiddle.

score 1 · Answer 2 · answered Dec 09 '15 at 10:47

if values of pitch etc are int use-

var  genere = 'auto';
var pitch = 0;
var   mood = 0;
var speed = 0;
var  identifier = 0;
if (genere !== "auto" || pitch !== 0 || mood !== 0 || speed !== 0 || identifier !== 0){
        alert("EXECUTING COMMAND");
}

else, values of pitch etc are string [ex: '0'] then use

var  genere = 'auto';
var pitch = "0";
var   mood = "0";
var speed = "0";
var  identifier = "0";
if (genere !== "auto" || parseint(pitch) !== 0 || parseint(mood) !== 0 || parseint(speed) !== 0 || parseint(identifier) !== 0){
        alert("EXECUTING COMMAND");
}

Javascript multiple OR statement not working properly

2 Answers2