When I create more than one boundary, every one but the most recent breaks [closed]

Question

So basically what I have is a function called createBoxBoundary, which, when the player position is within certain limits, sets boundary variables to true. Pretty straightforward. However, when I call this function more than once in the main game loop, only the one most recently called works. Below is a sampling of my code

function createBoxBoundary(x,y,width,height){
    //right boundaries
    if(playerXPos + 40 == x && playerYPos + 80 >= y && playerYPos < y + height){
        boundaryRight = true;
    } else{boundaryRight = false;}

    //bottom boundaries
    if(playerYPos == y + height && playerXPos + 40 >= x && playerXPos <= x + width){
        boundaryTop = true;
    } else{boundaryTop = false;}

    //left boundaries
    if(playerXPos == x + width && playerYPos + 80 >= y && playerYPos <= y + height){
        boundaryLeft = true;
    } else{boundaryLeft = false;}

    //bottom boundaries
    if(playerYPos + 80 == y && playerXPos + 40 >= x && playerXPos < x + width){
        boundaryBottom = true;
    } else{boundaryBottom = false;}
}

I've also set up a fiddle with the full game code. And if anyone has advice on a better way to do collisions/boundaries in javascript, I'm open to that as well. Any help is appreciated!

Answer 1

I made the collision variables reset on each animation loop and then only set to true if they are a collision. I also changed the section of the code you call the animation loop from, this should be done after the processing of the current loop. Also added in some fallback support for my browser whilst testing. Updated fiddle here:

http://jsfiddle.net/totovste/QdCMG/8/

Answer 2

You set everything to false on a failed check and it looks a lot like you share those variables between all players/boundaries. might make sense to store them in an extra object and set/test each object individually.