CODEWITHSUNDEEP
jqueryrecursion
Rosdi Kasim
Rosdi Kasim

Reputation: 25986

Sending data recursively using jquery $.ajax gives stack overflow error

Why am I getting "too much recursion" error when I do the following?

    function sendTheNames() {

       alert("start submitting names..");

        return function (array) {

            var name = $(array.shift()).text();

            $.ajax({
                url: "test.jsp?name=" + name,
                complete: function () {
                    if (array.length > 0) {
                        return arguments.callee(array);
                    }
                }
            });
        };

    }

    $(document).ready(function () {

        var selectedNames = [];
        $('ul li input:checked').each(function () {
            selectedNames.push($(this).parent());
        });

        alert("begin");

        sendTheNames()(selectedNames);

        alert("done");
    });

Upvotes: 2

Views: 3077

Answers (2)

deceze
deceze

Reputation: 522210

If you absolutely need asynchronous, separate calls, at least do a little simpler recursion along the lines of this:

var selectedNames = ['Abe', 'Burt', 'Chris'];

function sendNames() {
    var name = selectedNames.shift();

    $.ajax({
        url: "test.jsp?name=" + name,
        complete: function () {
            if (selectedNames.length > 0) {
                sendNames();
            }
        }
    });
}

sendNames();

Upvotes: 6

cobbal
cobbal

Reputation: 70753

because arguments.callee in that context refers to complete: function () {...}

You are recursing without getting any closer to a terminal condition. Essentially

function complete() {
    if (array.length > 0) {
        return complete(array);
    }
}

to have it recur to some other function put this inside the function you want to recur to:

var outerCallee = arguments.callee;

and recur using

return outerCallee(array);

Upvotes: 2

Related Questions