Checking if a sprite flipped?

[Noctune9]

I'm inheriting from a sprite in order to create my lighting system. Unfortunately, there doesn't seem to be any function which tells me if a sprite is flipped, making it impossible for me to implement the Render function properly.

I could implement it myself easily, but I think it would be better if it became fixed in SFML.

[Laurent]

I could implement it myself easily

You should do so, because I don't think that this feature will be implemented soon.

[Nexus]

I could implement it myself easily, but I think it would be better if it became fixed in SFML.

How? By overriding the FlipX() and FlipY() methods? That can be dangerous since those are not virtual functions.

Laurent, what is the reason not to provide IsFlippedX() and IsFlippedY() member functions? The sf::Sprite class already owns two corresponding bool variables. I don't think duplicating the information makes much sense.

[Laurent]

How?

Store your own booleans, that you update when you call FlipX/FlipY.
Or modify SFML to add the accessors inside sf::Sprite.

[Nexus]

Store your own booleans, that you update when you call FlipX/FlipY.

That's what I actually meant by overriding the methods. But your object can become inconsistent if you ever access it via sf::Sprite* or sf::Sprite& because FlipX() and FlipY() are not virtual.

Why are there no getters like IsFlippedX() and IsFlippedY()? Is there a special reason, or is this feature just required too rarely? And as I said, I think it's not very useful to have the two bools again, if they already exist in the base class. At least, I find it a bit questionable to provide setters only...

[Laurent]

Don't worry, it will probably be implemented one day

[Nexus]

Okay, thanks for the notice.

[Haze]

I just discovered there is currently no way to tell is a sprite is flipped, and found this old topic.

Laurent, do you still plan to add those two little getters IsFlippedX / IsFlippedY ?
I concur with Nexus, overriding FlipX/FlipY methods is just an hacky workaround.

[Laurent]

I think I'll just remove these functions from sf::Sprite, and instead allow negative scale factors.

[Haikarainen]

I think I'll just remove these functions from sf::Sprite, and instead allow negative scale factors.

That sounds like a REALLY great idea!

[gsaurus]

I think I'll just remove these functions from sf::Sprite, and instead allow negative scale factors.

That sounds like a REALLY great idea!

One that I requested a long while ago :lol:
One problem with the old flip functions is that it flips the texture, not the sprite. In most situations we want to flip around the sprite origin, which is, in practice, a negative scaling.