Discussion :

[ChPr]

Bonsoir à toutes et à tous,

Quels sont les avantages/inconvénients relatifs de ces deux manières d'obtenir un résultat (le code n'est ici qu'un exemple) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
#define DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(deg,min,sec) (deg) >= 0 ? ((float)(deg)+(float)(min)*0.0166666667+(float)(sec)*0.0002777778) : ((float)(deg)-(float)(min)*0.0166666667-(float)(sec)*0.0002777778)
 
float dmsTOdd(float deg, float min, float sec) {
  return (deg) >= 0 ? ((float)(deg) + (float)(min) * 0.0166666667 + (float)(sec) * 0.0002777778) : ((float)(deg) - (float)(min) * 0.0166666667 - (float)(sec) * 0.0002777778);
}

Merci pour vos renseignements.

Pierre

[dalfab]

Bonjour,

Partons de ton exemple :
combien vaut : 180+DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(10,0,0) ? réponse 10.0 !!
combien vaut : int dg=90; DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(++dg,0,0) ? réponse 92 !!

Si on ajoute inline avant la fonction : inline float dmsTOdd(float deg, float min, float sec) elle sera implémentée avec un compacité au moins équivalente à la macro.

Donc avantage à la macro pour un traitement du type fonction : NEANT

[ChPr]

J'avoue n'avoir strictement rien compris.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

combien vaut : 180+DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(10,0,0) ? réponse 10.0 !!

veut donc dire qu'on ne peut pas ajouter une macro et un nombre ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

combien vaut : int dg=90; DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(++dg,0,0) ? réponse 92 !!

Ici, cela veut dire que dg est incrémenté deux fois ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Si on ajoute inline avant la fonction : inline float dmsTOdd(float deg, float min, float sec) elle sera implémentée avec un compacité au moins équivalente à la macro.

Je te crois.

Mais de là en tirer la conclusion que tu fais, alors là, je suis complètement perdu.

Cordialement.

Pierre

[dalfab]

Une macro c'est en fait un remplacement d'un texte, ça peut avoir des effets indésirables.
180+DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(10,0,0) devient en remplaçant
180+(10) >= 0 ? ((float)(10)+(float)(0)*0.0166666667+(float)(0)*0.0002777778) : ((float)(10)-(float)(0)*0.0166666667-(float)(0)*0.0002777778), donc le 180 s'ajoute ici au test. La syntaxe est trompeuse!
int dg=90; DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(++dg,0,0) devient 180+(++dg) >= 0 ? ((float)(++dg)+(float)(0)*0.0166666667+(float)(0)*0.0002777778) : ((float)(++dg)-(float)(0)*0.0166666667-(float)(0)*0.0002777778).
Les macros présentent donc un piège. Les fonctions inline ont la capacité de mieux vérifier les types et d'éviter ces effets de bord.
Autrefois c'était un moyen d'optimiser du code, depuis il y a les fonctions inline et les template qui sont capable de faire au moins aussi bien et plus surement.

[ChPr]

Ah, et bien là, je comprends beaucoup mieux. Merci "dalfab".

Cordialement.

Pierre

[foetus]

Et ouais, tu as fais l'erreur de ne pas surparenthèser ta macro : c'est direct la baffe/ le point qui tue

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

#define DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(deg,min,sec) ((deg >= 0)? ((float)(deg)+(float)(min)*0.0166666667+(float)(sec)*0.0002777778): ((float)(deg)-(float)(min)*0.0166666667-(float)(sec)*0.0002777778))

Et ensuite tu as eu la deuxième réponse classique anti-macro : Comme tu es bête, il faut que tu fasses attention parce que ta macro tu peux la mettre partout/ l'utiliser à mauvais escient. Et même le compilateur ni peut rien.

Donc plus une macro de ce style:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
#define DEGREES_MINUTES_SECONDS_TO_DECIMAL_DEGREES(deg, min, sec, result) \
    if ((deg >= 0) { \
        result = ((float) (deg) + (float) (min) * 0.0166666667 + (float) (sec) * 0.0002777778); \
    } else { \
        result = ((float) (deg) - (float) (min) * 0.0166666667 - (float) (sec) * 0.0002777778); \
    }