Use jquery IF statement for two different IDs

Question

this is probably dead simple but I just can find a solution myself. I have this code:

      $('article li.active').each(function() {
        if ($(this).attr('id') == 'arrival-and-departure') {
          $('p.quot').hide();
        }
        else if ($(this).attr('id') == 'additional-needs') {
          $('p.quot').hide();
        }
        else {$('p.quot').show()}
      };

I was wondering how I could combine the two IFs so that I just need IF and Else. Any help would be very welcome, thanks!

you should use OR in you first if statement if v1 == "test" || v1 =="other"){...} — simon
– simon, Commented Nov 24, 2016 at 14:48

mvc_help · Accepted Answer · 2016-11-24 14:38:10Z

3

$('article li.active').each(function() {
    if ($(this).attr('id') === 'arrival-and-departure' || $(this).attr('id') === 'additional-needs') {
        $('p.quot').hide();
    }
    else {
        $('p.quot').show();
    }
});

The double-pipe operator acts as an "OR" operator. So check if it's "this" OR "that".

answered Nov 24, 2016 at 14:38

mvc_help

1879 bronze badges

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

Comments

Shomz · Accepted Answer · 2016-11-24 14:37:45Z

2

You should use OR:

$('article li.active').each(function() {
    if ($(this).attr('id') == 'arrival-and-departure' || 
        $(this).attr('id') == 'additional-needs') 
    {
      $('p.quot').hide();
    }
    else {$('p.quot').show()}
  };

answered Nov 24, 2016 at 14:37

Shomz

37.7k4 gold badges61 silver badges87 bronze badges

Comments

Martijn Feld · Accepted Answer · 2016-11-24 14:38:45Z

2

You can use && (and) and || (or)

var a = true;
var b = false;
var c = true;
if(a&&b) //false since b=false, only a=true
if(a&&c) //true since a&c are both true
if(a||b)//a=true so>will be true. Javascript even won't check B (keep in mind when using functions!

More here: http://www.w3schools.com/js/js_comparisons.asp

answered Nov 24, 2016 at 14:38

Martijn Feld

313 bronze badges

Comments

Santiago Profumo · Accepted Answer · 2016-11-24 14:45:26Z

2

You could also use switch statement instead of the if with the || operator, like this:

$('article li.active').each(function() {
    switch ($(this).attr('id')) {
        case 'arrival-and-departure':
        case 'additional-needs':
            $('p.quot').hide();
            break;
        default:
            $('p.quot').show();
    }
});

answered Nov 24, 2016 at 14:45

Santiago Profumo

211 bronze badge

Collectives™ on Stack Overflow

Use jquery IF statement for two different IDs

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related