Что не так с этим jquery each() select options?

Question

У меня есть ниже функция,

function setDropdownValue(dropdownId,selectedValue){ 
      $('#'+dropdownId+' option').each(function(){ 
       if ($(this).val().toLowerCase()==selectedValue.toLowerCase()){ 
        //found the option i was looking for, do what i want 
        $(this).val(selectedValue); 
       } 
      }); 
     } 

И я передаю dropdownId=$('#dp1") и selectedValue='Bat' и по какой-то причине моей каждой функции() не получает выполненную .. Я звоню как этот setDropdownValue($("#dp1"),'Bat'); я упускаю что-то ??

Answer 1

dropdownId должно быть просто строкой, почему вы проходите $('#dp1"), который является объектом jQuery?

Вместо этого вы должны просто передать 'dp1'.

Answer 2

Попробуйте как

$('#dp1 option').each(function(){ 
      if ($(this).val().toLowerCase()==selectedValue.toLowerCase()){ 
       //found the option i was looking for, do what i want 
       $(this).val(selectedValue); 
      } 
     }); 

почему нужно использовать dropdownId .... иногда это приводит к некоторым противоречиям

Answer 3

Я думаю, что лучшим решением было бы написать функцию следующим образом, как это будет поддерживать ваш строчная логика

function setDropdown(id, value){ 
    var dropdown = $('#' + id); 
    dropdown.find('option').each(function(){ 
     if($(this).val().toLowerCase() == value.toLowerCase()){ 
      dropdown.val($(this).val()); 
      return; 
     } 
    }); 
} 

Я бы назвал эту функцию как так ...

setDropdown('myDropdownId', '0000-AABBCC'); 

Я склонен полагать, что вы должны установить «val» на выбор, а не на сами опции. Я уже видел причуды с этим методом.