try/catchの約束事を適切な方法でブロックしていますか？

Question

私は約束を呼び出す関数を持っています。成功または失敗の場合、私は約束を返す以上のデータを返したいと思います。

私は、これはうまくいくかもしれないと思った：

function foo() { 
    const extra = 'bar' 
    return thepromise().then((res) => { 
    return { 
     result: res, 
     data: extra 
    } 
    }, (err) => { 
    // this will not happen if an error is thrown in the called promise 
    return { 
     result: res, 
     data: extra 
    } 
    }) 
} 

foo().then((res) => { }, (err) => { // error result ends up here }) 

しかし、これは動作しません。 thepromiseにエラーがスローされた場合、catchブロックは呼び出されず、代わりにfoo()のcatchブロックが呼び出されます。

これを処理する適切な方法は何ですか？私は、try/catchブロックで成功していますが、私はこれが最善のアプローチであることを確認していない：例外またはエラーの任意の並べ替えを投げるべきではない約束を返すように設計された機能

function foo() { 
    const extra = 'bar' 
    return new Promise((resolve, reject) => { 
    try { 
     return thepromise(p) 
    } catch (e) { 
     reject(e) 
    } 
    }) 
    .then(function(res) { 
    return { 
     result: res, 
     data: extra, 
     status: 'success' 
    } 
    }, function(err) { 
    return { 
     result: err, 
     data: extra 
     status: 'error' 
    } 
    }) 
} 

Answer 1

にエラーをthrowことができます。

import Promise from 'bluebird'; 

// ... 

return Promise 
    .try(thepromise) 
    .then(res => ({ 
    result: res, 
    data: extra 
    }) 
    .catch(err => ({ 
    result: res, 
    data: extra 
    }); 

Answer 2

。

もしそうであれば、それはバグであるとみなされるべきです。

バグのあるメソッドを修正するには、例外をスローするのではなく、返された約束を拒否します。

---

他のユーザーのAPIを使用しているなど、潜在的なバグを修正することができない場合があります。これが当てはまる場合、最初に最も重要なことは、オリジナルの著者にバグを報告し、根本的な問題を修正できるようにすることです。

問題を報告した後、あなたが問題を解決するために、簡単なユーティリティでバギーメソッドをラップすることができます

function fixBrokenPromise(promise/*, args...*/) { 
    var args = Array.prototype.slice.call(arguments, 1), 
     ret; 
    try { 
    ret = promise.apply(null, args); 
    } catch (ex) { 
    ret = Promise.reject(ex); 
    } 
    return ret; 
} 

としてこれを呼び出すことができます。エラーがで取り扱わ考えられている

fixBrokenPromise(thepromise/*, args you want to pass to thepromise */) 
    .then(...resolve..., 
     ...reject...); 

Answer 3

.catch()であり、これは、約束された約束を返して.then()に戻す。あなたはbluebirdはあなたが必要なものだけでないtry methodを持って、次の.catch()

function foo() { 
 
    const extra = 'bar' 
 
    return thepromise().then((res) => { 
 
    return { 
 
     result: res, 
 
     data: extra 
 
    } 
 
    }, (err) => { 
 
    throw new Error(JSON.stringify({ 
 
     result: err || "no rejection reason provided", 
 
     data: extra 
 
    })) 
 
    }) 
 
} 
 

 
var thepromise =() => Promise.reject(); 
 

 
foo().then(data => console.log("fulfilled", data)) 
 
.catch(err => console.log("catch", JSON.parse(err.message)))

あなたは約束に拡張を楽しまために喜んでいる場合