Как «это» можно сократить?

Question

[Rick]

Пример:

function getMod(m1, m2, m3) {
    $('#a1').hide();
    $('#a2').hide();
    $('#a3').hide();

    if (m1 == true) {
        $('#a1').show();
    }
    if (m2 == true) {
        $('#a2').show();
    }
    if (m3 == true) {
        $('#a3').show();
    }
}

Comments

[Роман Якушев]

и вы забыли тег jquery

Answer 1

[aen]

function getMod(m1, m2, m3) {
    $('#a1')[m1 ? 'show' : 'hide']();
    $('#a2')[m2 ? 'show' : 'hide']();
    $('#a3')[m3 ? 'show' : 'hide']();
}

Answer 2

[Тагир Валеев]

Код изначально кривой, но можно как-нибудь так:

function getMod(m1, m2, m3) {
    var map = {'#a1':m1, '#a2':m2, '#a3':m3}
    for(var id in map) {
      $(id)[map[id]?'show':'hide']();
    }
}

Comments

[aen]

hasOwnProperty забыли?

[Rick]

знаю что кривой, поэтому и спрашиваю у гуру :)

Answer 3

[Anatoliy Mikhailov]

var a=['a1','a2','a3'];

function getMod() {
   for(var i=0;i<a.length;i++){
     if(arguments[i]===true)$('#'+a[i]).show()
     else $('#'+a[i]).hide()
  }
}

Answer 4

[index0h]

function getModElement(selector, value) {
    $(selector)[value? 'show' : 'hide']();
}

function getMod(m1, m2, m3) {
    getModElement('#a1', m1);
    getModElement('#a2', m2);
    getModElement('#a3', m3);
}

Answer 5

[Роман Якушев]

Использовать общий класс к примеру:

function getMod(m1, m2, m3) {
    $('.a').hide();

    if (m1 == true) {
        $('.a').eq('1').show();
    }
    if (m2 == true) {
        $('.a').eq('2').show();
    }
    if (m3 == true) {
        $('.a').eq('3').show();
    }
}