Javascript 简化我的函数（循环、数组）？

我有一个重复了几次的函数，我相信可以简化并从数组发送变量

var i = masterdata.timing.split(',');
        var index = 0;
        for (index = 0; index < i.length; ++index) {
            $("#timing_" + i[index].trim()).prop('checked', true);
        }

    var i = masterdata.concern.split(',');
    var index = 0;
    for (index = 0; index < i.length; ++index) {
        $("#concern_" + i[index].trim()).prop('checked', true);
    }

    var i = masterdata.steps.split(',');
    var index = 0;
    for (index = 0; index < i.length; ++index) {
        $("#steps_" + i[index].trim()).prop('checked', true);
    }

---

你的想法应该行得通：

var i;
var index = 0;
var j = 0;
var chkgroup = ['timing', 'concern', 'steps'];
var currentGroup;

for (j = 0; j < chkgroup.length; ++j) {
    currentGroup = chkgroup[j];

    i = masterdata[currentGroup].split(',');

    for (index = 0; index < i.length; ++index) {
        $("#" + currentGroup + "_" + i[index].trim())
            .prop('checked', true);
    }        
}

请注意，中没有为…定义顺序，因此如果需要保证以特定顺序迭代对象属性，最好使用预定义数组

您还可以使用

$来获得乐趣。map

：

var values = $.map(masterdata, function (i, currentGroup) {
    return $.map(i.split(','), function (val) {
        return $('#' + currentGroup + '_' + val.trim());
    });
});

$(values).prop('checked', true);

var chkgroup=[‘计时’、‘关注’、‘步骤’]；
setProps（chkgroup，masterdata）；
函数setProps（c，m）{
$。每个（c，功能（i，组）{
VarI=m[组]分裂（'，'）；
var指数=0；
对于（索引=0；索引

事实上，您的代码仍然有点嘈杂且不可读，即开发人员必须花费至少几分钟的时间对其进行分析，以了解逻辑是否正确。使用loDash或下划线，可以简化代码并将其转换为：

   var selector = _.chain(masterdata)
        //pick only needed items 
        .pick(masterdata, 'timing', 'concern', 'steps')
        .map(function (item, key) {
            //split and transform the string into selector, e.g. #timing_1
            var ids = item.split(',');
            var mapped = _.map(ids, function (id) { return "#" + key + "_" + id.trim(); });
            return mapped.join();
        })
        .value()
        .join();
    $(selector).prop('checked', true);

---

和var i中的工作示例，masterdata.Timeing随类别而变化。@triplethreat77:已更新，请查看

var values = $.map(masterdata, function (i, currentGroup) {
    return $.map(i.split(','), function (val) {
        return $('#' + currentGroup + '_' + val.trim());
    });
});

$(values).prop('checked', true);

var chkgroup = [ 'timing', 'concern', 'steps' ];

setProps( chkgroup, masterdata );

function setProps( c, m ) {
    $.each(c, function(i, group) {
        var i = m[group]split(',');
        var index = 0;
        for (index = 0; index < i.length; ++index) {
            $("#" + group + "_" + i[index].trim()).prop('checked', true);
        }
    });
}

   var selector = _.chain(masterdata)
        //pick only needed items 
        .pick(masterdata, 'timing', 'concern', 'steps')
        .map(function (item, key) {
            //split and transform the string into selector, e.g. #timing_1
            var ids = item.split(',');
            var mapped = _.map(ids, function (id) { return "#" + key + "_" + id.trim(); });
            return mapped.join();
        })
        .value()
        .join();
    $(selector).prop('checked', true);