如何优化此JavaScript重复项

elvis 发表于 Dev

猫王

我编写了这段代码，但是由于我刚刚开始学习JS，因此无法找出优化此代码的最佳方法。因此，对每个if语句都进行了重复。

$(function() {
    var lang = $(".lang input[type='checkbox']");
    var gender = $(".gender input[type='checkbox']");
    if(lang.length == lang.filter(":checked").length){
        $('.lang').hide();
        $('.lang-all').click(function(){
            $('.lang-all').hide();
            $('.lang').slideToggle(200);
        });
    } else {
        $('.lang').show();
        $('.lang-all').hide();
    }

    if(gender.length == gender.filter(":checked").length){
        $('.gender').hide();
        $('.gender-all').click(function(){
            $('.gender-all').hide();
            $('.gender').slideToggle(200);
        });
    } else {
        $('.gender').show();
        $('.gender-all').hide();
    }
});

所以这是我的代码，如您在第15行上看到的if（gender ...我有一个以前的代码的副本，只是将变量从“ lang”更改为“ gender”。由于我有两个以上的变量，因此不想为每个代码都重复编写代码，所以我希望有一个优化它的解决方案。

马文·梅德罗斯（Marvin Medeiros）

您可以编写一个函数来使您的代码更抽象，请参见：

function isChecked(obj, jq1, jq2){
    if(obj.length == obj.filter(":checked").length){
        jq1.hide();
        jq2.click(function(){
            jq2.hide();
            jq1.slideToggle(200);
        });
    } else {
        jq1.show();
        jq2.hide();
    }
}

//Your jQuery code, more abstract
$(function() {
    var lang = $(".lang input[type='checkbox']");
    var gender = $(".gender input[type='checkbox']");

    isChecked(lang, $('.lang'), $('.lang-all'));
    isChecked(gender, $('.gender'), $('.gender-all'));
});