10 回答
TA贡献1804条经验 获得超2个赞
window.onload = function () { var d = document.getElementsByTagName('a'); // 假设现在有5个 a标签 for (var i = 0; i < d.length; i++) { d[i].onclick = function () { console.log(i) // 由于异步的关系 i 现在是5 , 因为绑定的方法中使用的i for循环之后的i d[i].innerHTML += d[i].innerHTML; } } // 利用闭包 for (var i = 0; i < d.length; i++) { d[i].onclick = (function (i) { return function () { console.log(i) // d[i].innerHTML += d[i].innerHTML; } })(i) } // 将每个i 保存到d[i]中的 index 属性中, 利用this.index 获取到 对应的 i for (var i = 0; i < d.length; i++) { d[i].index = i; d[i].onclick = function () { console.log(this.index) d[this.index].innerHTML += d[this.index].innerHTML; } } };
首先 , JavaScript是没有块级声明的概念的, 这样导致了 for循环中 这个i 是一直使用同一个变量的,而因为我们点击事件绑定的方法并没有将当时循环中的 i 保存下来,导致方法用的i 都是for循环之后的i 变量,加入循环了5次, 最终变量 i 已经变为5了, 而此时我们 使用的di 是没有这个元素的,因此控制台给我们报错了, 通过闭包,或者将循环中的i 保存到d[i]本身的属性里面, 都能解决你的问题, 最后还是得靠你自己去理解闭包 和 this的概念
TA贡献1804条经验 获得超8个赞
事件执行时,for循环早就结束了,这个时候i已经是最大值加1了。
解决方法是可以用闭包把每次循环i的值保存起来。
或者在事件函数内不使用循环变量i,而是用this获取触发事件的元素
document.getElementById("td0").innerHTML+=this.innerHTML;
TA贡献1810条经验 获得超5个赞
楼主如果改用jquery会简单很多。
新版本的jquery使用这种写法
$( "#dataTable tbody tr" ).on( "click", function() {
console.log( $( this ).text() );
});
TA贡献1752条经验 获得超4个赞
这位施主,你的函数出问题了
问题代码:
d[i].onclick = function() {
$("td0").innerHTML += d[i].innerHTML
}
第一个 d[i] ,这个 i 是for循环中的参数 i,没问题
第二个 d[i] ,这个 i 是那个 i ?
我知道你意思,这个d[i],是当前循环到的元素,可是在他并不能传递过去。
解决方法:
你使用this关键字替代onlcick函数中的 d[i] ,代码运行成功
添加回答
举报