问问题
158 次
0

下面的 Jquery 脚本需要很长时间。请给我解决方案,以便提高性能

为了

(i = 0; i < fooLength; ++i) {


  if (foo[i] != "") {

  var option = foo[i].split("$");

    if (option.length == 2) {

     if ($("#" + destination).find("option[value='" + option[1] + "']").length == 0)
     {

       $("#" + destination).append($('<option></option>').val(option[1]).html(option[0]));

       $("#" + source).find("option[value='" + option[1] + "']").remove();

     }

    }

   }

  }

我正在尝试将该值附加到列表框中。那里有近 5000 条记录,因此它迭代了 5000 次并且需要很长时间

4

2 回答 2

0

无需在每个循环中都选择元素:

var $dest = $("#" + destination), $src = $("#" + source);
for (i = 0; i < fooLength; ++i) {
    if (foo[i] != "") {
        var option = foo[i].split("$");
        if (option.length == 2) {
            if ($dest.find("option[value='" + option[1] + "']").length == 0) {
                $dest.append($('').val(option[1]).html(option[0]));
                $src.find("option[value='" + option[1] + "']").remove();
            }
        }
    }
}
于 2013-10-21T12:13:23.367 回答
0

我已经尝试过了,但没有太大改善。所以通过分析我发现 find() 需要更多时间。我用下面的代码替换了这段代码,并且有了很大的改进。

for (i = 0; i < fooLength; ++i) {
if (foo[i] != "") {
var option = foo[i].split("$");
if (option.length == 2) {

if ($dest.each(function(){("option[value='" + option[1] + "']").length == 0;})){
   $dest.append($('<option></option>').val(option[1]).html(option[0]));
   $src.each(function(){if($(this).val()=="option[value='" + option[1] + "']")$(this).remove();});

                }
            }
        }
    }
于 2013-10-22T08:31:16.127 回答