我有一个名为 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)之一?
最佳答案
new
没有害处。一遍又一遍地重复。
如果使用得当,new
效果很好。就像在 Backbone 中一样。
JSLint 是一个非常自以为是的 linter;仅仅因为 Crockford 说某些东西不好或不应该使用,并不意味着这是普遍真理。另外,他背后的推理更多的是最终掩盖了更经典的 oo 外观背后的 Javascript 原型(prototype)性质。
但是,如果你使用像 JSHint 这样的工具,你可以配置这些警告。
我更喜欢你的两个声明中的后者
$(function() {
var instantGarbage = new MainControllerView();
});
您可以使用 JSHint 的 unused: false
选项来抑制它。
或者您可以改为设置 nonew: false
。
这些可以在 .jshintrc
中设置,甚至可以通过注释在每个文件中设置:
/* jshint unused: false */
在文件的顶部。这将为文件关闭它。您甚至可以针对特定范围禁用它。
关于javascript - Backbone.js "use new for side effects"与 JSHint 是否相反?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/21170974/