Several things are going in here.
First, you do not need to initialize listings
as an array first.
Second, I think you're looking for for player in listings
.
Third, and I think that's the most important point :
request
sends an ajax request. Ajax requests are, by definition, asynchronous.
So what you actually push into your array is the result of "request" : probably nothing.
I'd recommend using a library such as async.js, a promise implementation, etc, or just a i = players.length
that you decrement and check each time a request succeeds, to ensure your code awaits all results before it executes.