javascript - Promise.all() 中的 Promise 有时返回未定义/crawler-request 包nodejs

Question

我不确定我使用的promise.all是否错误，或者我用于远程检索pdf并解析这些的nodejs包是否一次被太多的请求淹没。

https://codesandbox.io/s/sharp-wave-qikvb//这里是codesandbox

我尝试使用promise.all

let urls = [arrayofURLS];

function pdfData() {   
    return Promise.all(
          urls.map(item => {
            this.crawlPdf(item);
          })
        )
          .then(result => {
        // handle result
          })

} 

这是使用爬虫包(称为crawler-request)的功能:

    crawlPdf: async function(Url) {

      return new Promise(async function(resolve, reject) {
      let response = await crawler(Url);
      resolve(response.text);
   }

五分之二的请求通常是未定义的。 但有时一切正常o.O..

Answer 1

您必须将 promise 返回给 all 方法。现在您没有返回任何内容，因此它看起来像 Promise.all([undefined, undefined, undefined])

由于看起来您可以使用箭头函数，因此您可以将大括号替换为括号，或者将其放在一行并完全摆脱括号 - 这些格式始终返回函数体的结果。

urls.map(item => (
  this.crawlPdf(item)
));

urls.map(item => this.crawlPdf(item));

或者保持明确

urls.map(item => { return this.crawlPdf(item) });