Jquery скрипт занимает очень очень много времени

Question

Jquery скрипт занимает очень очень много времени

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 раз и занимает много времени

Shabz 21 окт. 2013, в 14:44

Источник

2

Потому что вы делаете тонны DOM поиска и операций DOM. Научитесь хранить вещи в переменных вместо того, чтобы искать их раз за разом.
epascarello 21 окт. 2013, в 12:12
4

codereview.stackexchange.com
undefined 21 окт. 2013, в 12:12
0

записи очень большие ?? откуда они ? Можем ли мы увидеть пример того, как выглядят записи? Кроме того, насколько я вижу, вам понадобятся только некоторые записи, сколько из 5000 «действительны» более или менее? Я думаю, что если у вас есть 5000 записей в базе данных, запросите действительные записи, чтобы сузить их, а затем добавьте в список, чтобы сэкономить много времени.
aleation 21 окт. 2013, в 12:17
0

Лучше, если спросить на codereview.stackexchange.com
epascarello 21 окт. 2013, в 12:44

Показать ещё 2 комментария

Теги:

javascript

jquery

2 ответа

Ещё вопросы

Потому что вы делаете тонны DOM поиска и операций DOM. Научитесь хранить вещи в переменных вместо того, чтобы искать их раз за разом.
записи очень большие ?? откуда они ? Можем ли мы увидеть пример того, как выглядят записи? Кроме того, насколько я вижу, вам понадобятся только некоторые записи, сколько из 5000 «действительны» более или менее? Я думаю, что если у вас есть 5000 записей в базе данных, запросите действительные записи, чтобы сузить их, а затем добавьте в список, чтобы сэкономить много времени.
Лучше, если спросить на codereview.stackexchange.com

Shabz · Answer 1 · 2013-10-22T05-58-00.000Z

Я пробовал это, но улучшения не было. Поэтому по анализу я обнаружил, что 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();});

                }
            }
        }
    }

Doorknob · Answer 2 · 2013-10-21T09-23-00.000Z

Нет необходимости выбирать элементы каждый цикл:

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