Need to push all JSON into an array from AJAX, in jQuery

Question

Here's the code:

var newFeeds = []; // ** GLOBAL **

$(document.body).click(function() {

$.ajax({
            type: "POST",
            url: "http://mysite.com/feed.php",
            success:  function (data) {
                        $(newFeeds).push(data);
                        alert(newFeeds.length);
                        },
            error: function(error){
                      alert('Error: ' + error);
                    },
            dataType: "json"
    });
});

I can get the data from the server. All is ok, but the array never fills up. But strangely newFeeds.length returns 0! Why? I need to take the arrived data and push it into an array for later use.

and not forgetting of course that newFeeds won't be filled until after the async call has completed — Alnitak
– Alnitak, Commented Jun 20, 2013 at 22:40
So you guys gonna help him, or pass over a beer and a bag of chips? — cssyphus
– cssyphus, Commented Jun 20, 2013 at 22:43

Sushanth -- · Accepted Answer · 2013-06-20 22:38:29Z

4

$(newFeeds).push(data)

supposed to be

newFeeds.push(data)

newFeeds is an array that you have declared.

var newFeeds = [];

But in the callback you are wrapping it like a jQuery Object

$(newFeeds)

answered Jun 20, 2013 at 22:38

Sushanth --

55.8k9 gold badges70 silver badges109 bronze badges

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

10 Comments

Ted Over a year ago

ok, after correcting that (my bad, forgot to undo all my tries) it still won't sum up. I get 3 results in the json. I'd expect to get 6 on the second click, but it remains 3 for some reason(?)

Alnitak Over a year ago

@Ted are you sure you're giving the callback a chance to complete before checking the array length (per comment above on the question) ?

Sushanth -- Over a year ago

How does data look like ?

Sushanth -- Over a year ago

@Alnitak.. I think he is checking for the array length inside the callback

Sushanth -- Over a year ago

@Ted. Glad to have helped :)

|

Collectives™ on Stack Overflow

Need to push all JSON into an array from AJAX, in jQuery

1 Answer 1

10 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

10 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related