Recursive function stopped after the first loop

Question

Context

I update several sites with Ajax calls, one by one to conserve the server.

I made a recursive function that runs himself again when an Ajax call is completed.

Issue

The function is stopped after the first loop.

Any idea ?

Code

var updateSite = function (site) {
    if (site.status == 'waiting for update') {
        updateStatus(i, site, 'update in progress');

        $.get(site.url)
            .success(function () {
                updateStatus(i, site, 'updated');
            })
            .error(function () {
                updateStatus(i, site, 'not updated');
            })
            .complete(function () {
                updateSite(allSites[i++]);
            });
    }
};

var i = 0;

updateSite(allSites[i]);

@Amberlamps -- the next call is initiated by the .complete function — Larry K
– Larry K, Commented Aug 14, 2012 at 15:46
@LarryK: This is an internal tool, if it works on chrome that's ok. :) — GG.
– GG., Commented Aug 14, 2012 at 15:50
@GG: If you use allSites.shift() instead of allSites[i] you would not need i in the first place, but I guess you need i for further processing. — Amberlamps
– Amberlamps, Commented Aug 14, 2012 at 15:52

IUnknown · Accepted Answer · 2012-08-14 15:45:37Z

5

Change complete function as below:

function () {
     updateSite(allSites[++i]);
}

answered Aug 14, 2012 at 15:45

IUnknown

22.5k4 gold badges41 silver badges71 bronze badges

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

Comments

Larry K · Accepted Answer · 2012-08-14 15:44:28Z

0

Any error messages?

Perhaps the var updateSite is not yet defined when the anonymous function you're using for complete is defined.

Try this:

var updateSite;
updateSite = function (site) {
    if (site.status == 'waiting for update') {
        updateStatus(i, site, 'update in progress');
.... // everything else the same...

answered Aug 14, 2012 at 15:44

Larry K

49.4k15 gold badges92 silver badges148 bronze badges

Collectives™ on Stack Overflow

Recursive function stopped after the first loop

Context

Issue

Code

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Context

Issue

Code

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related