Negative calculations when using triggernomatry so that an object faces cursor

[Jim70]

Sorry for the long title. Whenever the rotation of the object exceeds 180(degrees) when the rotation is supposed to be 350 its -10, 340 = -20, 330 = -30, and so on.

       const float PI = 3.14159265;

   int mx = sf::Mouse::getPosition(window).x;
   int my = sf::Mouse::getPosition(window).y;

   int tmp_rot = (atan2(my - y, mx - x)) * 180 / PI;

   rot = tmp_rot;

[Nexus]

This is expected, you should read the documentation of standard library functions

Also, correct C++ is std::atan2, not atan2.

[Jim70]

This is expected, you should read the documentation of standard library functions

Also, correct C++ is std::atan2, not atan2.

gotcha. Ive given a toss at fixing that with no success. Any suggestions for fixing?

[Nexus]

What is "fixing"? That's correct, as I said.

If you need a [0, 360[ range, use std::fmod(x + 360.f).

[Jim70]

What is "fixing"? That's correct, as I said.

If you need a [0, 360[ range, use std::fmod(x + 360.f).

Doesn't fmod require 2 arguments? and is x supposed to be the rotation or the x specified in the code?

[Nexus]

True, I forgot the second one. With a bit of thinking, I'm sure you'll find it out yourself
x is the value in [-180, 180[ that you want to map to [0, 360[.

By the way, you could have a look at Thor's Vectors module, it really simplifies operations on vectors and angles.

[Jim70]

True, I forgot the second one. With a bit of thinking, I'm sure you'll find it out yourself
x is the value in [-180, 180[ that you want to map to [0, 360[.

By the way, you could have a look at Thor's Vectors module, it really simplifies operations on vectors and angles.

Hm, so x = std::fmod(0, 360)?

[SpeCter]

More like x = std::fmod(x + 360,360)
Yours would yield 0 all the time.

[Laurent]

It can be simpler in this particular case:

if (x < 0)
    x += 360;