Question

可能重复：
Assign click handlers in for loop

我在javascript中有点天真。我有这个javascript代码应该在按钮点击运行。按钮是在循环中动态生成的。我期待的是每次点击按钮都会收到不同的变量。但我收到的始终是数组中的最后一个变量。请参阅下面的代码。有没有人在这里看到一些明显错误的东西？

<!doctype html>
<html> 
<head>      
<script  type="text/javascript"> 

var myCars=["Saab","Volvo","BMW"]; 

    function loadCars() {
var carList = document.getElementById('mydiv');

for(var i=0;i < myCars.length ; i++ ){

//Append button
        var el = document.createElement("button");
        var car = myCars[i];
        el.name = "view_car";
        el.id = "view_car";
        el.value = "View car";
        el.type = "button";
        el.title = "View car "+i;
        el.innerHTML = "View car "+car;
        el.onclick = function() { 
            showMe(car,i);
        };

    carList.appendChild(el);
}

}

function showMe(carName,index){

    alert("Selected car="+carName +" , index="+index);
}

    </script>

</head>

<body  onload="loadCars()">

Cars will be loaded here 
<div  id="mydiv" class="carlist"></div>

</body>
</html>

Button click output

Answer 1

没有明显的错误。但是，您在car函数中使用了i和onclick。两者都取决于外部car和i并受到操纵。您必须使用闭包来破坏此引用：

    el.onclick = (function(carname,index){
        return function() { 
            showMe(carname,index);
        };
    })(car,i);

将Javascript函数分配给按钮单击的问题

1 个答案: