藍鳥的未處理排斥

Question

我有以下代碼。當f2不會報錯時它工作正常。

如果有錯誤，它會生成一個Unhandled rejection Error。

重寫代碼以避免Unhandled rejection Error並正確傳播到catch在f1的正確方法是什麼？

let Bluebird = require('bluebird'), 
    mkdirp = Bluebird.promisify(require('mkdirp')), 
    request = Bluebird.promisify(require('request')), 
    writeFile = Bluebird.promisify(require('fs').writeFile); 

function f1() { 
    ......... 
    f2(path, fileName, options).then(.....).catch(....); 
} 

function f2(path, fileName, options) { 
    p = mkdirp(path).then(request(options).then(res => { 
     if (res[0].statusCode === 200) { 
      writeFile(fileName, res[0].body); 
      return res[0].body; 
     } else { 
      throw new Error(res[0].statusCode + ': ' + res[0].body); 
     } 
    })); 
    return p; 
} 

Answer 1

的問題是要傳遞一個承諾到.then()在f2。 .then()會忽略任何不是函數的東西，所以f2真的返回是mkdirp(this.path)的承諾，這是一個很大的錯誤，原因有幾個。如果在request(options)的then處理程序中引發錯誤，那麼將無法處理它。

此外，您沒有做任何事情來處理從writeFile可能的錯誤。如果您致電writeFile，則需要返回包含該承諾鏈的承諾鏈，或者在f2之內添加邏輯來處理它。

因爲它看起來像您可以在這裏並行運行mkdirp()和request()，但你不使用的mkdirp()的結果，我會說這是要走的路：

function f2(path, fileName, options) { 
    var p = mkdirp(path).return(request(options)).then(res => { 
     if (res[0].statusCode === 200) { 
      return writeFile(fileName, res[0].body) 
      .return(res[0].body); 
     } else { 
      throw new Error(res[0].statusCode + ': ' + res[0].body); 
     } 
    }); 
    return p; 
}