javascript - 是否可以对多个 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'); 并使用比较运算符 === 和 !== 分别代替 == 和 !=。

"The cyclomatic complexity of a section of source code is the count of the number of linearly independent paths through the source code." -- 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]();
        }
    });