Javascript callback() for two functions

Question

I have two questions regarding the callback on JavaScript.

First: Does the for loop run asynchronously? That is, in the function:

function postData(items, callback)
{
   for(var i=0 ; i < items.length ; i++)
   {
      //angularjs Post request
      $http.post("http://www....", item[i]);
   }

   callback();
}

Will the callback() wait until the cycle is finished?

Second: How can I wait for a callback of two functions?

I have another function deleteData(items, callback) that I want to run async with the postData(). When the two are done I want to do some "refresh".

For now I'm using:

function postData(items, callback)
{
  for(var i=0 ; i < items.length ; i++)
   {
      //angularjs Post request
      $http.post("http://www....", item[i]);
   }

   callback();
}

function deleteData(items, callback)
{
  for(var i=0 ; i < items.length ; i++)
   {
      //angularjs delete request
      $http.delete("http://www....", item[i].id);
   }

   callback();
}

function refresh()
{
  $http.get("www....").success....
}

function doit(post, delete)
{
   postData(post, function(){
      deleteData(delete, function(){
          refresh();
      })
   });
}

Is it possible to make postData and deleteData run assync and when the both are done run the refresh?

----- EDIT -----

To make sure that the callback is send after the cycle is done, I use the code like this:

function postData(items, callback)
{
  if(items.length == 0)
    callback();
  for(var i=0 ; i < items.length ; i++)
   {
      //angularjs Post request
      $http.post("http://www....", item[i]);

      if(i == items.length-1)
         callback();
   }

   callback();
}

But it looks ugly... I was hoping for a cleaner solution...

replying to Ivan, Angular has also callback for the http requests, I don't need to use $ajax and I don't want to callback for each http request

André Snede · Accepted Answer · 2015-05-18 11:31:52Z

2

Yes, using the $q promise library, you can achieve that. If you are using angular, $q is already in your project, just include it in the service/controller you are using.

$q.all does exactly what you are looking for.

Example of how it would look:

var promises = [];

// loop x times
for(var i = 0; i < items.length; i ++){
    var promise = $http.post("http://www....", item[i]);
    promises.push(promise);
}

// Refresh will be called when all posts are completed.
$q.all(promises).then(refresh);

edited May 18, 2015 at 11:31

answered May 18, 2015 at 10:22

André Snede

10.1k7 gold badges48 silver badges67 bronze badges

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

1 Comment

NunoRibeiro Over a year ago

That's exactly what I need... And thank you for the updated answer with the example

Ivan · Accepted Answer · 2015-05-18 10:28:28Z

-1

First: no, it will not wait.

Second: Use $.ajax :

function postData(items, callback)
{
  for(var i=0 ; i < items.length ; i++)
   {
      $.ajax({
        url: "http://www....",
        ...........
      }).done(function() {
        callback();
      });
   }
}

Or alternatively vanilla JS:

function postData(items, callback)
{
  for(var i=0 ; i < items.length ; i++)
   {
      var r = new XMLHttpRequest(); 
      r.open("POST", "http://www....", true);
      r.onreadystatechange = function () {
      if (r.readyState != 4 || r.status != 200) return; 
        callback();
      };
   }
}

edited May 18, 2015 at 10:28

answered May 18, 2015 at 10:20

Ivan

7895 silver badges17 bronze badges

5 Comments

Tomalak Over a year ago

The OP does not use jQuery.

Ivan Over a year ago

@Tomalak, sorry, but what is OP?

Tomalak Over a year ago

"Original Poster" - The person who started the thread. urbandictionary.com/define.php?term=op&defid=2172424

Ivan Over a year ago

@Tomalak, Thanks for clarifying. I've changed the answer according to your comment

NunoRibeiro Over a year ago

According to your answer it will send a callback for each post request... that's not what I want. I want a callback when all post request are done...

Collectives™ on Stack Overflow

Javascript callback() for two functions

2 Answers 2

1 Comment

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related