Question

我必须创建文件上传，将文件存储在Amazon S3存储桶中，然后将信息写入数据库。

  router.post('/report', upload.single('file'), function(req, res, next) {
    reportService
      .uploadReport(req.body, req.file)
      .then(data => {
        res.send(data);
      })
      .catch(err => {
        res.send(err);
      });
  });

这段代码处理呼叫。它会触发以下代码。

function uploadReport(report, file) {
  var objectParams = { Bucket: bucketName, Key: file.filename, Body: '' };
  var fs = require('fs');
  var fileStream = fs.createReadStream(file.path);
  fileStream.on('error', function(err) {
    console.log('File Error', err);
  });
  objectParams.Body = fileStream;
  return new Promise((resolve, reject) => {
    s3.putObject(objectParams)
    .promise()
    .then(
      data => {
        resolve(this.addReport(report, data.Location)
          .then(response => resolve(response))
          .catch(err => {
            reject(err);
          })
        );
      },
      err => {
        reject(err);
      }
    );
  });
}

这段代码将文件上传到我的s3存储桶中，上传完成后，它调用写入数据库的save report function。

function addReport(report, url) {
  return new Promise((resolve, reject) => {
    reportModel
      .addReport(report, url)
      .then(report => {
        resolve(report);
      })
      .catch(err => {
        reject(err);
      });
  });
}

我知道我的2个诺言可以单独工作，但是将它们组合成1个诺言不会触发任何功能。

我如何正确地将它们组合成1个诺言？

Answer 1

您似乎违反了What is the explicit promise construction antipattern and how do I avoid it?。您正在用承诺解决承诺。相反，您只想返回Promise：

return s3
    .putObject(objectParams)
    .promise()
    .then(data => this.addReport(report, data.location));

Answer 2

您将必须返回addReport Promise，然后在then回调中解析/拒绝原始Promise。

node.js-结合两个承诺

2 个答案: