由于异步回调,变量被覆盖

时间:2016-04-25 19:04:26

标签: javascript node.js asynchronous concurrency

我正在尝试使用node.js从文件系统访问多个文件。由于我正在尝试处理大量数据并且处理它们的顺序无关紧要,我想使用回调以异步方式执行此操作。我写了以下代码;

var fs = require('fs');

var basic_path="E:\\alphaDownload\\2016\\03";
var D1 = ['\\01','\\02','\\03','\\04'];
var D2 = ['\\00','\\01','\\02','\\03','\\04','\\05','\\06','\\07','\\08','\\09','\\10','\\11','\\12','\\13','\\14','\\15','\\16','\\17','\\18','\\19','\\20','\\21','\\22','\\23'];

var path=new Array(4);
for (var i=0;i<path.length;i++){
    path[i]=new Array(24);
}

for (var j1=0; j1<D1.length; j1++){
    for (var j2=0; j2<D2.length; j2++){
            path[j1][j2]=basic_path+D1[j1]+D2[j2];

            fs.readdir(path[j1][j2], function(err,items) {
                console.log("=============="+path[j1][j2]+"================");  
                console.log(items);
                if(err){throw err;}
                else{for (var i=0; i<items.length; i++) {
                    fs.readFile(path[j1][j2]+'\\'+items[i],'utf8',function read(error, data){
                        if (error) {
                            console.log("-------**"+error);
                        }
                        else {
                            console.log(data);}
                    });

                }}
            });

    }
}

但是由于readdirreadfile函数的异步性质,索引变量[j1]和[j2]的值发生了变化,并且{{1]中传递了单个路径[3] [23]功能。

任何人都可以设计一个出路。

2 个答案:

答案 0 :(得分:0)

使用async库。 async.each()允许异步调用集合的callbeack,当所有异步函数完成时,将调用主calbcak

<强>更新

在你的情况下,

async.map会更好

答案 1 :(得分:0)

因为索引var在每个循环中都被覆盖,所以你必须确定变量的范围。

这是你的情况

for(var i = 0; i < 3; i++) {
  setTimeout(function(){
    console.log(i);
  }, 100);
}

这里有两种方法

for(var i = 0; i < 3; i++) {
  setTimeout(function(n){
    console.log(n);
  }.bind(this, i), 100);
}

for(var i = 0; i < 3; i++) {
  setTimeout((function(n){
    return console.log(n);
  })(i), 100);
}

也许这个例子可以帮助你理解