web-dev-qa-db-fra.com

Rendre la boucle while synchrone

J'ai un code suivant

var page = 2;
var last_page = 100;
while(page <= last_page) {
  request("http://some_json_server.com/data?page=" + page, function (error, response, body) {
    if (!error && response.statusCode == 200) {
      store_data(body)
    }
    page++;
  });                
}

J'ai fait ce qui suit, mais il ne récupère en réalité rien. Est-ce que je le fais correctement?

var page = 2;
var last_page = 100;
while(page <= last_page) {
var async_arr = [];
async_arr.Push(
  function(next) {
    request("http://some_api_url?page=" + page, function (error, response, body) {
      if (!error && response.statusCode == 200) {
        store_data(body);
      }
    });
  }
);

async.series(
  async_arr, done
);
23
ericbae

Vous recherchez async.whilst () . Cette solution suppose que vous souhaitiez réellement effectuer chaque demande l'une après l'autre. Comme @UpTheCreek le mentionne (edit: le commentaire auquel j'ai fait référence a été modifié), il serait probablement possible de le faire de manière asynchrone et de suivre chaque résultat à l'aide de async.parallel.

var page = 2,
    lastPage = 100;

async.whilst(function () {
  return page <= lastPage;
},
function (next) {
  request("http://some_json_server.com/data?page=" + page, function (error, response, body) {
    if (!error && response.statusCode == 200) {
      store_data(body)
    }
    page++;
    next();
  });
},
function (err) {
  // All things are done!
});
22
Andreas Hultgren

Avec while, vous obtenez une boucle occupée, ce qui est contraire au but dans Node.

Faites-en une fonction récursive à la place. Chaque appel sera effectué dans un tick séparé.

var page = 2;
var last_page = 100;

(function loop() {
    if (page <= last_page) {
        request("/data?page=" + page, function (error, response, body) {
            if (!error && response.statusCode == 200) {
                store_data(body)
            }
            page++;
            loop();
        });
    }
}());
54
katspaugh

Vous pouvez également boucler votre boucle while en mode async et vous interrompre une fois votre promesse résolue/les conditions remplies.

    const request = require("request")

    ;(async()=>{

    let results = []
    while(true){
      await new Promise(resolve => {
        request('http://www.seanbehan.com/', (err, resp, body)=>{
          console.log(new Date, 'Downloading..')
          results.Push(body)
          resolve(body)
        })
      })

      if(results.length >= 5){
        break
      }
    }

    console.log(results)
  })()
0
seanbehan