2

今天，我尝试从我的服务器下载许多文件

下载.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。并有错误：

stream.js:81 抛出错误；// 管道中未处理的流错误。^ 错误：好的，关闭

此外，当我抛出循环时（例如：修复 dataJson.Person[i].Username，将 i 更改为 10）

运行代码后，系统返回正确的照片。

发生了什么？如何解决？

最良好的问候。！

4

2 回答 2

7

在 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被一次又一次地调用，直到文件数组中没有更多元素。

于 2013-08-19T10:07:05.377 回答
3

Javascript 没有循环块作用域，它具有基于函数的作用域。

这意味着，正如 Krasimir 指出的那样，您的 for 循环变量在使用完之前会相互覆盖。

因此，您至少需要将 for 循环的内部内容包装在一个函数中，否则看起来很奇怪的事情就会开始发生。

即使解决了这个问题，代码也会尝试同时进行所有下载，而 Krasimir 的答案可能会更清晰，因为他避免了这种情况。

尽管如此，使用函数来确保每个 for 循环执行都有自己的范围是一件好事。

修改for循环如下：

for (var i = 0; i < dataJson.Person.length; i++) {
      (function(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);
                });  // http.get
              }); // fs.writeFile
            }); // http.get
          } // else 
        }); // fs.exists
       })(i); // anonymous function to create scopes for loop
      } //for loop
于 2013-08-19T10:24:22.647 回答