javascript - Backbone.js "use new for side effects"与 JSHint 是否相反？

Question

我有一个名为 MainControllerView 的自包含的 Backbone.View 实现，它可以自行处理(即，没有理由对其进行外部引用。)。如果，在我的主要 Bootstrap 函数中，我像这样开始:

$(function() {
  new MainControllerView();
});

JSLint/JSHint 提示我正在使用“new for side effects”。阅读这个警告表明上面的代码被认为是臭代码。替代方案是根本不使用 new 并仅将构造函数作为函数调用，或者将其分配给变量。但是，不使用 new 直接将我的 MainControllerView() 作为函数调用会在主干代码中引发错误，因此这显然不是一种选择。在我看来，以下是更好的代码似乎是完全错误的:

$(function() {
  var instantGarbage = new MainControllerView();
}); 

事实上，这引发了一个不同的 JSLint 警告“instantGarbage 已定义但从未使用过。”显然，如果我这样做，我就很危险，如果我不这样做，我也很危险。那么，是否有不同的“正确”方式来处理这类事情呢？创建即时垃圾变量是否以某种方式成为“更好的代码”替代方案？ Backbone.js 是否以未经 Crockford 批准的方式利用“new”关键字？或者这只是“规则”的异常(exception)之一？

Answer 1

new 没有害处。一遍又一遍地重复。

如果使用得当，new 效果很好。就像在 Backbone 中一样。

JSLint 是一个非常自以为是的 linter；仅仅因为 Crockford 说某些东西不好或不应该使用，并不意味着这是普遍真理。另外，他背后的推理更多的是最终掩盖了更经典的 oo 外观背后的 Javascript 原型(prototype)性质。

但是，如果你使用像 JSHint 这样的工具，你可以配置这些警告。

我更喜欢你的两个声明中的后者

$(function() {
  var instantGarbage = new MainControllerView();
}); 

您可以使用 JSHint 的 unused: false 选项来抑制它。

或者您可以改为设置 nonew: false。

这些可以在 .jshintrc 中设置，甚至可以通过注释在每个文件中设置:

/* jshint unused: false */

在文件的顶部。这将为文件关闭它。您甚至可以针对特定范围禁用它。

查看 jshint documentation