为了账号安全,请及时绑定邮箱和手机立即绑定

在循环中添加'click'事件侦听器

在循环中添加'click'事件侦听器

汪汪一只猫 2019-08-13 15:14:52
在循环中添加'click'事件侦听器将onClickhtml标签中的标准重构为监听器,遇到了我的代码问题:var td;     for (var t=1;t<8;t++){         td = document.getElementById('td'+t);         if (typeof window.addEventListener==='function'){                 td.addEventListener('click',function(){                     console.log(td);             })}  }当td单击元素时,假设td用循环中的最后一个索引单击,例如7看起来像是eventListeners仅填充了此循环中的最后一个元素。循环初始化看起来正确 为什么会这样?这是实时代码
查看完整描述

3 回答

?
小唯快跑啊

TA贡献1863条经验 获得超2个赞

您需要将事件侦听器的赋值包装在闭包中,例如:

var td;for (var t = 1; t < 8; t++){
    td = document.getElementById('td'+t);
    if (typeof window.addEventListener === 'function'){
        (function (_td) {
            td.addEventListener('click', function(){
                console.log(_td);
            });
        })(td);
    }}


查看完整回答
反对 回复 2019-08-13
?
梦里花落0921

TA贡献1772条经验 获得超6个赞

发生了什么

变量td是在事件处理程序之外定义的,因此当您单击单元格时,您将记录它设置的最后一个值。

更技术上:每个事件处理函数都是一个闭包 - 一个引用外部作用域变量的函数。

一般的解决方案

这类问题的一般解决方案是从包装函数返回事件处理程序,将要“修复”的变量作为参数传递:

td.addEventListener('click', function(wrapTD) {
    return function() { console.log(wrapTD); }}(td));

参数现在绑定到调用的包装函数的范围。

更简单的解决方案:使用 this

然而,有一个更简单的选择。在事件处理程序中,this设置为定义处理程序的元素,因此您可以使用this而不是td

td.addEventListener('click', function() {
    console.log(this);});

更简单:没有循环!

最后,您可以for完全摆脱循环并在整个表上设置单个事件处理程序:

var table = document.getElementById('my-table');table.addEventListener('click', function(e) {
    if (e.target.nodeName.toUpperCase() !== "TD") return;

    var td = e.target;
    console.log(td);});

对于较大的表,这是一个更好的解决方案,因为您只用一个替换多个事件处理程序。请注意,如果将文本包装在另一个元素中,则需要对其进行调整以检查目标元素是否为a的后代td


查看完整回答
反对 回复 2019-08-13
?
万千封印

TA贡献1891条经验 获得超3个赞

所以这里发生的是你将变量'td'保留在事件监听器函数的范围内。只有1个'td'实例,每次for循环迭代时都会更新。因此,当for循环结束时,td的值现在设置为元素'#td7',并且您的事件处理程序只是记录td的当前值。

在上面的示例中,您只需记录'this':

var td;for (var t=1;t<8;t++){
    td = document.getElementById('td'+t);
    if (typeof window.addEventListener==='function'){
      td.addEventListener('click',function(){
        console.log(this);
      });
    }}

因为'this'被设置为一个事件被绑定的元素,用于执行事件处理程序。

我猜你在寻找关于在for循环中创建闭包时保持迭代器的更多答案。为此,您需要在for循环之外定义一个函数。

for (var t=1;t<8;t++){
  bind_event(t);}function bind_event(t) {
    var td = document.getElementById('td'+t);
    if (typeof window.addEventListener==='function'){
      td.addEventListener('click',function(){
        console.log(td);
      });
    }}

这样,每次运行bind_event时都会创建一个名为'td'的变量实例,并且该实例将保留在事件监听器函数的闭包中。值得注意的是bind_event中的't'也是一个新变量。


查看完整回答
反对 回复 2019-08-13
  • 3 回答
  • 0 关注
  • 388 浏览
慕课专栏
更多

添加回答

举报

0/150
提交
取消
意见反馈 帮助中心 APP下载
官方微信