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.

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

[Daïmanu]

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

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 :

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

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 :

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

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.

[Bousk]

...

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

La forward declaration ne fonctionne qu'avec des pointeurs

Et les références.

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.

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

[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