今天,我尝试从服务器上下载许多文件
download.js
function getPhotos(req, res) {
//Get User Photos
var fileReader = fs.readFile('./../data/user.json', 'utf8', function(err, data) {
if (err)
console.log(err);
else {
var dataJson = JSON.parse(data);
//console.log(dataJson.Person);
for (var i = 0; i < dataJson.Person.length; i++) {
var options = {
host : '10.16.47.128', // Local Server IPAddress
port : 2013, //Port
path : '/ExternalServer/avatar/' + dataJson.Person[i].Username + '.jpg',
};
//console.log(dataJson.Person[i].Username);
var fileAvatarPhotos = fs.createWriteStream('./../avatar/' + dataJson.Person[i].Username + '.jpg');
fs.exists(fileAvatarPhotos, function(exists) {//Check Exist File
if (exists) {
var req = http.get(options, function(res) {
//console.log(res);
res.pipe(fileAvatarPhotos);
});
} else {
var req = http.get(options, function(res) {
fs.writeFile(fileAvatarPhotos, '', function(err) {
if (err)
return console.log(err);
var req = http.get(options, function(res) {
//console.log(res);
res.pipe(fileAvatarPhotos);
});
});
});
}
});
}
}
});
//End Get User Photos
}
当我运行代码时:
node download.js
系统下载了所有照片,但照片大小为0kb。
并有错误:
另外,当我抛出循环(例如:修复dataJson.Person [i]。用户名,将i更改为10)时
运行代码后,系统会重新播放正确的照片。
发生什么事了?
如何解决?
最良好的问候。!
最佳答案
在for循环中实现这样的功能不是一个好主意。这是因为在循环中您具有异步操作,但是循环的每个迭代都会立即执行。在您的情况下,您要定义fileAvatarPhotos,它是一个WriteStream对象。 http.get方法是异步的,因此在其回调中,您可以使用为循环的第三次迭代或第四次或第一个迭代定义的fileAvatarPhotos。这取决于。您可以尝试将代码转换为如下形式:
var files = ["file1.jpg", "file2.jpg", "file3.jpg"];
var readFile = function(callback) {
if(files.length > 0) {
var file = files.shift();
http.get({path: file}, function(res) {
// ... process the result
readFile(callback);
})
} else {
callback();
}
}
readFile(function() {
console.log("reading finishes");
});
即反复调用readFile,直到files数组中没有更多元素为止。