提问者:小点点

没有使用promise下载图像[重复]


我需要下载所有的图像,并用它们生成word文档。 使用nodeJS和Meteor

WebApp.connectHandlers.use('/download', async function (req, res, next) {
  // ...

  const images = [];

  await lines.forEach(async (line, k) => {
    if (line.type && line.type === 'image') {
      images.push({
        id: line.id,
        file: line.id + '.jpg',
      });

      download_image(line.imageUrl, line.id + '.jpg');
    }
  });

  // ...

  // Then I use images[] to insert them into a Word document.
});

const download_image = (url, image_path) =>
  axios({
    url,
    responseType: 'stream',
  }).then(
    (response) =>
      new Promise((resolve, reject) => {
        response.data
          .pipe(fs.createWriteStream(image_path))
          .on('finish', () => resolve())
          .on('error', (e) => reject(e));
      })
  );

问题是,在我将图像插入Word文档之前,图像不会被下载。

如何在图像完成下载之前停止/等待? 我不太会承诺。 她不见了是什么?

谢啦!


共1个答案

匿名用户

它的常见错误是使用.foreach(或类似的数组方法),其中包含async函数。 异步函数只是表示它返回promise,而await的工作方式与将promises与then链接在一起相同。 因此,这一行wait lines.foreach(async(line,k)=>{将只创建并返回一堆承诺,但它不会等待里面的所有承诺完成。

WebApp.connectHandlers.use('/download', async function (req, res, next) {
  // ...

  const images = [];
  const promises = [];
  lines.forEach((line, k) => {
    if (line.type && line.type === 'image') {
      images.push({
        id: line.id,
        file: line.id + '.jpg',
      });

      promises.push(download_image(line.imageUrl, line.id + '.jpg'));
    }
  });
  // here you get array with all the images downloaded
  const downloadedImages = await Promise.all(promises);
  // this line will be executed after you download all images

  // ...
});

// This function would work same with or without the `async` keyword 
// (because async function return promise - you are returning the promise. 
// Async function allows to use await, but you are not using await in this function).
// However it is good practice to have all functions that returns promise 
// marked as `async` so you know that you receive promise from it.
const download_image = async (url, image_path) =>
  // Dont forget to return your promise otherwise you cannot await it
  return axios({
    url,
    responseType: 'stream',
  }).then(
    (response) =>
      new Promise((resolve, reject) => {
        response.data
          .pipe(fs.createWriteStream(image_path))
          .on('finish', () => resolve())
          .on('error', (e) => reject(e));
      })
  );