Question

我正在使用nodeJS和expressJS。我有2个功能，主要功能：

download(imgUrl, imgMD5, function(fileLength, fileSize) {
    console.log(fileLength);

    var qb = data.Img.query();
    qb.where('img_md5', '=', imgMD5).update({
        img_downloaded: 1
    }).then(function() {});
});

和外部功能

module.exports = function() {
    return function(uri, filename) {
        request(uri, function(err, res, callback) {
            fileLength = res.headers['content-length'];
            var mkdirs = function(fold, callback) {

                var pf = path.dirname(fold);

                fs.exists(pf, function(exists) {
                    var create = function() {

                        fs.mkdir(fold, callback);
                    };
                    if (exists) {
                        create();
                    } else
                        mkdirs(pf, create);
                })
            };
            var folder = ('./downloaded/' + yy + '/' + mm + '/' + dd + '/' + ho + '/');

            mkdirs(folder, function() {
                var r = request(uri).pipe(fs.createWriteStream(folder + filename));
                r.on('close');
            });
            callback(fileLength);
        });
    };
};

但是在运行时发出错误：

TypeError：string不是函数

我不知道我是否正确使用回调权？

谢谢

Answer 1

您的request()回调参数未经适当标记。传递给回调的第三个参数是响应中的整个缓冲（字符串）主体，而不是函数。这就是为什么它在callback(fileLength);抱怨的原因。

此外，由于您使用了回调（接收整个缓冲响应），因此无需再次请求URL。所以你可以改变这个：

mkdirs(folder, function(){
  var r = request(uri).pipe(fs.createWriteStream(folder + filename));
  r.on('close');
});

到此：

mkdirs(folder, function() {
  // `data` here is the third argument to the `request()` callback ...
  fs.writeFile(folder + filename, data);
});

保存额外的HTTP请求。

或者，如果您确实想要流式传输响应数据，可以这样做：

request(uri).on('response', function(res) {
  // ...

  mkdirs(folder, function() {
    res.pipe(fs.createWriteStream(folder + filename));
  });
});

Javascript回调函数不起作用

1 个答案: