プロミスが空のリストを返します

Question

ウェブスクレイピングでハイパーリンクアンカーのリストを取得する関数を記述しました。

これらのアンカーをすべてオブジェクト配列にプッシュします。この配列は、後でJson文字列にシリアル化されます。

Api.GetCourseSubmenuUrlの方法とApi.FilterSubmenuContentListの両方が約束を返します。

しかし、次のコードは、配列が.each() cheerio関数で埋められるのを待つことなく実行し続けます。なぜこれが起こるのですか？

cheerioの各メソッドは同期しています。

私のコードは、パッケージを使用します

：

• ブルーバード（https://github.com/petkaantonov/bluebird）
• チェリオ（https://github.com/cheeriojs/cheerio）
• 要求（https://github.com/request/request）

コード：

Connection.prototype.FillCourseWithSubmenuContent = function(course){ 
    var self = this; //This class 
    var submenuItems = []; 
    return new BPromise(function(resolve, reject){ 
     return Api.GetCourseSubmenuUrl(ApiConnection.authToken).then(function(response){ 
     return request.get({url: self.url + response.url + course.id, followRedirect: false, jar: cookiejar}, function(err,httpResponse,body){ 
      if(err){ 
      reject(err); 
      } 
      var cheerio = require('cheerio'); 
      var dashboardhtml = cheerio.load(body, { 
        normalizeWhitespace: true, 
        decodeEntities: true 
       } 
     ); 
      //Find all the links on the page 
      dashboardhtml('a').each(function(i, elem) { 
       console.log("Object:"); 
       console.log({"text":dashboardhtml(elem).text(), "url":dashboardhtml(elem).attr('href')}); 
       submenuItems.push({"text":dashboardhtml(elem).text().trim(), "url":dashboardhtml(elem).attr('href')}); 
      }); 
      resolve(); 
     }); 
     }).then(function(){ 
     console.log(submenuItems); 
     return Api.FilterSubmenuContentList(ApiConnection.authToken, submenuItems); 
     }); 
    }).catch(function(error){ 
     return reject(error); 
    }); 
}; 

Answer 1

この問題は、@ Bergi氏が指摘したPromise constructor antipatternを考慮して修正されました。

リクエストライブラリには約束のサポートがないので、私はまだそれを（Bluebird）約束の中にラップする必要がありました。

promisifyライブラリにも可能ですので、人生を楽にすることができます。しかし、ソリューションのデモンストレーションのために私は約束のラップルートを行った。

溶液：

Connection.prototype.FillCourseWithSubmenuContent = function(course){ 
    var self = this; //This class 
    var submenuItems = []; 
    return Api.GetCourseSubmenuUrl(ApiConnection.authToken).then(function(response){ 
     console.log(self.url + response.url + course.id); 
     return new BPromise(function(resolve, reject){ 
     request.get({url: self.url + response.url + course.id, followRedirect: false, jar: cookiejar}, function(err,httpResponse,body){ 
      if(err){ 
      reject(err); 
      } 
      var cheerio = require('cheerio'); 
      var dashboardhtml = cheerio.load(body, { 
        normalizeWhitespace: true, 
        decodeEntities: true 
       } 
     ); 
      //Find all the links on the page 
      dashboardhtml('a').each(function(i, elem) { 
       // console.log("Object"); 
       // console.log({"text":dashboardhtml(elem).text(), "url":dashboardhtml(elem).attr('href')}); 
       submenuItems.push({"text":dashboardhtml(elem).text().trim(), "url":dashboardhtml(elem).attr('href')}); 
      }); 
      return resolve(); 
     }); 
     }); 
    }).then(function(){ 
     console.log(submenuItems); 
     return Api.FilterSubmenuContentList(ApiConnection.authToken, submenuItems); 
    }); 
};