JS返回不等待等待

时间:2018-06-22 10:05:34

标签: javascript ecmascript-2017

我正在尝试使用异步等待。

我的应用这样启动

axios.get(`${ROOT_URL}/ccidxann.php`)
  .catch(err => console.error('axios error get', err))
  .then(async res => {
    const html = res.data
    const jobList = await getJobList(html)
    console.log(' after each jobList', jobList)
    // jsonfile.writeFileSync('jobs.json', jobList, { flag: 'a' })
  })
  .catch(err => console.error(err))

问题是jobList总是作为空数组返回。 这是getJobList函数

async function getJobList (html) {
  const jobList = []

  const dom = new JSDOM(html)
  const { window } = dom
  const $ = require('jquery')(window)
  const jobsInDom = $('table').first().find('td > a')

  // AWAIT HERE
  await jobsInDom.each(async function (index, jobElem) {
    const name = $(jobElem).text()
    const url = $(jobElem).attr('href')
    // Another AWAIT HERE
    const jobDetailsHTML = await getJobDetailsHTML(`${ROOT_URL}/${url}`)
      .catch(err => console.error('getJobDetailsHTML err', err))
    const domDetails = new JSDOM(jobDetailsHTML)
    const { window: windowDetails } = domDetails
    const $details = require('jquery')(windowDetails)
    const jobDetailsHTMLBody = $details('body').prop('outerHTML')
    jobList.push({
      _id: url,
      name,
      detailsHTML: jobDetailsHTMLBody
    })
    console.log('in each jobList', jobList)
  }) //each
  return jobList
}

如您所见,我将async关键字放在其前面,

并且我将async关键字放在了each的回调中

并将await放在所有异步操作的前面。

console.log回调中的each打印出填充有值的数组

但是看来returneach之前

即使前面带有await关键字。

这里也是getJobDetailsHTML,以防万一。这是一个简单的函数,似乎可以正常工作

async function getJobDetailsHTML (url) {
    // ANOTHER AWAIT HERE
    return await axios.get(url).data
}

1 个答案:

答案 0 :(得分:1)

我认为jobsInDom.each是同步功能,因此将await放在前面不会带来期望的效果。因此,您需要以某种方式获得承诺,该承诺将在完成所有作业的处理后解决,例如:

  // handles one element and returns promise
  const jobHandler = async jobElem => {
    const name = $(jobElem).text()
    const url = $(jobElem).attr('href')
    const jobDetailsHTML = await getJobDetailsHTML(`${ROOT_URL}/${url}`)
      .catch(err => console.error('getJobDetailsHTML err', err))
    const domDetails = new JSDOM(jobDetailsHTML)
    const { window: windowDetails } = domDetails
    const $details = require('jquery')(windowDetails)
    const jobDetailsHTMLBody = $details('body').prop('outerHTML')
    jobList.push({
      _id: url,
      name,
      detailsHTML: jobDetailsHTMLBody
    })
    console.log('in each jobList', jobList)
  }

  const promises = [];

  // start processing of each element
  jobsInDom.each((index, jobElem) => promises.push(jobHandler(jobElem));

  // wait for processing of all job elements
  await Promise.all(promises);