then節から約束の配列を返すことができます

Question

私の問題を解決しない同様の質問hereがありました。私は最初にカテゴリを取得し、次にカテゴリに基づいて、各カテゴリの情報を見つけることができる10時間ごとにcronジョブを実行しようとしています。どうすれば下のPromiseを簡略化できますか？私はBluebirdまたはQを使用していません。これはネイティブJSの約束です。同じコールバック地獄の約束を避けるようになっていたように正直なところ、以下のコードは、あなただけの.then()のためにインデントの余分なレベルの使用を停止する場合は、あなたは非常に単純な構造を持って、任意の提案

flipkart.getAllOffers = function() { 
    interval(43200,() => { 
     flipkart.findAllCategories() 
      .then((categories) => { 
       flipkart.save('flipkart_categories.json', categories) 
       if (categories) { 
        for (let item of categories) { 
         flipkart.findAllForCategory(item.category, item.top) 
          .then((items) => { 
           flipkart.save('flipkart_top_' + item.category + '.json', items) 
          }).catch((error) => { 
           console.log(error) 
          }) 
        } 
       } 
      }) 
      .catch((error) => { 
       console.log(error) 
      }) 
    }) 
} 

function interval(seconds, callback) { 
    callback(); 
    return setInterval(callback, seconds * 1000); 
} 

Answer 1

を探します。含ま

一つ.then()ハンドラ 含まif()声明 この修正版では別の非同期操作

が含まれているループ 、半分あなたのインデントが約束とは何の関係もありませんあなたのifとforから来ているため。残りは私にとって非常に論理的で、コールバック地獄のようには見えません。それはあなたが示すロジックを実装するために必要なものです。

flipkart.getAllOffers = function() { 
    interval(43200,() => { 
     flipkart.findAllCategories().then((categories) => { 
      flipkart.save('flipkart_categories.json', categories) 
      if (categories) { 
       for (let item of categories) { 
        flipkart.findAllForCategory(item.category, item.top).then((items) => { 
         flipkart.save('flipkart_top_' + item.category + '.json', items) 
        }).catch((error) => { 
         console.log(error) 
         throw error;  // don't eat error, rethrow it after logging 
        }); 
       } 
      } 
     }).catch((error) => { 
      console.log(error) 
     }) 
    }) 
} 

flipkart.save()も非同期であると約束を返した場合、あなたはおそらくあまりにも約束鎖にそれらをフックしたいです。あなたはすべての結果（あなたのタイトルが意味する何かが、あなたの質問doesnのを収集しようとしている場合

flipkart.getAllOffers = function() { 
    interval(43200,() => { 
     flipkart.findAllCategories().then(iterateCategories).catch((error) => { 
      console.log(error); 
     }) 
    }) 
} 

function iterateCategories(categories) { 
    flipkart.save('flipkart_categories.json', categories); 
    if (categories) { 
     for (let item of categories) { 
      flipkart.findAllForCategory(item.category, item.top).then((items) => { 
       flipkart.save('flipkart_top_' + item.category + '.json', items); 
      }).catch((error) => { 
       console.log(error); 
      }); 
     } 
    }  
} 

：あなたは常にこのようにも見て改善することがヘルパー関数を作成することができます

---

実際に言及していない場合）、これを行うことができます：

flipkart.getAllOffers = function() { 
    interval(43200,() => { 
     flipkart.findAllCategories().then(iterateCategories).then((results) => { 
      // all results here 
     }).catch((error) => { 
      console.log(error); 
     }); 
    }) 
} 

function iterateCategories(categories) { 
    flipkart.save('flipkart_categories.json', categories); 
    let promises = []; 
    if (categories) { 
     for (let item of categories) { 
      let p = flipkart.findAllForCategory(item.category, item.top).then((items) => { 
       flipkart.save('flipkart_top_' + item.category + '.json', items); 
      }).catch((error) => { 
       console.log(error); 
      }); 
      promises.push(p); 
     } 
    } 
    // return promise here that collects all the other promises 
    return Promise.all(promises); 
}