javascript if if else (not else if)

Question

I have a simple bit of code that checks some conditions, and I want each condition to cause a unique output, but if no conditions are met then to cause another unique output.

Is there any way to create an else that is only triggered if all previous if statements fail? I am aware that the below code carries out the same purpose, but it seems cumbersome and it would be nicer to have a quicker way of checking all of the variables than copy and paste all of them into another if statement.

var boolean1 = true,
  boolean2 = true,
  boolean3 = false;

if (boolean1) {
  alert("boolean1");
}
if (boolean2) {
  alert("boolean2");
}
if (boolean3) {
  alert("boolean3");
}
/* and so on */
if (!boolean1 && !boolean2 && !boolean3 /* etc */ ) {
  alert("none");
}

it would potentially be easier, if the booleans would be in an array, is that possible? — Nils
– Nils, Commented Jan 18, 2017 at 18:35
@zero298 — No: looks for the first case clause whose expression evaluates to the same value as the result of the input expression. If boolean1 is true, it won't check the boolean2 case. — Quentin
– Quentin, Commented Jan 18, 2017 at 18:39
@zero298 My understanding is switch case evaluates based on the outcome of one expression. This one deals with many Boolean variables, is that possible with switch case? — Thangadurai
– Thangadurai, Commented Jan 18, 2017 at 18:41
the accepted answer here gives a shortcut to this problem stackoverflow.com/questions/3463833/… — Thangadurai
– Thangadurai, Commented Jan 18, 2017 at 18:45

Cᴏʀʏ · Accepted Answer · 2017-01-18 19:06:10Z

1

To make this scale, you will need to change your data structure to something you can iterate over. That way your processing logic remains the same and you can supply any size input. Here's one implementation where I use an object with properties representation your Boolean values. The checkResults function below iterates over the property names in the object, checks the value for each one, and performs an alert and sets a flag if any value is true. At the end it checks if any value was true and alerts with that case.

function checkResults(resultMap) {
    var any = false;
    Object.keys(resultMap).forEach(function(key) {
        if (resultMap[key] === true) {
          any = true;
          alert(key);
        }
    });
    if (!any)
      alert('none');
}

checkResults({
  'boolean1': false,
  'boolean2': true,
  'boolean3': false
});

You can build your input object property by property if you have to:

var results = {};
results['boolean1'] = true;
results['boolean2'] = false;
...
checkResults(results);

And the checkResults function always stays the same.

edited Jan 18, 2017 at 19:06

answered Jan 18, 2017 at 18:50

Cᴏʀʏ

108k20 gold badges169 silver badges201 bronze badges

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

2 Comments

Philip Eagles Over a year ago

Accepting this as the answer as it is the most compact method I've seen. As a side note, adding a second object that stores a string for each result (e.g. the "alert('boolean1')") that is called with an eval("script string goes here") would make it compatible for other code blocks.

Cᴏʀʏ Over a year ago

@PhilipEagles: True, but there's no need for eval or even a second object -- in fact, if you ever find yourself using eval, you've got a smell that needs to be removed. You could alter this to use an array of objects that contains your "rules": [{ name: 'boolean1', value: true, message : function() { alert('boolean1 was bad!'); }, ...]. Then you can adjust the function to loop over the array objects and invoke the message function stored there.

Quentin · Accepted Answer · 2017-01-18 18:38:07Z

1

The only other way I can see to do this is for each function to set a different boolean and then test that.

var boolean1 = true,
    boolean2 = true,
    boolean3 = false,
    any = false;


if (boolean1) {
  alert("boolean1");
  any = true;
}
if (boolean2) {
  alert("boolean2");
  any = true;
}
if (boolean3) {
  alert("boolean3");
  any = true;
}
/* and so on */
if (!any) {
  alert("none")
}

This probably isn't much of an improvement though.

answered Jan 18, 2017 at 18:38

Quentin

949k136 gold badges1.3k silver badges1.4k bronze badges

Comments

Nina Scholz · Accepted Answer · 2025-02-27 13:21:19Z

1

You could reduce the array of entries and return either the result of the former iteration if the boolean value is false or true in the other case. Finally check the returned value and show a message.

function checkResults(results) {
    const
        check = results => Object
            .entries(results)
            .reduce((r, [k, v]) =>  {
                if (!v) return r;
                console.log(k);
                return true;
            }, false);

    if (!check(results)) console.log('none');
}

checkResults({ boolean1: false, boolean2: true, boolean3: true });
checkResults({ boolean1: false, boolean2: false, boolean3: false });

edited Feb 27 at 13:21

answered Feb 27 at 10:08

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Comments

Sushanth -- · Accepted Answer · 2017-01-18 18:41:54Z

0

You could maintain a flag which when passes the condition alerts none

var boolean1 = true;
var boolean2 = true;
var boolean3 = false;
var allConditionFailed = true;

if (boolean1) {
  alert("boolean1");
  allConditionFailed = false;
}
if (boolean2) {
  alert("boolean2");
  allConditionFailed = false;
}
if (boolean3) {
  alert("boolean3");
  allConditionFailed = false;
}

if(allConditionFailed) {
   alert("none");
}

answered Jan 18, 2017 at 18:41

Sushanth --

55.8k9 gold badges70 silver badges109 bronze badges

Comments

ibrahim mahrir · Accepted Answer · 2017-01-18 18:42:38Z

0

var accumulated = false;

acumulated = acumulated || boolean1;
if (boolean1) {
  alert("boolean1");
}

acumulated = acumulated || boolean2;
if (boolean2) {
  alert("boolean2");
}

acumulated = acumulated || boolean3;
if (boolean3) {
  alert("boolean3");
}

if(!acumulated) {
  alert("none");
}

edited Jan 18, 2017 at 18:42

answered Jan 18, 2017 at 18:35

ibrahim mahrir

31.8k5 gold badges50 silver badges78 bronze badges

Comments

Bergi · Accepted Answer · 2017-01-18 18:51:26Z

No, there is none. Just imagine the control flow diagram for your code, there's no way to convert that to a structured program (or even an unstructured one) without evaluating a boolean in multiple places or without introducing helper variables.

If you want to check every condition only once (in the execution I mean, not in the code), you could do

if (boolean1) {
  alert("boolean1");
  if (boolean2) {
    alert("boolean2");
  }
  if (boolean3) {
    alert("boolean3");
  }
} else {
  if (boolean2) {
    alert("boolean2");
    if (boolean3) {
      alert("boolean3");
    }
  } else {
    if (boolean3) {
      alert("boolean3");
    } else {
      alert("none")
    }
  }
}

but that is arguably much uglier than your original solution.

I suspect what you actually want is an array of options:

var booleans = [true, true, false];
var alerts = ["boolean1", "boolean2", "boolean3"];
for (var i=0; i<booleans.length; i++)
  if (booleans[i])
    alert(alerts[i]);
if (booleans.every(b => !b))
  alert("none");

Rabbi Shuki Gur · Accepted Answer · 2017-01-18 19:26:26Z

0

If you can save them as an array then you can use array.indexOf() to make sure that true doesn't exist in the array (all tests fail), and if it does loop through the array with array.forEach() to do something with each of them.

So your code will look something like this:

// The array of conditions.
var bools = [true, true, false];

// Check if none of them are true (all fail).
if (bools.indexOf(true) === -1) {
    alert("none");
} else {
// Loop over them and do something based if the current one is true.
bools.forEach(function(b, i) {
    b ? alert("boolean" + (i + 1)) : '';
});
};

answered Jan 18, 2017 at 19:26

Rabbi Shuki Gur

1,72620 silver badges37 bronze badges

Comments

Nils · Accepted Answer · 2017-01-18 19:56:24Z

0

if an array is possible try this:

var booleans = [false, false, false];
var i = 0;
while (i < booleans.length) {
    if (booleans[i] === true) {
        alert("boolean"+i);
    }
    i++;
}
i = 0;
while (booleans[i] === false && i < booleans.length) {
    i++;
}
if (i == booleans.length) {
    alert('none');
}

// Filter would work like this:
var bools = booleans.filter(function (val, ind) { return !val; });
if (bools.length  > 0) alert('none');

You can read more about filter() here: https://msdn.microsoft.com/de-de/library/ff679973(v=vs.94).aspx

Other possibilities are using .every() as @Bergi mentioned.

edited Jan 18, 2017 at 19:56

answered Jan 18, 2017 at 18:44

Nils

9871 gold badge10 silver badges20 bronze badges

2 Comments

Nils Over a year ago

it is also possible to use array functions such as filter, if your javascript edition supports it, but I did not show this here.

Philip Eagles Over a year ago

could you explain the filter function you mention? I'm not entirely sure what you mean by it.

ahuemmer · Accepted Answer · 2025-02-27 10:27:23Z

0

var boolean1 = true,
    boolean2 = true,
    boolean3 = false;

if(boolean1 || boolean2 || boolean3) {

    if (boolean1) {
        alert("boolean1");
    }
    if (boolean2) {
        alert("boolean2");
    }
    if (boolean3) {
        alert("boolean3");
    }

} else {
    alert("none");
}

When all booleans are false, it checks only one condition and runs the else part directly. If true, it checks four if-conditions, that are also checked on starting, so it is the better one.

edited Feb 27 at 10:27

ahuemmer

2,12717 gold badges30 silver badges38 bronze badges

answered Feb 27 at 4:48

Muskan Girdhar

1

Comments

Akash Bhatt · Accepted Answer · 2025-02-27 10:40:00Z

0

you can use this alternative approach
    var boolean1 = true,
  boolean2 = true,
  boolean3 = false;

let anyConditionMet = false;

if (boolean1) {
  alert("boolean1");
  anyConditionMet = true;
}
if (boolean2) {
  alert("boolean2");
  anyConditionMet = true;
}
if (boolean3) {
  alert("boolean3");
  anyConditionMet = true;
}

if (!anyConditionMet) {
  alert("none");
}

answered Feb 27 at 10:40

Akash Bhatt

314 bronze badges

2 Comments

Lã Ngọc Hải Feb 28 at 4:04

This is duplicated to Quentin's answer

Akash Bhatt May 20 at 7:19

yes i think both are almost same didn't notice that

Teddy Sterne · Accepted Answer · 2017-01-18 18:59:35Z

-1

This is not super readable but this would work:

var boolean1 = true,
    boolean2 = true,
    boolean3 = false;

[(boolean1 && !alert("boolean1")),
    (boolean2 && !alert("boolean2")),
      (boolean3 && !alert("boolean3"))]
      .some(function(passed) {
        return passed;
      }) || alert("none");

edited Jan 18, 2017 at 18:59

answered Jan 18, 2017 at 18:40

Teddy Sterne

14.3k2 gold badges49 silver badges52 bronze badges

1 Comment

Bergi Over a year ago

Actually that doesn't work, as alert always returns undefined.

Erangad · Accepted Answer · 2017-01-18 19:09:28Z

-1

you can use a for loop to reduce the size of the code.

var boolean1 = true,
  boolean2 = true,
  boolean3 = false;
  none=true;
for(i=1;i<4;i++){
    var variable=window["boolean"+i];
    if (variable) {
      alert("boolean"+i);
      none=false;
    }
}

if (none) {
  alert("none");
}

answered Jan 18, 2017 at 19:09

Erangad

87911 silver badges18 bronze badges

Collectives™ on Stack Overflow

javascript if if else (not else if)

12 Answers 12

2 Comments

Comments

Comments

Comments

Comments

Comments

Comments

2 Comments

Comments

2 Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

12 Answers 12

2 Comments

Comments

Comments

Comments

Comments

Comments

Comments

2 Comments

Comments

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related