2

我是 Javascript 的新手。我想将 onclick 事件添加到表行。我没有使用 JQuery。

我遍历行并使用闭包来确保我拥有每一行的外部函数的状态。循环工作。使用警报，我看到为每次迭代分配的功能。但是当我单击该行时，不会显示任何警报。下面是可以加载的 HTML 和代码。

为什么表格行事件不起作用？

<!doctype html>
<html lang="en">
<body>
<script>
function example4() {
    var table = document.getElementById("tableid4");
    var rows = table.getElementsByTagName("tr");
    for (var i = 0; i < rows.length; i++) {
        var curRow = table.rows[i];
        //get cell data from first col of row
        var cell = curRow.getElementsByTagName("td")[0];
        curRow.onclick = function() {
            return function() {
                alert("row " + i + " data="+ cell.innerHTML);
            };
        };
    }
}
function init() { example4(); }
window.onload = init;   
</script>
<div>
Use loop to assign onclick handler for each table row in DOM. Uses Closure.
    <table id="tableid4" border=1>
      <tbody>
        <tr><td>Item one</td></tr>
        <tr><td>Item two</td></tr>
        <tr><td>Item three</td></tr>
      </tbody>
    </table>
</div>
</body>
</html>
4

3 回答 3

9

这似乎是规范的方式

演示

function example4() {
    var table = document.getElementById("tableid4");
    var rows = table.rows; // or table.getElementsByTagName("tr");
    for (var i = 0; i < rows.length; i++) {
        rows[i].onclick = (function() { // closure
            var cnt = i; // save the counter to use in the function
            return function() {
              alert("row"+cnt+" data="+this.cells[0].innerHTML);
            }    
        })(i);
    }
}
window.onload = function() { example4(); }​

更新：@ParkerSuperstar 建议不需要 (i) 中的 i。我没有对此进行测试，但他的小提琴似乎有效。

于 2012-12-26T18:04:47.747 回答
1

我不太清楚你为什么在这里使用闭包，你能更详细一点吗？

您没有看到所需警报的原因是在 onclick 函数中，您正在返回另一个函数。IE：

window.onload = function() {
    return function() {
        alert("Closure... why?");
    };
};

像这样的东西不会真正起作用，因为你从来没有调用嵌套函数......在不使用闭包的情况下尝试它，或者评论解释你为什么想要一个闭包，因为你的解释对我来说没有多大意义。

于 2012-12-26T18:03:23.947 回答
0

你只需要删除一个额外的函数，脚本就会像这样

<script>
 function example4() {
    var table = document.getElementById("tableid4");
cells = table.getElementsByTagName('td');

 for (var i=0,len=cells.length; i<len; i++){
 cells[i].onclick = function(){
    alert(this.innerHTML);

}
 }
}
 function init() { example4(); }
 window.onload = init;   
 </script>
于 2012-12-26T18:10:19.633 回答