私の約束が解決しないのはなぜですか？

Question

私は2つの約束を持っています。 1つは解決されていないと私はなぜ知りません。

processTradeOffer：'プラン'と呼ばれるオブジェクトをproccedしようと約束のチェーン。 identifyOffer varを返します。"valida" | "受け入れ可能" | "denegable"。 'valida'の場合、オファーからアイテムを特定する必要があります。したがって、別の非同期関数が必要ですidentifyItems（offer）は返されますvar 'offerState' "denegable" | "受け入れられる"とし、私たちは申し出を拒否または受け入れることができます。

私は知っている問題は、ステートメント（offerState == 'valida'）にありません。 コード：

const processTradeOffer = function(offer) { 
    return new Promise(function(resolve, reject) { 
     identyOffer(offer) 
      .then(function(offerState) { 
       return finishTradeOffer(offer, offerState); 
      }).then(function() { 
       console.log('aqui'); 
       return resolve(); 
      }) 
    }) 
} 
const finishTradeOffer = function(offer, offerState) { 

    return new Promise(function(resolve, reject) { 
     if (offerState == 'aceptable') { 
      acceptTradeOffer(offer).then(function() { 
       return resolve(); 
      }) 
     } else if (offerState == 'denegable') { 
      declineTradeOffer(offer).then(function() { 
       console.log('here'); 
       return resolve(); 
      }) 
     } else if (offerState == 'valida') { 
      identifyItems(offer).then(function(offerState) { 
       finishTradeOffer(offer, offerState); 
      }) 
     } 
    }) 
} 

はconsole.log（ 'ここ'）は正常に解雇とはconsole.log（ 'AQUI'）はいけませんさ。

Answer 1

まず、Promise constructor antipatternの使用を避けてください。あなたの機能はすでに約束を返す。

catchコールバックを追加して、考えられるエラーを処理します。

const processTradeOffer = function(offer) { 
    return identyOffer(offer) 
    .then(function(offerState) { 
     return finishTradeOffer(offer, offerState); 
    }) 
    .then(function() { 
     console.log('aqui'); 
    }) 
    .catch(err => console.log(err)); 
} 

const finishTradeOffer = function(offer, offerState) { 
    switch (offerState) { 
    case 'aceptable': 
     return acceptTradeOffer(offer); 
    case 'denegable': 
     return declineTradeOffer(offer); 
    case 'valida': 
     return identifyItems(offer) 
     .then(function(offerState) { 
      return finishTradeOffer(offer, offerState); 
     }); 
    default: 
     return Promise.resolve(); 
} 

Answer 2

問題は、このブロックであることである：

} else if (offerState == 'valida') { 
     identifyItems(offer).then(function(offerState) { 
      finishTradeOffer(offer, offerState); 
     }) 
    } 

あなたはとても機能はとてもコールバックのいずれかを呼び出さずにフォールスルーresolve()またはreject()のいずれかを呼び出していない、最終的には、「AQUI」ブロックが呼び出されることは決してありません。

Answer 3

コードを次のように記述してください。

const processTradeOffer = function(offer) { 
    const result = identyOffer(offer) 
    .then(offerState => finishTradeOffer(offer, offerState)) 

    result.then(() => console.log('aqui')); 

    return result; 
}; 

const finishTradeOffer = function(offer, offerState) { 
    switch(offerState) { 
    case 'aceptable': return acceptTradeOffer(offer); 

    case 'denegable': { 
     const result = declineTradeOffer(offer); 
     result.then(() => console.log('here')); 
     return result; 

    case 'valida': 
     return identifyItems(offer) 
     .then(offerstate => finishTradeOffer(offer, offerState)); 

    default: 
     throw "Invalid value for offerstate!!"; 
    } 
}; 

基本的なポイントは、約束が解決されるようにvalidaのケースを処理することです。さらに、「明白な約束のコンストラクタ」を取り除いています。