JavaScript许诺.then（）无法执行

Question

我正在尝试下载一堆文件，为每个文件存储一些数据，然后使用该数据进一步处理它们。 稍后，我将使用Promise.map(queue,(x)=>get_image(...x),{concurrency: 1}).all().then( )之后将它们一起处理。 但是，在我的.then()调用之一之后，似乎没有任何进一步的.then()调用起作用：即使'processing downloaded image'正在'processing downloaded image' ，也从未打印过'download processed' 。 有人可以告诉我我所缺少的吗？

var get_image = (page_string, row, col) => {
  var url = get_url(page_string,row,col)
  var path = 'results/'+page_string+'/'+page_string.match(/f\d\d\d\w/)[0]+'_r'+pad_int(row)+'_c'+pad_int(col)+'.jpg'
  y = download(url,path)
    .then(process_downloaded(path))
    .then((x) => {console.log('download processed'); return x})
  return y
}

var download = (url, dest) => {
  console.log('attempting download')
  console.log(url)
  var y = new Promise((res,err) => {
    request(url).pipe(fs.createWriteStream(dest))
      .on('close', () => res)
      .on('error', (e) => {fs.unlink(dest); err(e)})
    })
  return y
}

var process_downloaded = (dest) => {
  var p = dest.match(/_f\d\d\d\w/)[0]
  console.log(dest,'processing downloaded image')
  var r = parseInt(dest.match(/r\d+/)[0].slice(1))
  var c = parseInt(dest.match(/c\d+/)[0].slice(1))
  var record = {id: p, r: r, c: c}
  if(fs.readFileSync(dest).toString('utf8').slice(0, 5) === 'http:'){fs.unlink(dest); return {id: p, r: 0, c: 0}} // 404 error
  return record
}

Answer 1

该代码至少存在两个问题： .on('close', () => res)将不会调用res()函数； 它必须是.on('close', () => res())或.on('close', res) 。

同样，在.then(process_downloaded(path)) ： process_downloaded(path)是一个对象，但是.then()需要一个函数作为参数。 .then(() => process_downloaded(path))正常工作。

为了避免将来发生此类错误， Promise.config({ warnings: true, longStackTraces: true, cancellation: true, monitoring: true })可以帮助您； 启用后，上述错误会产生警告，尽管第一个错误不会。 （ http://bluebirdjs.com/docs/api/promise.config.html ）

同样， fs.unlink(dest)在代码中使用了两次，但不意味着没有回调就被调用，并产生警告。 如果由于某种原因花费了太长时间才能完成，则此后将图像合成在一起时该文件可能仍然存在。 我改用fs.unlinkSync(dest) 。 有性能成本，但是我不打算为此而追求高性能。

通过这些更改，代码将按预期工作。

var get_image = (page_string, row, col) => {
  var url = get_url(page_string,row,col)
  var path = 'results/'+page_string+'/'+page_string.match(/f\d\d\d\w/)[0]+'_r'+pad_int(row)+'_c'+pad_int(col)+'.jpg'
  y = download(url,path)
    .then(() => process_downloaded(path))
    .then((x) => {console.log('download processed',x); return x})
  return y
}

var download = (url, dest) => {
  console.log('attempting download')
  console.log(url)
  var y = new Promise((res,err) => {
    request(url).pipe(fs.createWriteStream(dest))
      .on('close', res)
      .on('error', (e) => {fs.unlinkSync(dest); err(e)})
    })
  return y
}

var process_downloaded = (dest) => {
  var p = dest.match(/arundel_ms_263_f\d\d\d\w/)[0]
  console.log(dest,'processing downloaded image')
  var r = parseInt(dest.match(/r\d+/)[0].slice(1))
  var c = parseInt(dest.match(/c\d+/)[0].slice(1))
  var record = {id: p, r: r, c: c}
  if(fs.readFileSync(dest).toString('utf8').slice(0, 5) === 'http:'){fs.unlinkSync(dest); return {id: p, r: 0, c: 0}} // 404 error
  return record
}