forループ内で非同期要求によってデータが取り込まれた後に配列を返すことができません

Question

ループがforeachのループごとに非同期に配置されている配列を返そうとしています。しかし、私が帰ってくると、私はundefinedしか得られません。

var navigateAndFetchPages = function (data) { 
    var countryPages = []; 
    data.forEach(function (val) { 
    Rq(val.esomar_url) 
     .then(function (data) { 
      var $ = cheerio.load(data), 
       pages_elem = $('.mt0.mb0-5.pt0').find('a').not('.active'); 
      countryPages.push({country_name: val.country_name, links: pages_elem}); 
    }) 
    }); 
    return countryPages; 
}; 


var scraper = { 
    extract: function (dir) { 
     return landingPage(dir) 
      .then(function (countries) { 
       return navigateAndFetchPages(countries) 
      }) 
      .then(function (p) { 
       p() 
      }) 
      .catch(); 
    } 
}; 

Answer 1

あなたは置き換えることができ.map()、.forEach()ためPromise.all()、Promise.all()

var navigateAndFetchPages = function(data) { 

    return Promise.all(data.map(function(val) { 
    return Rq(val.esomar_url) 
     .then(function(data) { 
     var $ = cheerio.load(data), 
      pages_elem = $('.mt0.mb0-5.pt0').find('a').not('.active'); 
      return { 
      country_name: val.country_name, 
      links: pages_elem 
      }; 
     }); 
    })); 

}; 

navigateAndFetchPages 
.then(function(countryPages) { 
    // do stuff with `countryPages` 
}) 
.catch(function(err) { 
    console.log(err); 
}); 

Answer 2

最初にチェーン.then()で得られた配列内に含まれる.then()からreturnオブジェクト：以下

は、コードブロックでありますまず、コードを適切にインデントしてください - それははるかに簡単です！

第2に、コードの非同期性を取り入れる必要があります。 Rqは、おそらく、非同期関数です。これは、操作全体を非同期にします。つまり、navigateAndFetchPagesは、データを返そうとするのではなく、約束を返す必要があります。

これを行う簡単な方法はArray#mapです。

var navigateAndFetchPages = function(data) { 
    return data.map(function(val) { 
    return Rq(val.esomar_url).then(function(data) { 
     var $ = cheerio.load(data), 
      pages_elem = $('.mt0.mb0-5.pt0').find('a').not('.active'); 
     return { 
      country_name: val.country_name, 
      links: pages_elem 
     }; 
     }) 
    }); 
}; 

navigateAndFetchPagesへの呼び出しの結果は、約束の配列になります。

あなたはPromise.allを完了するために、それらすべてを待つことができます。

Promise.all(navigateAndFetchPages(data)).then(function(result) { 
    // result is an array of objects containing your data 
}); 

更新された質問は、これは約束のチェーンで呼び出さそのものであることを示しています。これに対処するのは簡単です：

var scraper = { 
    extract: function (dir) { 
     return landingPage(dir) 
      .then(function (countries) { 
       return Promise.all(navigateAndFetchPages(countries)); 
      }) 
      .then(function (p) { 
       p() 
      }) 
      .catch(); 
    } 
}; 

あなたはnavigateAndFetchPages関数内Promise.allを置くこともできますが、私ではない、場合にあなたは将来的には個々のページのデータにアクセスする必要があります。あなたのAPIですが、それはあなた次第です。