2016-08-22 4 views
2

私はウェブサイトからデータを引き出しています。引っ張られるデータはパニーナイズされる。最初のページにリクエストを行うと、nextCursorを含む応答が返されます。このカーソルは、ページ2の結果を取得するために使用する必要があります。各ページ要求にはnextCursorが必要です。プロミスとループを使用していますか?

ループを作成する方法が見つからないため、私はこの約束を使用して実装するのに苦労しています。これは私がそれを約束せずに動作すると仮定する方法である(ないテストし、それは私が何をしようとしている実証):あなたは、ループ内の反復回数を見ることができるように

let nextCursor = argv.initalCursor 

do { 
    let r = request('http://example.com/items.php?cursor=' + nextCursor, function(err, resp, body) { 
    if(err) throw new Error(err) 

    // Do something with the data 

    nextCursor = JSON.parse(body)['nextCursor'] 
    }) 
} while(nextCursor) 

は不明です。応答にnextCursorがなくなるまでループします。

私がしたいことは、約束を使用してこれを実装することです。私は、各リクエストが最後のリクエストに依存しているため、同様の方法で動作するループを作成する方法はわかりません。

これはどのように約束を使用して動作しますか?

return self.cursorRequestの場合、私の現在の解決策は失敗しています。スクリプトは単に実行を停止します。

'use strict' 

let Promise = require('bluebird') 
let _ = require('lodash') 

class Event { 
    constructor(session) { 
    this.session = session 
    this.scrapedIDs = [] // I don't like this! 
    } 

    parseGuestsAndCursor(json, guestType) { 
    let ids = json['payload'][guestType]['sections'][2][1].map(function(user) { 
     return user['uniqueID'] 
    }) 

    return { 
     ids: _.uniq(ids), 
     cursor: json['payload'][guestType]['cursor'] 
    } 
    } 

    cursorRequest(cursor, eventID, guestType) { 
    let self = this 

    return new Promise(function(resolve, reject) { 
     let url = `https://example.com/events/typeahead/guest_list/?event_id=${eventID}&tabs[0]=${guestType}&order[${guestType}]=affinity&bucket_schema[${guestType}]=friends&cursor[${guestType}]=${cursor}&dpr=1&__user=${self.session.uid}&__a=1` 

     self.session.request(url, function(err, resp, body) { 
     if(err) reject(err) 
     let json 

     try { 
      json = JSON.parse(body.substring(9)) 
     } catch(err) { 
      reject(err) 
     } 

     resolve(self.parseGuestsAndCursor(json, guestType)) 
     }) 
    }) 
    } 

    members(eventID, limit, guestType) { 
    let self = this 
    let ids = [] 

    return new Promise(function(resolve, reject) { 
     let url = `https://example.com/events/typeahead/guest_list/?event_id=${eventID}&tabs[0]=watched&tabs[1]=going&tabs[2]=invited&order[declined]=affinity&order[going]=affinity&order[invited]=affinity&order[maybe]=affinity&order[watched]=affinity&order[ticket_purchased]=affinity&bucket_schema[watched]=friends&bucket_schema[going]=friends&bucket_schema[invited]=friends&bucket_schema[ticket_purchased]=friends&dpr=1&__user=${self.session.uid}&__a=1` 

     self.session.request(url, function(err, resp, body) { 
     if(err) reject(new Error(err)) 
     let json, guests 

     try { 
      json = JSON.parse(body.substring(9)) 
      guests = self.parseGuestsAndCursor(json, guestType) 
     } catch(err) { 
      reject(err) 
     } 

     self.cursorRequest(guests.cursor, eventID, guestType).then(function(guests) { 

      self.scrapedIDs.concat(guests.ids).map(function(user) { 
      return user['uniqueID'] 
      }) 

      if(guests.cursor) { 
      return self.cursorRequest(guests.cursor, eventID, guestType) 
      } 
      else { 
      resolve(self.scrapedIDs) 
      } 
     }) 
     }) 
    }) 
    } 
} 

module.exports = Event 
+0

私は全然そこにどんな約束が表示されません。 –

+0

@ T.J.Crowder私は約束でやろうとしていることをどのように実装するかわからないので、そうはありません。だから、私はこのようなことを約束どおりにするにはどうしたらよいのでしょうか?私は、ソリューションを必要とせず、助けることができるあらゆるリソースへのポインタです。 – BugHunterUK

+0

解決する前にすべてのデータが取得されるまで待つことを約束しますか? –

答えて

3

プロセスは非同期なので、ループ構造をまったく使用しません。自分自身を呼び出す関数を使用するだけです(間接的に別の関数を介して)。

あなたの場合、これを約束で実装したいと言われているので、ここでその方法を説明します(実際にコードで表現するのが最も簡単です)。これを行うには

var p = new Promise(function(resolve, reject) { 
    let nextCursor = argv.initialCursor; 

    doRequest(); 

    function doRequest() { 
     request('http://example.com/items.php?cursor=' + nextCursor, handleResult); 
    } 

    function handleResult(err, resp, body) { 
     if (err) { 
      // Got an error, reject the promise 
      reject(err); 
     } else { 
      // Do something with the data 

      // Next? 
      nextCursor = JSON.parse(body)['nextCursor']; 
      if (nextCursor) { 
       // Yup, do it 
       doRequest(); 
      } else { 
       // No, we're done 
       resolve(/*...resolution value here...*/); 
      } 
     } 
    } 
}); 

(ES2015のバージョンは基本的に同じに見えます。)

+0

'最大呼び出しスタックサイズを超過しました 'というメッセージが表示されるので、これは動作しません。 – BugHunterUK

+0

@BugHunterUK:実行しているバージョンにタイプミスがある必要があります。 'request(...、handleResult);'行に 'handleResult'の後に'() 'を間違って置いたわけではありません。これらの関数が存在しないことが重要であること、関数参照を渡したい、それを呼び出さないことが重要です。 –

1

もう一つの方法は、あなたが使用している非同期機能をpromisify、その上に構築することです。

このアプローチの利点は、あなたが約束を要求し、他のタイプを作りたい場合は、makeRequest()機能を再利用することができるように機能がモジュール化されていることである。

let nextCursor = argv.initalCursor 

function requestPromise(url) { 
    return new Promise(function (resolve, reject) { 
     request(url, function (err, resp, body) { 
      if (err) { reject(new Error(err)); } 
      else { resolve({ resp: resp, body: body}); } 
     }); 
    }); 
} 

function queryCursor(cursor) { 
    return requestPromise('http://example.com/items.php?cursor=' + cursor) 
     .then(function (result) { 
      // do something with result 

      var nextCursor = JSON.parse(result.body).nextCursor; 

      if (nextCursor) { 
       return queryCursor(nextCursor); 
      } 
     }); 
} 

queryCursor(nextCursor) 
    .catch(function (err) { 
     // deal with err 
    }); 
+0

自分の現在のソリューションで質問を更新しました。私はそれを働かせるように見えない。私が 'リターン'すると、スクリプトが停止するだけです。 – BugHunterUK

関連する問題