Javascript FOR Loop Callback

Question

There's a list of items IDs gameIds[i]_container that are displayed in a FOR loop. On These are being displayed without a problem.

I want the console to log the item's gameIds[i] on that items click. I believe this is supposed to be accomplished with a callback, but currently nothing occurs when items are clicked on.

Please help!

Code:

//CLICKING ON GAMES
//Active Game - bring user to game
//Newly Invited Game - ask user to accept invite, then either bring to game or delete from players list
//Rematch Game - recreate previous game

//Function
function gameAccess(i) {
    //REMATCH GAME
        if (currentRound[i] == roundWinners[i].length && currentRound[i] != 0 && currentRound[i] == numberOfRounds[i]){
            console.log("You clicked " + gameIds[i]);
        }
        //NEWLY INVITED GAME
        if (currentRound[i] == 0 && currentUserId != creator[i]) {
            console.log("You clicked " + gameIds[i]);
        }
        //ACTIVE GAME
        else{
            console.log("You clicked " + gameIds[i]);
        }
}

//Callback
function gameAccessCallback(i){
    return function(){
        gameAccess(i);
    };
}

//Iterate Through
for (i = 0; i < numberOf; i++) {
    document.getElementById(gameIds[i] + "_container ").click(gameAccessCallback(i));
};

Your question mentions gameIds[i]_container, but that is not valid Javascript. — jfriend00
– jfriend00, Commented Dec 26, 2014 at 21:08
Similar questions: stackoverflow.com/questions/8623088/… and stackoverflow.com/questions/18880895/… and stackoverflow.com/questions/21487187/… and stackoverflow.com/questions/22273839/… — jfriend00
– jfriend00, Commented Dec 26, 2014 at 21:11
I don't think this is duplicate. His gameAccessCallback function solves the problem in those duplicates. He's just not binding the event properly. — gen_Eric
– gen_Eric, Commented Dec 26, 2014 at 21:12

gen_Eric · Accepted Answer · 2014-12-26 21:09:49Z

3

The problem here is that you are doing document.getElementById(...).click(). DOMElements don't have a click method! jQuery does, but it doesn't look like you are using that here.

Try this:

for (i = 0; i < numberOf; i++) {
    document.getElementById(gameIds[i] + "_container ").addEventListener('click', gameAccessCallback(i));
}

answered Dec 26, 2014 at 21:09

gen_Eric

228k42 gold badges304 silver badges343 bronze badges

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

Collectives™ on Stack Overflow

Javascript FOR Loop Callback

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related