Colliding problem...

[Ionut]

Hello!I have a real bad time with this for a long time.The only last piece of that "map" is collideble and this is making me a lot of problems.If I solve this a lot of things will run out nicely.
The minimal Code:

int mapArray[45][5] =
{
        {1,0,0,0,0},
        {1,0,0,0,0},
        {1,0,0,0,0},
        {1,0,0,0,0},
        {1,0,0,0,0},
...//and so
{1,0,0,0,0},
    };
//In the game loop
    if(Keyboard::isKeyPressed(Keyboard::Up))
    {
       upkey = true;
       upkeytimer.restart().asSeconds();
    }
    else if(!Keyboard::isKeyPressed(Keyboard::Up))
    {
       upkey = false;
    }
    if(upkey == true)
    {
    for(float up = upkeytimer.getElapsedTime().asSeconds();up < 2;up++) //Longer it's pressed longer it jumps
     {
        pos.y -=up;
     }
    }
    if(upkey == false && !SAdventurer.getGlobalBounds().intersects(STileGrass.getGlobalBounds())) //It collides only with the last piece,why?
    {
        pos.y += 2;
    }
//After window.clear()
for(int i = 0;i<43;i++)
    {
        for(int j = 0; j<5;j++)
    {
        if(mapArray[i][j] == 1)
        {
            STileGrass.setPosition(i * 70, (j * 70) + offsety);     //The "map" builder
            window.draw(STileGrass);
        }
    }
    }
window.display();
 

[Laurent]

It collides only with the last piece,why?

Because STileGrass is the last tile, after the drawing loop of the previous iteration of the main loop?

[Ionut]

Ok,so how to make all the pieces collideble ?

[Hapax]

Loop through them and test each one.

[Ionut]

I don't really get it...what do you mean when saying:"Loop through them and test each one."Sorry if I sound like a noob .

[Laurent]

what do you mean when saying:"Loop through them and test each one."

He means what you already do for drawing.

for(int i = 0;i<43;i++)
    {
        for(int j = 0; j<5;j++)
    {
        if(mapArray[i][j] == 1)
        {
            STileGrass.setPosition(i * 70, (j * 70) + offsety);     //The "map" builder
            /* put your collision check here instead of window.draw(STileGrass); */
        }
    }
    }

Sorry if I sound like a noob

You don't sound like a noob, you are one But why would you be sorry for that? Everyone is a noob, before being... something else

[Ionut]

The collision check :

if(upkey == false && !SAdventurer.getGlobalBounds().intersects(STileGrass.getGlobalBounds()))
        {
        pos.y += 2;
        }

The "Loop through them and test each one" :

for(int i = 0;i<43;i++)
    {
        for(int j = 0; j<5;j++)
    {
        if(mapArray[i][j] == 1)
        {
            STileGrass.setPosition(i * 70, (j * 70) + offsety);
        if(upkey == false && !SAdventurer.getGlobalBounds().intersects(STileGrass.getGlobalBounds()))
        {
        pos.y += 2;
        }
        }
    }
    }

Well you see,the problem is that it isn't working(and I know I'm not doing it right,but I don't know what am I doing wrong),if the "up" key isn't pressed and the player doesn't touches the ground, the gravitation will come in,and it really goes like this...just not like I want,it amplifies(what the for loop is for),but it only is going right with the last piece of the map.
Another problem is that I don't really know if it is really a tile map,because a tile map is something like this: http://imgur.com/htqx9tZ
,while my "map" is something like:
http://imgur.com/a/xtIGP
Note:This are not screenshots,these are from google.