Javascript asynchronous loop

Question

Want to make a lot of asynchronous requests. The number of requests from 10 to 1000, should write down the answer in array (order is not important, no dependency)

   while (i<5) {
        api.get("get", offset: i, function(result) {
            datag = datag.concat(data.id);
        })
    i++;
    }

Is it right way? Is there a particular? the synchronous javascript used with asynchronous requests and i afraid of unexpected situations

What is api.get? Why so many request? Is there not a way you could do it in less requests? — putvande
– putvande, Commented Mar 14, 2014 at 16:00
This is an example query to api. Practical task - collecting public page posts from social network. — user3356784
– user3356784, Commented Mar 14, 2014 at 16:05

user229044 · Accepted Answer · 2014-03-14 16:05:58Z

1

No, this is not the correct way. You are making the requests in parallel. It may work for 5 request, but for thousands, some will surely timeout.

You need to make each request trigger the next request in its success timeout:

var i = 0;

function success (result) {
   datag = datag.concat(result.id);

   // launch the next request
   if (++i < 5) {
     api.get("get", offset: i, success)
   }
}

// launch the first request
api.get("get", offset: i, success);

answered Mar 14, 2014 at 16:05

user229044♦

241k41 gold badges346 silver badges350 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Javascript asynchronous loop

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest