2017-05-30 149 views
1

我是JavaScript新手。这里是我的代码可变变量可从闭包访问

...... 
var filename, result, user=["a", "b", "c","d"]; 
for(var p=0;p<user.length;p++) 
     { 
     filename=userID[p]+'_'+user[p]+'.json'; 

     fs.readFile(filename, function read(err, data) 
     { 
      if (err) {throw err;} 
      result = data.toString(); 

      if (result.charAt(result.length-1) === ',') 
      result = result.substring(0,result.length-1) + ']}'; 

      console.log(p+filename+result+"\n\n"); //here 

      }); 

     } 

在“这里” P的值总是显示最后一个索引,在这种情况下3的各个版本中也文件名是最后我iteration.How可以得到正确的p在“here”中的值。对于我的编译器显示“可变变量是可以从闭包访问”。

+3

这是因为'readFile'是异步的,到'read'被调用时,'p'已经递增到'3'。使用[let binding](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Statements/let)(即'let p = 0')而不是'var'。或者你可以使用'user.forEach' – naomik

回答

1

@naomik评论似乎对任何在循环体内的异步工作总是不好的主意。因为循环不会等到响应。

var filename, result, user=["a", "b", "c","d"]; 
var p=0; 
function readfileUnitil(p) 
    { 
    filename=userID[p]+'_'+user[p]+'.json'; 

    fs.readFile(filename,function(err, data) 
    { 
     if (err) {throw err;} 
     result = data.toString(); 

     if (result.charAt(result.length-1) === ',') 
     result = result.substring(0,result.length-1) + ']}'; 

     console.log(p+filename+result+"\n\n"); //here 
     if(p<user.length){ 
     p++; 
     readfileUnitil(p); 
     } 
     }); 
    } 
} 

我做喜欢这种递归的,而不是让变量,因为这是更具可读性和记忆不会被浏览器希望帮助举行。

1

filename这种情况下范围for loop块,filename值已经结合fs.readFile。简单的方法来解决这个问题,请按照块代码

var filename, result, user = ["a", "b", "c", "d"]; 
for (var p = 0; p < user.length; p++) { 
    filename = userID[p] + '_' + user[p] + '.json'; 

    (function(file) { 
     fs.readFile(filename, function read(err, data) { 
      if (err) { 
       throw err; 
      } 
      result = data.toString(); 

      if (result.charAt(result.length - 1) === ',') 
       result = result.substring(0, result.length - 1) + ']}'; 

      console.log(p + filename + result + "\n\n"); //here 

     }); 

    })(filename) 

}