关闭的合理用途？

Question

最近，我在公司的生产代码中看到了带有闭包函数的返回。如return (function() {...}()); 和我不喜欢看到这个，但我不是权威来源。我想我会问 StackOverflow 为什么以及何时使用这个好/坏。

* 注意 *

说你不关心命名空间污染，因为所有这些函数都已经关闭了。

示例 1a：

function foo(bar) {
    return {
        x: 1 + bar,
        y: 1 - bar,
        duration: (function() {
            var i = 0,
                len = 5;
            var results = 0;

            for (; i < bar; i++) {
                results += 1 + (results * bar);
            }

            return results;
        }())
    };
}

示例 1b：

function barProcess(bar) {
    var i = 0;
    var len = 5;
    var results = 0;

    for (; i < bar; i++) {
         results += 1 + (results * bar);
    }

    return results;
}

function foo(bar) {
    return {
        x: 1 + bar,
        y: 1 - bar,
        duration: barProcess(bar)
    };
}

示例 1c：

function foo(bar) {
    var i = 0;
    var len = 5;
    var results = 0;

    for (; i < bar; i++) {
         results += 1 + (results * bar);
    }

    return {
        x: 1 + bar,
        y: 1 - bar,
        duration: results
    };
}

观察：

示例 1a：
如果需要，内部函数会利用闭包。

示例 1b：
如果barProcess 可能需要一些闭包，这可能会使参数列表变长并且难以维护。

示例 1c：
每次调用没有额外的函数创建。 它最容易调试（在我看来）。

如果有人能给我一些技术原因，说明为什么应该使用示例 a、b 或 c，那就太好了。

---

所以，我蹩脚的答案并没有让我满意，所以我尝试了这个。 JS Perf Tests。我认为这让我的观察不会太远。

Answer 1

我认为不可能为每个简单案例在示例 a、b 和 c 之间选择一个明显的赢家。

在您给出的示例中，版本 (c) 已经足够好，所以我会使用它。也就是说，版本 (a) 将 i、len 和结果变量保持在更紧密的范围内，因此如果您想让“x”和“y”与持续时间“b”更分离，这绝对是一个好方法。

我不喜欢仅仅出于组织或范围的原因将事物拆分为单独的命名函数，就像您在示例 (b) 中所做的那样，因为它往往会使代码更加复杂且难以遵循。但是，如果 barProccess 是一个实际可重用的抽象，您可以为其指定一个清晰的名称，那么将其分开可能是一个好主意。

说你不关心命名空间污染，因为所有这些函数都已经关闭了。

我认为你在这里有点夸张。内部 IFFE 不像外部 IFFE 那样 100% 推荐，但进一步限制范围仍然完全可以，我认为它不值得与你的同事争吵。

Answer 2

1a：没有形成闭包，因为外部函数foo() 在内部函数执行时没有返回。内部函数是一个匿名的“自执行”子程序。一个自执行函数可以形成一个闭包，而这个函数既不需要也不需要。这是构建代码的一种稍微“华丽”的方式。

1b：没有形成闭包，因为外部函数foo() 直接调用命名函数barProcess()，它返回一个数值，而不是一个函数。如果还要从代码中的其他地方调用 barProcess()，这种方法会很有用。

1c：没有形成闭包。 foo() 包含一个简单的代码块，它易于理解并且肯定会完成这项工作。由于 javascript for 循环没有自己的范围，foo() 将简化为：

function foo(bar) {
    for(var i=0, results=0; i<bar; i++) {
         results += 1 + (results * bar);
    }
    return {
        x: 1 + bar,
        y: 1 - bar,
        duration: results
    };
}