Node.js 中的闭包有何不同？

Question

我经常使用 javascript，但昨天，我开始使用 Node.js。这是一个在文件夹的文件上运行 jslint 的小脚本。对于此示例，我将命令更改为调用 ls 而不是 jslint。

var sys = require("sys");
var fs = require("fs");
var cp = require('child_process');

var path = fs.realpathSync("./src/");

fs.readdir(fs.realpathSync("./src/"), function(err, files) {
  for (var i = 0; i < files.length; i++) {
    var filename = files[i];
    var complete = path + filename;

    // Run jslint on each file
    var jslint = cp.exec("ls " + complete, function(error, stdout, stderr) {
      console.log(filename + " : " + stdout);
    });    
  }
});

输出是这样的：

jskata.nofreeze.js : /home/dan/php/jskata/src/jskata.undo.js

jskata.nofreeze.js ：/home/dan/php/jskata/src/jskata.nofreeze.js

jskata.nofreeze.js ：/home/dan/php/jskata/src/jskata.timezone.js

为什么 console.log(filename + " : " + stdout); 行总是打印 jskata.nofreeze.js 文件名何时应该明显匹配 ls 的结果？ Node.js 中的闭包和作用域与 javascript 中的闭包和作用域不同吗？

Answer 1

不，它们没有任何不同，这只是 JavaScript 中闭包最常见的陷阱。

看，你在一个循环中，你在循环的每次迭代中分配局部变量filename，因此你实际上覆盖了它的值。由于它始终是相同的局部变量，并且闭包按引用工作，因此回调内的 filename 值会在循环的每次迭代中获得更新。

“jskata.nofreeze.js”恰好是目录中的最后一个文件，因此它也是分配给 filename 的最后一个值。

要解决这个问题，您需要传递 filename per 值：

// Run jslint on each file
(function(c, f) {
    cp.exec("cat " + c, function(error, stdout, stderr) {
      console.log(f + " : " + stdout);
    });
})(complete, filename);

这使得整个事情正常工作，尽管它有点难看。如果您不想让更多的匿名函数扰乱内部循环，您当然可以将其移至命名函数中。

Answer 2

编写此代码的代码有点令人困惑

for (var i = 0; i < files.length; i++) {
    var filename = files[i];

，因为 javascript 中的变量仅具有函数级别的范围，因此每次循环时都会重用 filename 变量。

将代码写入此：

var filename;
for (var i = 0; i < files.length; i++) {
    filename = files[i];

可能会更清楚问题出在哪里。以及为什么正如前面的答案所示，您需要将文件名传递给您的函数。