我有一个重复几次的函数,我相信可以从数组中简化和发送变量。
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);
}
也许只是将类别更改为变量并从数组中发送catergories?
var chkgroup = [
'timing, concern, steps'
]
答案 0 :(得分:4)
你的想法应该可以正常工作:
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);
}
}
如果chkgroup
数组与keys
中的对象masterdata
非常匹配,则可以改为使用外部for..in
循环:
var i;
var index = 0;
var currentGroup;
for (currentGroup in masterdata) {
i = masterdata[currentGroup].split(',');
for (index = 0; index < i.length; ++index) {
$("#" + currentGroup + "_" + i[index].trim())
.prop('checked', true);
}
}
请注意,for...in
没有定义订单,因此如果您需要保证按特定顺序迭代对象属性,最好使用预定义的阵列。
你也可以使用$.map
:
var values = $.map(masterdata, function (i, currentGroup) {
return $.map(i.split(','), function (val) {
return $('#' + currentGroup + '_' + val.trim());
});
});
$(values).prop('checked', true);
答案 1 :(得分:0)
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);
}
});
}
答案 2 :(得分:0)
事实上,您的代码仍然有点嘈杂且无法读取,即开发人员必须花费至少几分钟来分析它,以便了解逻辑是否正确。通过使用loDash或Underscore,您的代码可以简化并转化为:
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);
上制作样本