篮子onclick处理器

Question

我目前正在一篮子，但似乎无法得到clickHandler函数工作。当我按下按钮时，它总是会返回“data-id”4.任何建议？这里有一段代码和一个js小提琴。

basket.clickHandler = function(){ 
    var targets = document.getElementsByClassName("addButton"); 
    for(var i = 0; i < targets.length; i++) { 
     var target = targets[i]; 
     target.onclick = function() { 
      var shopId = target.getAttribute("data-id"); 
      var item = shop.items[shopId]; 

      basket.add(item); 
     }; 
    }; 
}; 

http://jsfiddle.net/g1L5mwk4/

Answer 1

的问题，也是一个通用的解决方案的说明可以在这里找到：JavaScript closure inside loops – simple practical example

在你的具体的例子，你应该通过this，而不是循环指的是元素本身变量：

var shopId = this.getAttribute("data-id");