命名空间模式导致 JSLint '函数在定义之前被使用' 错误答案

【问题标题】：Namespacing pattern causes JSLint 'function was used before it's defined' error命名空间模式导致 JSLint '函数在定义之前被使用' 错误
【发布时间】：2012-05-18 16:57:48
【问题描述】：

我正在使用以下命名空间模式：

var MyNamespace = new function () {
    var foo = function () {
        bar();
    };
    var bar = function () {
        alert("bar");
    };
    this.init = function () {
        foo();
    };
};

$(document).ready(function() {
    MyNamespace.init();
});

JSLint 抱怨 bar 在定义之前就被使用了。然而foo 直到bar 被声明后才会被调用。该代码适用于我尝试过的所有浏览器：http://jsfiddle.net/jDKvz/

模式是每个How do I declare a namespace in JavaScript?，第二个答案。

我需要在这里修复什么，还是应该忽略 JSLint？

【问题讨论】：

好吧，如果你颠倒两个函数的顺序，JSLint 会很高兴，但你是否“需要”这样做是主观的。我不同意 JSLint 的所有观点，但在这种情况下，我可能只是更改代码以使其关闭。
@nnnnnn 我的真实项目要复杂得多，例如可能包含相互调用的函数。
在这种情况下，我很乐意在这一点上忽略 JSLint。虽然我可能会尝试使用 function foo() ... 语法而不是 var foo = function()... 来查看它是否有任何不同，因为必须记住你必须真正关心的 lint 警告/错误是很烦人的......
@Andrew 你用的是什么版本的 JSLint？ hosted version 抱怨 new function 是一个“奇怪的结构”，但没有说 bar 是未定义的。

标签： javascript namespaces jslint javascript-namespaces

【解决方案1】：

我怀疑这是因为提升，变量和函数声明被解释器提升到顶部，很可能是它的看法：

var MyNamespace = new function () {
    var foo;
    var bar;

    foo = function () {
        bar();
    };

    bar = function () {
        alert("bar");
    };

    this.init = function () {
        foo();
    };
};

现在foo = function () {bar();}; 内部的bar() 尚未被解析，它只是一个变量bar，而不是此时要调用的函数。

话虽如此，如果您的代码运行良好，您可以继续使用它，打开 strict mode 也很有帮助。

【讨论】：

【解决方案2】：

bar 是一个函数表达式而不是一个函数声明，这意味着它只有在被赋值后才能工作。您可以改用函数声明，jslint 不会再抱怨了。

function bar() {
    ...
}

另外，new Function 是不必要的，function 就足够了。

编辑：

似乎在这种情况下摆脱new 使用自动调用函数(function(){}()) 或对象文字{}

【讨论】：

它调用以下函数作为构造函数。
(function(){ ... }())怎么样
这个也可以，也是我喜欢的，不过这里也可以用new。
(function(){ ... }()) 在这里不起作用，构造函数什么都不返回。
关于此答案中的第一句话，问题中的代码按原样工作，因为bar 是在实际使用时分配的。这里的问题是有效的代码和 JSLint 完全满意的代码之间的区别。

【解决方案3】：

编辑：认为 JS 关闭了值，而不是变量。

JSLint 正在抱怨，因为它没有看到变量提升正在进行。只需在使用它们的函数之前定义其他函数使用的函数。它有助于使您的代码更具可读性，并关闭 JSLint。

例如：

var MyNamespace = new function () {
    var bar = function () {
        alert("bar");
    };

    var foo = function () {
        bar();
    };

    this.init = function () {
        foo();
    };
};

【讨论】：

“闭包规则”？再想一想。 “对未定义的值关闭...” - 对变量而不是值的关闭。问题中显示的代码将起作用，因为bar 是由调用foo() 的时间定义的。反转函数应该会关闭 JSLint，但原来的方式其实并没有错。
好吧，将答案更改为关于可读性，而不是关于错误的关闭内容。我不会告诉别人使用函数语句。