Forum de la communauté SFML

Général => Suggestions de nouvelles fonctionnalités => Discussion démarrée par: Valchap le Octobre 18, 2019, 10:27:37 pm

Titre: Angles en radians pour Transformable
Posté par: Valchap le Octobre 18, 2019, 10:27:37 pm

Bonjour,

Comme les fonctions trigonométriques du c++ utilisent des radians il serait pratique d'avoir des méthodes pour définir des rotations sans devoir faire de transformation en degrés.

Je propose donc d'ajouter les méthodes suivantes dans Transformable pour simplifier les rotations.

setRotationInRadians(float angle);
rotateInRadians(float angle);
getRotationInRadians();
 

Si l'idée semble convenable je peux faire la pull request pour ajouter ces méthodes

Titre: Re: Angles en radians pour Transformable
Posté par: Laurent le Octobre 19, 2019, 05:54:38 pm

Pourquoi doubler toutes les fonctions qui prennent des angles en paramètre, plutôt que d'ajouter une unique fonction radiansToDegrees ?

Titre: Re: Angles en radians pour Transformable
Posté par: Valchap le Octobre 19, 2019, 09:12:45 pm

Je trouvais un peu plus clean d'ajouter trois méthodes pour les radians que de devoir imbriquer une fonction de conversion à chaque appel des méthodes pour les degrés.

Titre: Re: Angles en radians pour Transformable
Posté par: Laurent le Octobre 19, 2019, 09:25:30 pm

D'un point de vue de la conception c'est pourtant plus "clean" de centraliser la conversion dans une unique fonction, et de garder une seule version des fonctions dans l'API. C'est juste plus verbeux.

Titre: Re: Angles en radians pour Transformable
Posté par: Valchap le Octobre 20, 2019, 07:39:46 pm

Dans ce cas je vais continuer d'utiliser une fonction de conversion.  ;D