Comparing javascript array values

Question

window.onload=function(){
        contents = new Array();
        painted = new Array();
        keys = 0;
        for(var i = 0; i < 4; i++){
            contents[i] = '';
            painted[i] = false;
        }
        contents[0] = 1;

    }

    function clicked(canvasNumber){
        if(contents[canvasNumber-1] == 1;){
            alert("you won!");
        }
    }

I am trying to get a pop up saying, "you won!" when canvas 1 is clicked, but the contents of my contents array are not matching up to trigger that alert. What am I doing wrong?

I also want to add that all variables have already been declared. — Michael Bowen
– Michael Bowen, Commented Oct 14, 2013 at 14:29
In addition to the semi-colon answer below, have you defined contents outside the functions to make it a global? — Steve Chambers
– Steve Chambers, Commented Oct 14, 2013 at 14:36

carter · Accepted Answer · 2013-10-14 14:33:10Z

1

if(contents[canvasNumber-1] == 1;){

should be

if(contents[canvasNumber-1] == 1){

with no semicolon in the if statement

Also your indexes are most likely not aligned.

answered Oct 14, 2013 at 14:33

carter

5,5145 gold badges34 silver badges40 bronze badges

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

Collectives™ on Stack Overflow

Comparing javascript array values

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related