是否可以对多个 array.indexOf 使用 switch case

Question

我有一些使用 javascript 的特定数组的 if 条件

if (activity.indexOf("strategy session") != -1) {
    $("#FoPStrategySession").show();
}

if (activity.indexOf("sessions") != -1) {
    $("#acprojectname").show();
    if (supportmodel == "Level") {
        $(".accombohide").hide();
        $("[title='Test']").val("NA");
        $("[title='Test2']").val("NA");
    }
}

if (activity.indexOf("virtual") != -1) {
    if (supportmodel == "Level") {
        $(".lvl3_consult").hide();
        $("[title='Test']").val("NA");
        $("[title='Test2']").val("NA");
    }
}

if (activity.indexOf("Other") != -1) {
    $("#acactivityother").show();
}

还有其他方法可以高效使用 switch case 或任何其他方法编写此代码吗？

Answer 1

无需多个if() 或switch() 语句。

您可以降低圈复杂度（现在是 7）并最终得到更好的代码。请注意，已经重构了一些 jQuery 选择器 $('[title="Test"], [title="Test2"]').val('NA'); 并分别使用比较运算符 === 和 !== 而不是 == 和 !=。

“一段源代码的圈复杂度是通过源代码的线性独立路径的数量的计数。” -- http://en.wikipedia.org/wiki/Cyclomatic_complexity

还创建了变量以避免 jQuery 多次在 DOM 中搜索相同的选择器。

代码：

var $foPStrategySession = $('#FoPStrategySession'),
    $acprojectname =  $('#acprojectname'),
    $titleTests = $('[title="Test"], [title="Test2"]'),
    $acactivityother = $('#acactivityother'),
    $accombohide = $('.accombohide'),
    $lvl3_consult = $('.lvl3_consult'),
    obj = {
        'strategy session': function () {
            $foPStrategySession.show();
        },
        'sessions': function () {
            $acprojectname.show();
            if (supportmodel === 'Level') {
                $accombohide.hide();
                $titleTests.val('NA');
            }
        },
        'virtual': function () {
            if (supportmodel === 'Level') {
                $lvl3_consult.hide();
                $titleTests.val('NA');
            }
        },
        'Other': function () {
            $acactivityother.show();
        }
    };

    Object.keys(obj).forEach(function (o) {
        if (activity.indexOf(o) !== -1) {
            obj[o]();
        }
    });

Answer 2

首先，您不应该重新查询 DOM：

var strategySession = $("#FoPStrategySession");
var acprojectname = $('#acprojectname');

// and so on..

稍后您将使用上面创建的引用访问这些 DOM 元素。

现在您可以使用Array.prototype.forEach 通过switch 来简化您的代码：

activity.forEach(function(act) {
    // I suggest you that you lower case each activity
    // to avoid further issues...
    switch(act.toLowerCase())
         case "strategy session":
              strategySession.show();
              break;

         // other cases...

         default:
             throw Error("Not supported activity");
    }
});

旁注：我知道activity 是一个数组。否则，您将使用相等运算符检查activity 是否是某个特定的string，我的答案应该需要一些重构。如果我误认为 activity 是 array，请注释掉我的答案...。