.then(success、fail)はいつpromiseのアンチパターンと見なされますか?

193
user2127480 2014-07-10 09:31.

ブルーバードプロミスFAQを見てみましたが、そこには.then(success, fail)アンチパターンであると書かれています。トライアンドキャッチについては、その説明がよくわかりません。次のこれの何が問題になっていますか?

some_promise_call()
.then(function(res) { logger.log(res) }, function(err) { logger.log(err) })

この例は、正しい方法として次のことを示唆しているようです。

some_promise_call()
.then(function(res) { logger.log(res) })
.catch(function(err) { logger.log(err) })

違いは何ですか?

6 answers

223
Bergi 2014-07-10 10:35.

違いは何ですか?

.then()コールは、コールバックがエラーをスローする場合には拒否されます約束を返します。つまり、成功loggerが失敗した場合、エラーは次の.catch()コールバックに渡されますが、failと一緒に行われるコールバックには渡されませんsuccess

ここだ制御フロー図は:

同期コードで表現するには:

// some_promise_call().then(logger.log, logger.log)
then: {
    try {
        var results = some_call();
    } catch(e) {
        logger.log(e);
        break then;
    } // else
        logger.log(results);
}

2番目log(の最初の引数のようなもの.then())は、例外が発生しなかった場合にのみ実行されます。ラベル付けされたブロックとbreakステートメントは少し奇妙に感じます、これは実際にはpythonが持っているものtry-except-elseです(推奨読書!)。

// some_promise_call().then(logger.log).catch(logger.log)
try {
    var results = some_call();
    logger.log(results);
} catch(e) {
    logger.log(e);
}

catchロガーも成功ロガーの呼び出しからの例外を処理します。

違いはこれだけです。

トライアンドキャッチについては説明がよくわかりません

議論は、通常、処理のすべてのステップでエラーをキャッチしたいので、チェーンで使用するべきではないということです。すべてのエラーを処理する最終ハンドラーは1つだけであることが期待されますが、「アンチパターン」を使用すると、一部のthenコールバックのエラーは処理されません。

ただし、このパターンは実際には非常に便利です。まさにこのステップで発生したエラーを処理し、エラーが発生しなかったとき、つまりエラーが回復不能なときにまったく異なることを実行したい場合。これは制御フローを分岐さいることに注意してください。もちろん、これが望まれることもあります。


次のこれの何が問題になっていますか?

some_promise_call()
.then(function(res) { logger.log(res) }, function(err) { logger.log(err) })

コールバックを繰り返さなければならなかったこと。あなたはむしろしたい

some_promise_call()
   .catch(function(e) {
       return e; // it's OK, we'll just log it
   })
   .done(function(res) {
       logger.log(res);
   });

.finally()これに使用することも検討してください。

37
acjay 2014-07-10 10:30.

2つは完全に同一ではありません。違いは、最初の例では、successハンドラーでスローされた例外をキャッチしないことです。したがって、よくあることですが、メソッドが解決済みのpromiseのみを返す必要がある場合は、末尾のcatchハンドラー(またはthen空のsuccessパラメーターを持つ別のハンドラー)が必要です。確かに、thenハンドラーが失敗する可能性のあることを何もしない可能性thenがあります。その場合、1つの2パラメーターを使用しても問題ありません。

しかし、リンクしたテキストのポイントは、then一連の非同期ステップをチェーンする機能において、コールバックと比較して最も役立つことだと思います。実際にこれを行うと、2パラメーター形式のthen微妙な動作が期待どおりになりません。 、上記の理由により。ミッドチェーンで使用すると、特に直感に反します。

複雑な非同期処理をたくさん行って、私が認める以上にこのようなコーナーにぶつかった人として、このアンチパターンを避けて、別のハンドラーアプローチを使用することを強くお勧めします。

18
aWebDeveloper 2016-03-21 04:55.

両方の長所と短所を調べることで、どちらが状況に適しているかを計算で推測できます。これらは、promiseを実装するための2つの主要なアプローチです。どちらにもプラスとマイナスがあります

キャッチアプローチ

some_promise_call()
.then(function(res) { logger.log(res) })
.catch(function(err) { logger.log(err) })

利点

  1. すべてのエラーは1つのcatchブロックによって処理されます。
  2. thenブロックで例外をキャッチすることもできます。
  3. 複数の成功コールバックの連鎖

短所

  1. チェーンの場合、異なるエラーメッセージを表示することが難しくなります。

成功/エラーアプローチ

some_promise_call()
.then(function success(res) { logger.log(res) },
      function error(err) { logger.log(err) })

利点

  1. きめ細かいエラー制御が得られます。
  2. データベースエラー、500エラーなど、さまざまなカテゴリのエラーに対して共通のエラー処理機能を使用できます。

短所

  1. catch成功のコールバックによってスローされたエラーを処理する場合は、さらに別のものが必要になります
2
bitfishxyz 2018-11-17 18:34.

簡単な説明:

ES2018では

引数onRejectedを指定してcatchメソッドを呼び出すと、次の手順が実行されます。

  1. この値をpromiseとします。
  2. 戻る?Invoke(promise、 "then"、«undefined、onRejected»)。

つまり:

promise.then(f1).catch(f2)

等しい

promise.then(f1).then(undefiend, f2)
1
Jayant Varshney 2019-05-03 10:25.

を使用.then().catch()すると、ワークフローを実行するために必要なPromiseChainingを有効にできます。データベースからいくつかの情報を読み取る必要があり、それを非同期APIに渡し、応答を操作する必要がある場合があります。応答をデータベースにプッシュバックすることをお勧めします。これらすべてのワークフローをコンセプトで処理することは実行可能ですが、管理が非常に困難です。より良い解決策はthen().then().then().then().catch()、一度のキャッチですべてのエラーを受け取り、コードの保守性を維持できるようにすることです。

1
VenCKi 2019-09-16 08:30.

を使用して、promiseで成功と失敗のハンドラーをチェーンthen()catch()ます。catch()によって返される約束に基づいて動作しthen()ます。それは処理します、

  1. 約束が拒否された場合。写真の#3を参照してください
  2. then()の成功ハンドラーでエラーが発生した場合は、以下の行番号4から7の間。写真の#2.aを参照してください(失敗したコールバックthen()はこれを処理しません)。
  3. then()の失敗ハンドラーでエラーが発生した場合は、以下の8行目。写真の#3.bを参照してください。

1. let promiseRef: Promise = this. aTimetakingTask (false); 2. promiseRef 3. .then( 4. (result) => { 5. /* successfully, resolved promise. 6. Work on data here */ 7. }, 8. (error) => console.log(error) 9. ) 10. .catch( (e) => { 11. /* successfully, resolved promise. 12. Work on data here */ 13. });

:多くの場合、catch()が既に記述されている場合、障害ハンドラーは定義されていない可能性があります。編集:のエラーハンドラーが定義されていない場合にのみreject()呼び出す結果にcatch()なります。写真の#3に注意してください。8行目と9行目のハンドラーが定義されていない場合に呼び出されます。then()catch()

then()コールバックがそれを処理している場合、によって返されるpromiseにはエラーがないため、これは理にかなっています。

Related questions

MORE COOL STUFF

ケイト・ブランシェットは3日間一緒に夫と一緒に寝て、25年経ってもまだ夫と結婚しています

ケイト・ブランシェットは3日間一緒に夫と一緒に寝て、25年経ってもまだ夫と結婚しています

ケイト・ブランシェットは、夫に会ったとき、典型的な交際のアドバイスに逆らいました。

マイケルシーンが非営利の俳優である理由

マイケルシーンが非営利の俳優である理由

マイケルシーンは非営利の俳優ですが、それは正確にはどういう意味ですか?

ホールマークスターのコリンエッグレスフィールドがRomaDramaLiveでスリル満点のファンと出会う![エクスクルーシブ]

ホールマークスターのコリンエッグレスフィールドがRomaDramaLiveでスリル満点のファンと出会う![エクスクルーシブ]

特徴的なスターのコリン・エッグレスフィールドは、RomaDrama Liveでのスリル満点のファンとの出会いについて料理しました!加えて、大会での彼のINSPIREプログラム。

「たどりつけば」をオンラインでストリーミングできない理由

「たどりつけば」をオンラインでストリーミングできない理由

ノーザンエクスポージャーが90年代の最も人気のある番組の1つになった理由を確認するには、Blu-rayまたはDVDプレーヤーをほこりで払う必要があります。

バイオニック読書はあなたをより速く読むことができますか?

バイオニック読書はあなたをより速く読むことができますか?

BionicReadingアプリの人気が爆発的に高まっています。しかし、それは本当にあなたを速読術にすることができますか?

ドミニカのボイリング湖:アクセスは簡単ではありませんが、ハイキングする価値があります

ドミニカのボイリング湖:アクセスは簡単ではありませんが、ハイキングする価値があります

ドミニカのボイリング湖は、世界で2番目に大きいボイリング湖です。そこにたどり着くまでのトレッキングは大変で長いですが、努力する価値は十分にあります。

私たちの水をきれいに保つのを助けるためにあなたの髪を寄付してください

私たちの水をきれいに保つのを助けるためにあなたの髪を寄付してください

サロンからのヘアトリミングや個人的な寄付は、油流出を吸収して環境を保護するのに役立つマットとして再利用できます。

ホワイトハウスの最も記憶に残る結婚式を見てください

ホワイトハウスの最も記憶に残る結婚式を見てください

過去200年以上の間にホワイトハウスで結婚したのはほんの数人です。彼らは誰でしたか、そしてそこで結婚式を獲得するために何が必要ですか?

グッドジョブ、ESPN

グッドジョブ、ESPN

今日のホワイトハウスの記者会見で、報道官のサラ・ハッカビー・サンダースは、スポーツセンターのホストであるイェメル・ヒルのドナルド・トランプに関する最近のツイートについてコメントするよう求められ、大統領と彼の政策を白人至上主義者として説明した。ヒルは彼女のつぶやきのためにESPNによって公に叱責されました。

アマゾンからのこのレミントンツールセールで髪を整える、スタイルを整える、乾かす、または取り除く

アマゾンからのこのレミントンツールセールで髪を整える、スタイルを整える、乾かす、または取り除く

基本的に頭のあらゆる部分から髪の毛を整えたり、ブロードライしたり、まっすぐにしたり、脱毛したりする必要がある場合は、このレミントンゴールドボックスが最適です。今日だけ、Amazonは、すでに人気のあるShortcut Pro Self-HaircutKitやPearlPro Ceramic Flat Ironのように、グルーミングをはるかに簡単にするヘアツールをマークダウンしています。

カナダの元桂冠詩人が、史上最高の文化の盗用でトゥパックとマヤアンジェロウから盗んだ

カナダの元桂冠詩人が、史上最高の文化の盗用でトゥパックとマヤアンジェロウから盗んだ

トゥパックシャクール(ティムモーゼンフェルダー/ゲッティイメージズ); マヤアンジェロウ(マーティンゴッドウィン/ゲッティイメージズ)移動、テイラースウィフト。ケンドールとカイリーは、必要な数の座席を持っています。

テスラは、ハリケーンイルマによる避難を容易にするために、フロリダでの車両の範囲を拡大しています

テスラは、ハリケーンイルマによる避難を容易にするために、フロリダでの車両の範囲を拡大しています

写真:Tesla Motorsフロリダに住んでいて、Tesla Model S、Model X 60、またはModel 60Dを使用している場合、車の自律性は50 km(約30マイル)高くなります。これは失敗ではなく、ハリケーンイルマによる避難作業を容易にするために会社自身が命じた自治権の一時的な延長です。

Zendaya Wishes Boyfriend Tom Holland Happy Birthday with Cuddly Photo: He 'Makes Me the Happiest'

Zendaya Wishes Boyfriend Tom Holland Happy Birthday with Cuddly Photo: He 'Makes Me the Happiest'

Zendaya shared a sweet photo in honor of boyfriend Tom Holland's 26th birthday Wednesday

小さな女性:脳卒中を患った後に病院から解放されたアトランタのジューシーな赤ちゃん:「まだ癒し」

小さな女性:脳卒中を患った後に病院から解放されたアトランタのジューシーな赤ちゃん:「まだ癒し」

シーレン「Ms.JuicyBaby」ピアソンは、先月脳卒中で入院した後、「もう一度たくさんのことをする方法を学ばなければならない」ため、言語療法を受けていることを明らかにしました。

エマストーンは彼女のクリフサイドマリブビーチハウスを420万ドルでリストアップしています—中を見てください!

エマストーンは彼女のクリフサイドマリブビーチハウスを420万ドルでリストアップしています—中を見てください!

オスカー受賞者の世紀半ばの家には、3つのベッドルーム、2つのバス、オーシャンフロントの景色があります。

ジーニー・メイ・ジェンキンスは、母乳育児の経験の中で、彼女は「本当に、本当に落ち込んでいる」と言います

ジーニー・メイ・ジェンキンスは、母乳育児の経験の中で、彼女は「本当に、本当に落ち込んでいる」と言います

ジーニー・メイ・ジェンキンスは、生後4か月の娘、モナコに母乳育児をしていると語った。

投資ノート:Bioscout AU$300万シード

投資ノート:Bioscout AU$300万シード

Bioscoutは、農家を運転席に置くという使命を負っています。Artesian(GrainInnovate)やUniseedと並んで、最新のシードラウンドでチームを支援できることをうれしく思います。問題真菌症による重大な作物の損失は、農民にとって試練であることが証明されています。

リトルマーケットリサーチ1| 2022年のクイックグリンプス遠隔医療市場

リトルマーケットリサーチ1| 2022年のクイックグリンプス遠隔医療市場

遠隔医療は、パンデミック後の時代では新しいものではなく、時代遅れの分野でもありません。しかし、業界を詳しく見ると、需要と供給の強力な持続可能性と、米国で絶え間ない革命となる強力な潜在的成長曲線を示しています。

スタートアップ資金調達環境:タイのスタートアップエコシステムの次は何ですか?

スタートアップ資金調達環境:タイのスタートアップエコシステムの次は何ですか?

2021年は、世界的なベンチャーキャピタル(VC)の資金調達にとって記録的な年でした。DealStreetAsiaによると、東南アジアも例外ではなく、この地域では年間で記録的な25の新しいユニコーンが採掘されました。

ムーアの法則を超えて

ムーアの法則を超えて

計算に対する私たちの欲求とムーアの法則が提供できるものとの間には、指数関数的に増大するギャップがあります。私たちの文明は計算に基づいています—建築と想像力の現在の限界を超える技術を見つけなければなりません。

Language