jQuery: лучшая практика для заполнения выпадающего? - PullRequest
261 голосов
/ 02 мая 2009

Пример, который я вижу опубликованным все время, кажется неоптимальным, потому что он включает в себя конкатенацию строк, что, похоже, не является jQuery. Обычно это выглядит так:

$.getJSON("/Admin/GetFolderList/", function(result) {
    for (var i = 0; i < result.length; i++) {
        options += '<option value="' + result[i].ImageFolderID + '">' + result[i].Name + '</option>';
    }
});

Есть ли лучший способ?

Ответы [ 15 ]

427 голосов
/ 04 мая 2009

Андреас Греч был довольно близко ... на самом деле this (обратите внимание на ссылку this вместо элемента в цикле):

var $dropdown = $("#dropdown");
$.each(result, function() {
    $dropdown.append($("<option />").val(this.ImageFolderID).text(this.Name));
});
67 голосов
/ 02 мая 2009
$.getJSON("/Admin/GetFolderList/", function(result) {
    var options = $("#options");
    //don't forget error handling!
    $.each(result, function(item) {
        options.append($("<option />").val(item.ImageFolderID).text(item.Name));
    });
});

То, что я делаю выше, - это создание нового элемента <option> и добавление его в список options (при условии, что options - это идентификатор выпадающего элемента.

PS Мой javascript немного ржавый, поэтому синтаксис может быть не идеальным

40 голосов
/ 02 мая 2009

Конечно - сделайте options массивом строк и используйте .join('') вместо += каждый раз в цикле. Небольшое снижение производительности при работе с большим количеством опций ...

var options = [];
$.getJSON("/Admin/GetFolderList/", function(result) {
    for (var i = 0; i < result.length; i++) {
        options.push('<option value="',
          result[i].ImageFolderID, '">',
          result[i].Name, '</option>');
    }
    $("#theSelect").html(options.join(''));
});

Да. Я все еще работаю со строками все время. Верьте или нет, это самый быстрый способ создания фрагмента DOM ... Теперь, если у вас есть только несколько вариантов, это не будет иметь большого значения - используйте технику Dreas демонстрирует , если вам нравится стиль , Но имейте в виду, что вы вызываете внутренний анализатор HTML браузера i*2 раз, а не один раз, и модифицируете DOM каждый раз в цикле ... с достаточным количеством опций. в конечном итоге вы заплатите за это, особенно в старых браузерах.

Примечание: Как указывает справедливость, это развалится, если ImageFolderID и Name не не закодированы должным образом ...

21 голосов
/ 19 апреля 2013

Или, может быть:

var options = $("#options");
$.each(data, function() {
    options.append(new Option(this.text, this.value));
});
17 голосов
/ 01 июля 2009

Самый быстрый способ это:

 $.getJSON("/Admin/GetFolderList/", function(result) {
        var optionsValues = '<select>';
        $.each(result, function(item) {
            optionsValues += '<option value="' + item.ImageFolderID + '">' + item.Name + '</option>';
        });
        optionsValues += '</select>';
        var options = $('#options');
        options.replaceWith(optionsValues);
    });

В соответствии с эта ссылка является самым быстрым способом, потому что вы заключаете все в один элемент при выполнении любого вида вставки DOM.

11 голосов
/ 16 марта 2016

Я обнаружил, что это работает с сайта jquery

$.getJSON( "/Admin/GetFolderList/", function( data ) {
  var options = $("#dropdownID");
  $.each( data, function(key, val) {
    options.append(new Option(key, val));
  });
});
4 голосов
/ 06 мая 2016

Другой подход с ES6

fetch('https://restcountries.eu/rest/v1/all')
  .then((response) => {
    return response.json()
  })
  .then((countries) => {
    var options = document.getElementById('someSelect');
    countries.forEach((country) => {
      options.appendChild(new Option(country.name, country.name));
    });
  })
4 голосов
/ 04 марта 2016

Я читал, что использование фрагментов документа эффективно, поскольку позволяет избежать перевыполнения страницы при каждой вставке элемента DOM, а также хорошо поддерживается всеми браузерами (даже IE 6).

var fragment = document.createDocumentFragment();

$.each(result, function() {
  fragment.appendChild($("<option />").val(this.ImageFolderID).text(this.Name)[0]);
});

$("#options").append(fragment);

Впервые я прочитал об этом в Курсе наилучшей практики JavaScript CodeSchool .

Вот сравнение разных подходов , спасибо автору.

2 голосов
/ 15 марта 2012
$.get(str, function(data){ 
            var sary=data.split('|');
            document.getElementById("select1").options.length = 0;
            document.getElementById("select1").options[0] = new Option('Select a State');
            for(i=0;i<sary.length-1;i++){
                document.getElementById("select1").options[i+1] = new Option(sary[i]);
                document.getElementById("select1").options[i+1].value = sary[i];
            }
            });
2 голосов
/ 02 мая 2009

Я использую плагин выбора jquery. Это превращает ваш пример в:

$('#idofselect').ajaxAddOption('/Admin/GetFolderList/', {}, false);
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...