0

我有这个 javascript 对象,其中包含城市列表以及状态的依赖项详细信息。

var cityList = {
'1' : ['CITY_CODE_A','City A', '2','11','2'],
'2' : ['CITY_CODE_B','City B','','11',''],
'3' : ['CITY_CODE_C','City C', '','11','']
};
var stateList = {
'11' : ['STATE_CODE_A', 'State A'],
'12' : ['STATE_CODE_B', 'State B'],
'13' : ['STATE_CODE_C', 'State C'],
'14' : ['STATE_CODE_D', 'State D']
};

根据选择的状态,键被传递给一个函数,该函数将键与城市列表中的值(数组中的索引 3 元素)匹配,如果找到,则形成一个选项标签<select>。这是代码:

$.each(cityList, function(key, value) {
  if(selectedVal == value[3]){
  $("select[id='"+selectid+"']").append('<option value="'+value[0]+'" code="'+key+'">'    +value[1] + '</option>'); 
  }
});

现在,当这个城市列表增长到 5 万条记录 (2 MB) 时,一个州有 5 万个城市(性能测试),上面的代码需要 2 分钟才能在本地运行代码的 Windows 7 机器上加载城市列表下拉列表。尝试使用 native for/ in-loop 但没有得到令人满意的结果。

关于如何减少时间的任何想法?我需要更改 JS 对象结构吗?

4

2 回答 2

0

我也是更好地使用搜索而不是 50k 滚动项的朋友。但要回答您的问题,我会使用以下方法改进您的代码:

var html = '';
$.each(cityList, function (key, value) {
    html += '<option value="' + value[0] + '" code="' + key + '">' + value[1] + '</option>'
});

$("select[id='1']").append(html);

这将大大加快您的option创建速度,因为 DOM 只会更改一次,而不是像您在示例中所说的 50k 次。(简短演示

于 2013-09-26T09:20:43.837 回答
-2

编辑:@downvoters,你能解释一下为什么你认为这是错误的吗?但首先尝试运行这个测试

如果您想获取特定州的城市,您可以创建一个以州 ID 为键的地图,

var cityList = {
   11 : [ ['CITY_CODE_A','City A', '2','11','2'], ['CITY_CODE_B','City B','','11',''] ],
   12 : [ ['CITY_CODE_C','City C', '2','12','2'], ['CITY_CODE_D','City D','','12',''] ]
}

现在你只需要迭代citylist[ selectedVal ]

$.each(cityList[selectedVal], function(key, value) {
  $("select[id='"+selectid+"']").append('<option value="'+value[0]+'" code="'+key+'">'    +value[1] + '</option>'); 
});

要进一步优化它,您应该避免<option>在循环中进行 DOM 操作(添加元素),而是将其缓冲:

var $selectEl = $('<select />');
$.each(cityList[selectedVal], function(key, value) {
  $selectEl.append('<option value="'+value[0]+'" code="'+key+'">'    +value[1] + '</option>'); 
});
$selectEl.attr( 'id', selectid );
$("select[id='"+selectid+"']").replaceWith( $selectEl );
于 2013-09-26T08:43:16.007 回答