SFML 2.0 AABB Class Help

[eXpl0it3r]

Code: [Select]

min.x = playerTexture.getSize().x;
max.y = playerTexture.getSize().y;
And

Code: [Select]


        min.x = blockTexture.getSize().x;
max.y = blockTexture.getSize().y;


This is correct, isn't it?

Not really... Where are you checking positions? And why is your min depended on the size?
You most likely would want to go with the sprite instead.

        min = playerSprite.getPosition();
        max.x = playerSprite.getGlobalBounds().width;
        max.y = playerSprite.getGlobalBounds().height;

And

        min = blockSprite.getPosition();
        max.x = blockSprite.getGlobalBounds().width;
        max.y = blockSprite.getGlobalBounds().height;

Alternatively you can also use the global bound for the min, just access .top and .left.

[Tweezy]

Code: [Select]

//level.cpp
       
   min.x = blockSprite.getGlobalBounds().left;
   min.y = blockSprite.getGlobalBounds().top;
 
   max.x = blockSprite.getGlobalBounds().width;
   max.y = blockSprite.getGlobalBounds().height;
 
//player.cpp
       
   min.x = playerSprite.getGlobalBounds().left;
   min.y = playerSprite.getGlobalBounds().top;
 
   max.x = playerSprite.getGlobalBounds().width;
   max.y = playerSprite.getGlobalBounds().height;
Now it is always returning true

[eXpl0it3r]

Now it is always returning true

As this requires very little code to test, could you write an example from ground that can be directly compiled?

Tbh as this is a very simple uni task, shouldn't you be able to solve it on your own with a bit more thinking and debugging?

[Tweezy]

Just found the issue. I was setting the blockSprite position in my game::init(). Now, for some reason this was creating a block of 600 + 50 instead of 200*50. Moving the setPosition to my default constructor fixed this bug.

[eXpl0it3r]

That's why this

Yes, just add some nice std::cout or set a break point for all the numbers and check them manually.

should have been your first action.

Never trust your code, always make sure it does what it should.

[Tweezy]

That's why this

Yes, just add some nice std::cout or set a break point for all the numbers and check them manually.

should have been your first action.

Never trust your code, always make sure it does what it should.

Yeah, thanks for the advice! Should be able to sort this now, thanks for your help

[eXpl0it3r]

No problem and good luck!

[math1992]

Code: [Select]

min.x = playerTexture.getSize().x;
max.y = playerTexture.getSize().y;
And

Code: [Select]


        min.x = blockTexture.getSize().x;
max.y = blockTexture.getSize().y;


This is correct, isn't it?

Not really... Where are you checking positions? And why is your min depended on the size?
You most likely would want to go with the sprite instead.

        min = playerSprite.getPosition();
        max.x = playerSprite.getGlobalBounds().width;
        max.y = playerSprite.getGlobalBounds().height;

And

        min = blockSprite.getPosition();
        max.x = blockSprite.getGlobalBounds().width;
        max.y = blockSprite.getGlobalBounds().height;

Alternatively you can also use the global bound for the min, just access .top and .left.

You forgot to add position to max, it would be:

        min = playerSprite.getPosition();
                        max = min;       //max = playerSprite.getPosition();
        max.x += playerSprite.getGlobalBounds().width;
        max.y += playerSprite.getGlobalBounds().height;

And

        min = blockSprite.getPosition();
                        max = min;
        max.x += blockSprite.getGlobalBounds().width;
        max.y += blockSprite.getGlobalBounds().height;