Définition de 2 classes possédant chacune des Ptr vers des objets de l'autre

Bonjour,

Le titre n'est pas très représentatif mais je ne sais pas comment le dire...

Contexte :

Je développe une application utilisant un maillage (algo de DELAUNAY) et je cré plusieurs objet dont "Triangle" et "Segment". Pour mettre en place cet algo j'ai besoin que ma classe Triangle possède 3 pointeurs vers des objets type Segment (logique un triangle à bien 3 arêtes) donc la pas de Pb mais j'ai également besoin que ma classe Segment possède 2 pointeurs vers des objets type Triangle (les 2 triangles juxtaposés du maillage partageant le segment en question.)

J'ai donc voulu déclarer mes classes comme cela :

Pour la classe Triangle :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#ifndef TRIANGLE #define TRIANGLE   #include <Segment.h>   class Triangle {   public :   Segment *S1, *S3, *S3; }; #endif

---

Pour la classe Segment :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#ifndef SEGMENT #define SEGMENT   #include <Triangle.h>   class Segment {   public :   Triangle *Tv1, *Tv2; }; #endif

---

Le Hic c'est que ca plante à la compilation sur la ligne 11 de la classe Triangle : 'Segment' does not name a type'.

=> Si je comprend bien c'est qu'il ne connait pas le type 'Segment' à ce moment là alors qu'il est bien défini en include.

Si je supprime #include <Triangle.h> de la classe Segment, le problème disparait mais je ne peux alors plus avoir de pointeurs vers des triangles dans cette classe ce qui est problématique.

Je sens bien que ca ne peut pas marcher comme cela puisque chaque classe a besoin de l'autre pour être définie mais comment solutionner proprement ce problème ?

Merci d'avance pour votre aide, et désolé si la réponse est obvious, je débute en programmation.

Bien cordialement,

Nico

Il faut utiliser une forward declaration dans les header.
En fait il faut toujours préférer les forward declaration dans les header et favoriser les include dans les fichiers sources.
:faq: http://cpp.developpez.com/faq/cpp/?p...-une-a-l-autre

Merci Bousk pour ta réponse.

J'ai consulté le lien dans la FAQ qui explique ce qu'est la déclaration anticipée. J'ai modifié ma classe Segment ainsi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

#ifndef SEGMENT #define SEGMENT #include <Triangle.h> : // Ligne supprimée Class Triangle; class Segment { public : Triangle *Tv1, *Tv2; };

---

Cela compile sans problème.

Par contre peux tu me détailler ce que tu veux dire par :

Citation:

---

En fait il faut toujours préférer les forward declaration dans les header et favoriser les include dans les fichiers sources.

---

Car au final je déclare toujours mes classes dans un header (un par classe) et y ajoute en tête tous les includes nécessaires au fonctionnement de la classe. Exemple concret :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#ifndef TRIANGLE #define TRIANGLE   #include <QPoint> #include <QVector>   #include <cmath> #include <C_Mesh/Segment.h>   #include <C_Generique/Maths.h>   class Triangle {   // } #endif

---

Comment devrais je appliquer ce principe ici par exemple ? le pourrais je ?

Merci encore pour ton intervention

Bonjour.

La forward declaration ne fonctionne qu'avec des pointeurs, si tu déclares :

Code:

---

1 2 3

class Foo{ Objet obj; };

---

, tu seras obligé d'inclure la déclaration d'Objet (#include<objet.h> //ou autre.

Si tu utilises exclusivement des pointeurs d'Objet

Code:

---

1 2 3

class Foo{ Objet *obj; };

---

, tu peux sans problème écrire class Objet; avant de déclarer class Foo {}.
En revanche l'include sera déplacé dans la définition de Foo, dans le fichier cpp.

Merci Daïmanu pour ta réponse, je crois que je comprend bien la déclaration anticipée.

Est ce obligatoire de placer le #include <Objet.h> dans le fichier Foo.cpp ou pourrait on créer un fichier GenericInclude.h qui contiendrait tous les includes nécessaires à toutes les classes du projet ? L'avantage serait de ne les spécifier qu'une seule fois ...

Tu peux créer un gros fichier d'include.

L'inconvénient est que ça va allonger la compilation, vu que chaque fichier cpp va recevoir tout le contenu de tous les fichiers inclus.
De plus, il risque d'y avoir des problèmes de collision si les fichiers définissent des macros / classes / variables avec le même nom.

Généralement on recommande de n'inclure uniquement ce dont on a besoin.

Ah ok car meme si je cré un gros fichier d'include, il faudra que je le mette en entete de chaque fichier .cpp ? Dans ce cas c'est clair ca vaut pas le coup. je pensais ne l'inclure que dans le main par exemple...

Autrement je viens de tester un exemple ou j'ai fait :
- une déclaration anticipée d'une classe A dans une classe B
- alors que B contient des pointeurs vers A ET des objets de type A
- sans ajouter #include <A> dans le fichier B.cpp

=> Ca a compiler parfaitement... Est ce que cela est normal ? Pourrait s'expliquer par le fait que #include <A> est présent dans d'autres fichiers que B.cpp ?

Citation:

---

Envoyé par nicosmile_700

Ah ok car meme si je cré un gros fichier d'include, il faudra que je le mette en entete de chaque fichier .cpp ?

---

C'est recommandé (mais pas obligatoire).

Citation:

---

Envoyé par nicosmile_700

Ca a compiler parfaitement... Est ce que cela est normal ?

---

Ça arrive souvent qu'on est pas rigoureux avec les includes.
Prenons un exemple

Code:

---

1 2 3

class Mot { std::string m_mot; };

---

Ça ne compilera pas, puisque Mot ne sait pas ce qu'est un std::string.

Par contre

Code:

---

1 2 3

//fichier quelconque cpp #include <string> #include "mot.h"

---

fera que ça compile, car le string a été inclus avant le mot.
Voila un article qui parle de ce sujet.

Citation:

---

Envoyé par nicosmile_700

...

---

Ca veut juste dire ce qui est écrit : préférer des forward declaration aux includes dans les headers. :weird:
Quand à ton exemple, vu que tu coupes le corps de la classe, on pourra difficilement te dire autre chose que "peut-être".

Citation:

---

Envoyé par Daïmanu

La forward declaration ne fonctionne qu'avec des pointeurs

---

Et les références.

Citation:

---

Envoyé par nicosmile_700

Est ce obligatoire de placer le #include <Objet.h> dans le fichier Foo.cpp

---

Si tu manipules un objet contenu dans Objet.hpp alors oui. :weird:

Citation:

---

Envoyé par nicosmile_700

ou pourrait on créer un fichier GenericInclude.h qui contiendrait tous les includes nécessaires à toutes les classes du projet ? L'avantage serait de ne les spécifier qu'une seule fois ...

---

- toutes les classes n'ont pas besoin de tous les includes
- on essaye de limiter les dépendances en général
- si t'as envie de passer plus de temps à compiler qu'à programmer tu peux t'y risquer

Citation:

---

Envoyé par Bousk

Il faut utiliser une forward declaration dans les header.
En fait il faut toujours préférer les forward declaration dans les header et favoriser les include dans les fichiers sources.
:faq: http://cpp.developpez.com/faq/cpp/?p...-une-a-l-autre

---

Je suis d'accord mais il faut préciser que ce n'est possible qu'à condition d'utiliser des pointeurs ou des références vers les objets ces ces classes, pas directement des instances. Ce qui est d'ailleurs aussi une bonne pratique.

Citation:

---

Envoyé par olreak

Je suis d'accord mais il faut préciser que ce n'est possible qu'à condition d'utiliser des pointeurs ou des références vers les objets ces ces classes, pas directement des instances. Ce qui est d'ailleurs aussi une bonne pratique.

---

Conceptuellement si tu arrives à avoir une classe A composée d'une classe B elle-même composée d'une classe A

Code:

---

1 2 3

class A { B b; }; class B { A a; };

---

C'est même plus du wtf là, c'est carrément de la folie. Tu te retrouves avec A qui contient un B qui contient un A, qui..... :cfou:
Je préfère pas penser que quelqu'un puisse l'imaginer.

Merci à tous pour vos réponses, je pense avoir bien saisi la bonne facon de faire. Ne reste plus qu'à l'appliquer en reprenant le code déjà écrit !

Encore merci