Discussion :

[antonien]

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 :

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

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 :

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

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 :

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

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 :

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

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 :

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

1
2
3
4
 
#include "autrefichier.h"
 
(etc...)

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

Code :

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

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.

[Invité]

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

[antonien]

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)

[gl]

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 ?)

[antonien]

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?

[Joel F]

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 :

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

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.