ifndef et redéfinition

Bonjour

Je coince sur un problème tout bête.
J'ai défini dans un fichier.h un certain nombre de structures avec des fonctions amies :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33

#ifndef M_PI_2 #define M_PI_2 1.57079632679489661923 #endif   #ifndef H2D_H #define H2D_H   typedef struct h2Dvector { double x, y, t;   h2Dvector () : x(0), y(0), t(0) {}; h2Dvector (double a, double b, double c) : x(a), y(b), t(c) {}; // binary operators friend h2Dvector operator + ( const h2Dvector&, const h2Dvector&); friend h2Dvector operator - ( const h2Dvector&, const h2Dvector&); friend h2Dvector operator * ( double, const h2Dvector& ); friend h2Dvector operator * ( const h2Dvector&, const h2Dmatrix&);   } h2Dpoint, h2Dvector, *p2Dpoint, *p2Dvector;   h2Dvector operator + ( const h2Dvector& V1, const h2Dvector& V2) { h2Dvector V; V.x = V1.x + V2.x; V.y = V1.y + V2.y; if ((V1.t > 0) || (V2.t > 0)) V.t = 1; else V.t = 0; return V; }   (et coetera...)   #endif

---

J'ai, comme il se doit, encadré ces déclarations par un #ifndef et un #endif.
Lorsque je fais une fois appel à ce fichier .h, tout se passe bien. Je compile et ça s'exécute comme il faut.

entête hTopo.h :

Code:

---

1 2 3 4 5 6 7 8 9

#ifndef HTOPO_H #define HTOPO_H   #include "h2D.h" // le fichier d'entête ci-dessus   (etc...)   #endif

---

fichier hTopo.cpp :

Code:

---

1 2 3 4

#include "htopo.h"   (etc...)

---

Si maintenant j'ajoute un autre fichier faisant appel à ce fichier hTopo, patatras, ça ne marche plus !

entête autrefichier.h :

Code:

---

1 2 3 4 5 6 7 8 9

#ifndef OFICHIER_H #define OFICHIER_H   #include "hTopo.h" // le fichier d'entête ci-dessus   (etc...)   #endif

---

fichier autrefichier.cpp :

Code:

---

1 2 3 4

#include "autrefichier.h"   (etc...)

---

Je me retrouve avec une erreur LNK2005 de redéfinition de structire et de fonctions :

Code:

---

1 2	hTopo.obj : error LNK2005: "struct h2Dvector __cdecl operator+(struct h2Dvector const &,struct h2Dvector const &)" (??H@YA?AUh2Dvector@@ABU0@0@Z) déjà défini(e) dans autrefichier.obj

---

Quelqu'un a-t'il une idée qui pourrait m'aider ?
Merci par avance.

Declare tes fonctions libres définis dans les fichiers headers comme étant inline

Et bien oui, ça marche ! Merci...

Mais ça veut dire que le code sera ajouté par le compilateur à chaque appel de la fonction, n'est-ce pas ? Ce n'est pas vraiment optimum en terme de taille du code objet...

(et encore une fois merci pour cette réponse ultra rapide)

Citation:

---

Envoyé par antonien

Mais ça veut dire que le code sera ajouté par le compilateur à chaque appel de la fonction, n'est-ce pas ? Ce n'est pas vraiment optimum en terme de taille du code objet...

---

Pas nécessairement, c'est au final le compilateur qui a le dernier mot et il peut très bien ne pas inliner la fonction malgré la présence du inline dans le code.
Si tu souhaites vraiment avoir ces fonctions définies dans le header, il n'y a pas trop le choix.

La question que je me pose surtout est pourquoi définir ces fonctions dans un .h et pas dans un .cpp ? (Et également pourquoi faire un typedef de ta structure sous plusieurs noms, y compris en masquant des pointeurs parfois ?)

Tu as raison pour le typedef, il n'apporte rien et je j'ai enlevé.
Pour les fonctions, je croyais qu'il fallait mettre les fonctions indépendantes amies dans le header avec les structures qui font appel à elles. N'est-ce pas le cas?

non.

d'autant plus que ici le friend ne sert a rien. La bonne facon d'ecrire ce genre d'operateur est d'ecrire un operateur += membre dans la classe et faire que operator+ appele ce dernier.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

struct h2Dvector { double x, y, t;   h2Dvector () : x(0), y(0), t(0) {}; h2Dvector ( const h2Dvector& src ) : x(src.x), y(src.y), t(src.z) {}; h2Dvector (double a, double b, double c) : x(a), y(b), t(c) {};   h2Dvector& operator+=(const h2Dvector& r) { x += r.x; y += r.y; z += r.z; return *this; } }   h2Dvector operator + ( const h2Dvector& l, const h2Dvector& r) { h2Dvector that(l); that += r; return that; }

---

Enfin on est en C++, pas en C, le typedef struct est inutile.
Je rajouterais que tes membres x,y,z pourrait etre privés et accessible via des accesseurs.

Bravo! C'est une écriture très élégante et qui évite de passer par les "friend" que je ne maîtrise pas très bien.
Merci du tuyau.