Discussion :

[vince3320]

Bonjour,

je viens d'hériter d'un code et je remarque les lignes suivantes :

Code :

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

1
2
3
4
5
6
7
8
 
// fichier toto.c
 
static float sqrarg;
 
 
// fichier toto.h
#define SQR(a) 		((sqrarg = (a)) == 0.0 ? 0.0 : sqrarg * sqrarg)

Quel est l'interêt de tout cela?
Y a t'il une raison a toute cette "complication"?

Ne peut on pas plutot faire :

Code :

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

1
2
 
#define SQR(a) 		(a) a * a

Merci

[Laurent Gomila]

Une façon bien sale de ne pas évaluer deux fois le paramètre a.
En vrai C++ (note que ton code est en C) on ferait ceci :

Code :

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

1
2
3
4
5
template <typename T>
inline T Sqr(T a)
{
    return a * a;
}

[vince3320]

Une façon bien sale de ne pas évaluer deux fois le paramètre a.

C'est vrai. Mais tu fais une affectation de plus...
Est ce horrible d'évaluer un paramètre 2 fois?

Oui ce code est du C compilé en C++. J'aurai peut-etre du aller sur le forum C...

Pour toi la bonne methode est celle du template?

Quel est la difference entre

Code :

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

1
2
3
4
5
6
 
template <typename T>
inline T Sqr(T a)
{
    return a * a;
}

et

Code :

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

1
2
 
#define SQR(a) 		(a) a * a

hormis le langage?

Merci!

[Médinoc]

Oui ce code est du C compilé en C++.

Très mauvais. Tu dois corriger cela immédiatement.

Quel est la difference entre <snip> et <snip> hormis le langage?

Si tu fais SQR(a++), tu peux avoir une surprise.
Avec Sqr(a++), aucun problème.

[vince3320]

Très mauvais. Tu dois corriger cela immédiatement.

Je dois compiler en C++...
Pourquoi est ce si mauvais?

Je pose beaucoup de questions...

[Médinoc]

Le C n'est pas un sous-ensemble du C++.

Autant une bonne partie des headers fait partie du sous-ensemble commun aux deux langages, autant le code C lui-même ne doit pas approcher un compilateur C++: Il faut toujours séparer en deux fichiers source.

Bref, toto.h peut sûrement être inclus en C++, mais pas toto.c.
De plus, je me méfie des headers qui utilisent une variable globale statique: Cela signifie généralement qu'ils sont dédiés au fichier source où ladite variable globale est déclarée, et que cette partie du header (voire le header entier) devrait être directement dans le fichier source.