Question

下面的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次，需要很长时间

Answer 1

无需为每个循环选择元素：

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();
            }
        }
    }
}

Answer 2

我试过这个，但没有太大改进。所以通过分析我发现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();});

                }
            }
        }
    }

Jquery脚本花了很长时间

2 个答案: