Javascript 在node.js中将承诺与流一起使用

我重构了一个简单的实用程序来使用承诺。它从web上获取pdf并将其保存到磁盘。保存到磁盘后，应在pdf查看器中打开该文件。文件显示在磁盘上并且有效，shell命令打开OSX预览应用程序，但弹出一个对话框，抱怨文件为空

一旦文件流写入磁盘，执行shell函数的最佳方式是什么

// download a pdf and save to disk
// open pdf in osx preview for example
download_pdf()
  .then(function(path) {
    shell.exec('open ' + path).code !== 0);
  });

function download_pdf() {
  const path = '/local/some.pdf';
  const url = 'http://somewebsite/some.pdf';
  const stream = request(url);
  const write = stream.pipe(fs.createWriteStream(path))
  return streamToPromise(stream);
}

function streamToPromise(stream) {
  return new Promise(function(resolve, reject) {
    // resolve with location of saved file
    stream.on("end", resolve(stream.dests[0].path));
    stream.on("error", reject);
  })
}

在这一行

stream.on("end", resolve(stream.dests[0].path));

您正在立即执行

resolve

，调用

resolve

的结果（这将是未定义的，因为

resolve

返回的结果）用作

流的参数

.on
的第二个参数必须是函数，而不是调用函数的结果

因此，需要修改代码

stream.on("end", () => resolve(stream.dests[0].path));

或者，如果你是老派：

stream.on("end", function () { resolve(stream.dests[0].path); });

另一种老派的方式是

stream.on（“end”，resolve.bind（null，stream.dests[0].path））

不，不要这样做：p请参阅注释
经过一系列尝试后，我找到了一个一直运行良好的解决方案。有关更多信息，请参见JSDoc注释

/**
 * Streams input to output and resolves only after stream has successfully ended.
 * Closes the output stream in success and error cases.
 * @param input {stream.Readable} Read from
 * @param output {stream.Writable} Write to
 * @return Promise Resolves only after the output stream is "end"ed or "finish"ed.
 */
function promisifiedPipe(input, output) {
    let ended = false;
    function end() {
        if (!ended) {
            ended = true;
            output.close && output.close();
            input.close && input.close();
            return true;
        }
    }

    return new Promise((resolve, reject) => {
        input.pipe(output);
        input.on('error', errorEnding);

        function niceEnding() {
            if (end()) resolve();
        }

        function errorEnding(error) {
            if (end()) reject(error);
        }

        output.on('finish', niceEnding);
        output.on('end', niceEnding);
        output.on('error', errorEnding);
    });
};

用法示例：

function downloadFile(req, res, next) {
  promisifiedPipe(fs.createReadStream(req.params.file), res).catch(next);
}

更新。我已将上述函数作为节点模块发布：
其他解决方案可以如下所示：

const streamAsPromise = (readable) => {
  const result = []
  const w = new Writable({
    write(chunk, encoding, callback) {·
      result.push(chunk)
      callback()
    }
  })
  readable.pipe(w)
  return new Promise((resolve, reject) => {
    w.on('finish', resolve)
    w.on('error', reject)
  }).then(() => result.join(''))
}

你可以像这样使用它：

streamAsPromise(fs.createReadStream('secrets')).then(() => console.log(res))

在最新的NodeJ中，特别是在stream v3中，您可以执行以下操作：

const finished=util.promisify（stream.finished）；
const rs=fs.createReadStream（'archive.tar'）；
异步函数run（）{
等待完成（rs）；
log（'流已完成读取'）；
}
run（）.catch（console.error）；
rs.resume（）；//把水排干。

你的代码不工作吗？@Brad，显然不是，因为弹出一个对话框，抱怨文件是空的：p@JaromandaX这个问题被编辑了。这现在更有意义了。这是一个很好的解释。谢谢。请注意，
bind
解决方案会立即计算
stream.dests[0]。path
，而不是在流结束时。这可能适用于静态文件目标，但通常不适用于使用异步结果进行解析。如果是，我将删除它。是否在任何位置添加内容类型？这里还可以应用一些小的改进，如end（）&&resolve（）和end（）&&reject（）。您是否将“内容类型”作为HTTP头？是的，完全正确，因为当出现错误时，响应将是文本，否则将是管道/文件类型如果您谈论的是“用法示例”，那么您可能不应该使用该代码，它还远远没有做好生产准备。当然，真正的服务器会考虑内容类型。从StackOverflow复制随机代码时请注意。但是我回答的主要部分是
promisifiedPipe（）
函数，它在生产中经过了战斗测试。我建议只复制它。@JanakaBandara你能补充一些细节吗？在这种情况下，我应该遇到问题？对不起，我的错！当我设法让我的代码段正常工作时，出现了
.on（'end'
和
.on（'finish'
处理程序；但现在它似乎只有
。on（'finish'
就足够了（很可能我一开始就错了，用
end
，后来才添加了
finish
）请随意离开它。实际的流处理和周围的事件令人困惑，不必担心。祝你有一个美好的一天！