FireProise.all（）のネストされた約束がすべて解決された後

Question

Firebase APIを使用してHacker Newsストーリーのすべてのコメントを再帰的に取得しようとしています。ストーリーはkidsプロパティを持ちます。これはコメントを表すIDの配列です。各コメントには独自のkidsプロパティがあり、その子コメントを指しています。コメント全体次第お知らせし、その後

function getItem(id) { 
    return api 
     .child(`item/${id}`) 
     .once('value') 
     .then((snapshot) => { // this is a Firebase Promise 
      let val = snapshot.val() 

      if (val.kids) { 
       val.replies = val.kids.map((id) => getItem(id)) 
      } 

      return val 
     }) 
} 

そして：私は、私は次の関数を使用してこれを行うことが考え

[{ 
    'title': 'comment 1', 
    'replies': [{ 
    'title': 'comment 1.1' 
    }, { 
    'title': 'comment 1.2' 
    'replies': [{ 
     'title': 'comment 1.2.1' 
    }] 
    }] 
}] 

：私は、全体のコメントツリーの配列を作成するように見える何かをしたいですすべてのO以来、理にかなっている（何が起こって終わることはPromise.all().then()火災で

getItem(storyId) 
    .then(story => { 
     // The story and all of its comments should now be loaded 
     console.log(story) 
    }) 

コメント約束の最初のレベルが解決したら：ツリーは使用してフェッチされていますf commentPromisesが解決しました。）しかし、ネストされた約束がすべて解決されたら、私は知りたいです。これどうやってするの？

Answer 1

内val.repliesを設定します他の答えと同様に約束する。これは「explicit promise constructor anti-pattern」です。もう少し合理化すると、次のことができます。

function getItem(id) { 
    return api 
    .child(`item/${id}`) 
    .once('value') 
    .then(snapshot => { 
     const val = snapshot.val(); 
     return Promise.all((val.kids || []).map(getItem)) 
     .then(kidsVals => { 
      val.replies = kidsVals; 
      return val; 
     }); 
    ); 
    }); 
} 

明示的な拒否処理は必要ありません。拒否は自然にトップレベルに伝播します（あなたが望むものと仮定して）。

Answer 2

IMP：トーラザブロの答えを参考にしてください。それは私よりはるかに優れています。

=========私は、これは動作するはずだと思う

：

function getItem(id) { 
    return new Promise(function(resolve, reject) { 
     api 
     .child(`item/${id}`) 
     .once('value') 
     .then((snapshot) => { // this is a Firebase Promise 
      let val = snapshot.val() 

      if (val.kids) { 
       let replies = val.kids.map((id) => getItem(id)) 
       Promise.all(replies).then(replies => { 
        val.replies = replies // <<<<<< try this 
        resolve(val) // we want to resolve val, not val.replies 
       }, err =>{ 
        reject(err) 
       }) 
      } else { 
       resolve(val) 
      } 
     }) 
     .catch((err) => { // if there was some error invoking api call we need to reject our promise 
      reject(err); 
     }) 
    } 
} 

編集： はラッパーの必要はありませんthen