Discussion :

[nicosmile_700]

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 :

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

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 :

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

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

[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.
http://cpp.developpez.com/faq/cpp/?p...-une-a-l-autre

[nicosmile_700]

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 :

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

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 :

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 :

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
#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

[Daïmanu]

Bonjour.

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

Code :

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

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 :

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

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.

[nicosmile_700]

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 ...

[Daïmanu]

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.

[azias]

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.
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.

[Bousk]

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 :

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

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.....
Je préfère pas penser que quelqu'un puisse l'imaginer.

[nicosmile_700]

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