Как подождать, пока поток закончится? (Nodejs)

У меня есть массив циклов из promises, поэтому я использовал Promise.all, чтобы пройти через них и затем вызвать потом.

let promises = [];
promises.push(promise1);
promises.push(promise2);
promises.push(promise3);

Promise.all(promises).then((responses) => {
  for (let i = 0; i < promises.length; i++) {
    if (promise.property === something) {
      //do something
    } else {
      let file = fs.createWriteStream('./hello.pdf');
      let stream = responses[i].pipe(file);
      /*
         I WANT THE PIPING AND THE FOLLOWING CODE 
         TO RUN BEFORE NEXT ITERATION OF FOR LOOP
      */
      stream.on('finish', () => {
        //extract the text out of the pdf
        extract(filePath, {splitPages: false}, (err, text) => {
        if (err) {
          console.log(err);
        } else {
          arrayOfDocuments[i].text_contents = text;
        }
      });
    });    
  }
}

обещание1, обещание2 и обещание3 - это некоторые HTTP-запросы, и если один из них является приложением /pdf, я пишу его в поток и разбираю текст из него. Но этот код запускает следующую итерацию перед анализом теста из pdf. Есть ли способ заставить код ждать, пока конвейер в поток и извлечение не закончатся, прежде чем перейти к следующей итерации?

Ответы

Ответ 1

Также будет работать что-то вроде следующего. Я использую этот шаблон довольно часто:

let promises = [];
promises.push(promise1);
promises.push(promise2);
promises.push(promise3);

function doNext(){
  if(!promises.length) return;
  promises.shift().then((resolved) =>{
    if(resolved.property === something){
      ...
      doNext();
    }else{
      let file = fs.createWriteStream('./hello.pdf');
      let stream = resolved.pipe(file);
      stream.on('finish', () =>{
        ...
        doNext();
      });
    }

  })
}
doNext();

или разбить обработчик на контроллер и обработчик Promisified:

function streamOrNot(obj){
  return new Promise(resolve, reject){
    if(obj.property === something){
      resolve();
      return;
    }
    let file = fs.createWriteStream...;
    stream.on('finish', () =>{
      ...
      resolve();
    });
  }
}

function doNext(){
  if(!promises.length) return;
  return promises.shift().then(streamOrNot).then(doNext);
}

doNext()

Ответ 2

Без async/await это довольно противно. С помощью async/await просто сделайте это:

Promise.all(promises).then(async (responses) => {
  for (...) {
    await new Promise(fulfill => stream.on("finish", fulfill));
    //extract the text out of the PDF
  }
})

Ответ 3

Вы можете записать часть else внутри функции, вызванной самим собой. Так что обработка потока будет происходить параллельно

(function(i) {
    let file = fs.createWriteStream('./hello.pdf');
    let stream = responses[i].pipe(file);
  /*
     I WANT THE PIPING AND THE FOLLOWING CODE 
     TO RUN BEFORE NEXT ITERATION OF FOR LOOP
  */
    stream.on('finish', () => {
      //extract the text out of the pdf
      extract(filePath, {splitPages: false}, (err, text) => {
      if (err) {
        console.log(err);
      } 
      else {
        arrayOfDocuments[i].text_contents = text;
      }
    });
  });    
})(i)

Кроме того, вы можете обрабатывать потоковое содержимое как часть самого оригинального/индивидуального обещания.

На данный момент вы создаете обещание и добавляете его в массив, вместо этого добавляете prom.then к массиву (что также является обещанием). И внутри обработчика, тогда вы делаете свой потоковый материал.