在匿名函数中使用'this'，IDE：可能无效的用法

Question

在匿名函数中使用'this'，IDE：可能无效的用法

7

下一个实际可用的函数，从最佳实践角度来看是否不好？IDE正在警告我：

“潜在的无效使用'this'。检查JavaScript中'this'是否在同一闭包或外部内容中。”

$(document).on('change', '#select-all', function(){
    if( this.checked ) {
      $(this).closest('table').find('input[name="row-id"]').each( function() {
        this.checked = true; // Here
      })
    }
    else {
      $(this).closest('table').find('input[name="row-id"]').each( function() {
        this.checked = false; // Here
      });
    }
  });

当我勾选一个ID为select-all的复选框时，它会将所有其他复选框都标记为已选择。

- JorgeeFG

2个回答

3

@Jorge 这与JavaScript中闭包的作用域和this的使用有关。

如需进一步阅读，请参阅：http://javascriptplayground.com/blog/2012/04/javascript-variable-scope-this/

我没有完全阅读它，但它总结得相当不错。

- gmaliar

链接已于2016年2月失效。 - Perry Tew

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- VisioN · Accepted Answer

很可能是因为您的IDE不知道您所使用的函数中的对象this是什么，因此提示您this可能指向window对象或另一个上下文。

顺便说一下，您的代码可以重写为：

$(document).on("change", "#select-all", function() {
    $(this)
      .closest("table")
      .find("input[name='row-id']")
      .prop("checked", this.checked);
});