Bienvenue, Invité. Merci de vous connecter ou de vous inscrire.
Avez-vous perdu votre e-mail d'activation ?

Auteur Sujet: Angles en radians pour Transformable  (Lu 55 fois)

0 Membres et 1 Invité sur ce sujet

Valchap

  • Newbie
  • *
  • Messages: 3
    • Voir le profil
Angles en radians pour Transformable
« 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

Laurent

  • Administrator
  • Hero Member
  • *****
  • Messages: 32143
    • Voir le profil
    • SFML's website
    • E-mail
Re: Angles en radians pour Transformable
« Réponse #1 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 ?
Laurent Gomila - SFML developer

Valchap

  • Newbie
  • *
  • Messages: 3
    • Voir le profil
Re: Angles en radians pour Transformable
« Réponse #2 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.

Laurent

  • Administrator
  • Hero Member
  • *****
  • Messages: 32143
    • Voir le profil
    • SFML's website
    • E-mail
Re: Angles en radians pour Transformable
« Réponse #3 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.
Laurent Gomila - SFML developer

Valchap

  • Newbie
  • *
  • Messages: 3
    • Voir le profil
Re: Angles en radians pour Transformable
« Réponse #4 le: Octobre 20, 2019, 07:39:46 pm »
Dans ce cas je vais continuer d'utiliser une fonction de conversion.  ;D