Что не так в этом коде jquery $ .each()?

Question

Javascript

$.each(['#clk','#clk1'], function() 
    { 
     $(this).click(function() { 
      alert("click") 
     }); 
    }); 

HTML

<a href="#" id="clk">click me</a> 
    <a href="#" id="clk1">click me</a> 

Нет окно предупреждения при щелчке ссылки. ОБНОВЛЕНИЕ:
У меня более 1 идентификатора. Я показал только один, чтобы упростить эту проблему.

Answer 1

Чтобы получить значение объекта String, вам необходимо использовать String.toString().

Непонятно, зачем нужен массив селекторов, но здесь два решения;

Ваше решение с использованием String.toString();

// Array of strings to be used as element selectors. 
var selectors = ['#element1', '#element2']; 

// Using $.each() 
$.each(selectors, function() { 
    // String.toString() returns the value of the String object. 
    var $this = $(this.toString()); 

    $this.click(function() { 
     console.log('Clicked element(1) =', this.id || this); // DEBUG 
    }); 
}); 

альтернативное решение с использованием String.join();

// Using String.join() 
$(selectors.join(',')).click(function(event) { 
    event.preventDefault(); // This is to not follow the link 

    // Notice that "this" now referes to the current/clicked element 
    // and not any string value from the "selectors" array. 
    console.log('Clicked element(2) =', this.id || this); // DEBUG 
}); 

Смотрите мой demo.

Если вам действительно не нужен массив селекторов, я бы порекомендовал простой multiple selector;

$('#element1, #element2').click(function() { ... }); 

Answer 2

Вы могли бы еще больше упростить его:

$("#clk").click (function() { 
    alert("click"); 
}); 

Answer 3

Во-первых, почему вы используете foreach конструкцию при переборе над идентификатором? Когда вы используете и id, предполагается, что это ТОЧНО ОДИН элемент с данным идентификатором. Таким образом, это должно быть прекрасно:

$("#clk").click(function() { 
    alert("click"); 
}); 

Во-вторых, каждый перебирает массив, ваш массив является #clk. Просто строка, ничего больше. Вся ваша функция получает два параметра: 0' (the index of the element) and the string # clk` (значение в этом индексе). Строка НЕ ​​разрешена для объекта JS.

Answer 4

IMO, чистый способ решить вашу постановку задачи будет:

$("a[id^='clk']").click(function() { 
     alert("click"); 
}); 

Селектор ^= бы выбрать все якоря, идентификатор начинается с «CLK» и связать функцию щелчка, чтобы все те, , Вы можете узнать больше об этом селекторе here