Discussion :

[Jloox]

Bonjour,

J'ai un petit problème assez bizarre. Enfin c'est toujours ce qu'on dit quand on a pas compris comment le résoudre ...

Je vais essayer de me faire comprendre : j'ai reproduit de manière simplifiée mon problème qui ne contient plus que 3 classes. En plus, elles sont quasi vides !! Il est clair que le problème vient de l'inclusion des fichiers.

Premièrement j'ai une classe A. J'ai donc un fichier A.h qui ressemble à ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#if !defined (A_H)
  #define A_H
 
#include "Z.h"
 
class A  {
     A();
     ~A();
 };
 
#endif

Si j'inclus ici le fichier Z.h c'est parce que j'ai besoin d'utiliser des objets Z dans cette classe. Le fichier d'entête de la classe Z ressemble à ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#if !defined (Z_H)
  #define Z_H
 
#include "B.h"
 
class Z  {
    Z();
    ~Z();
};
 
#endif

Rien de mal encore ... J'imagine que vous voyez où on va : j'ai inclus le fichier B.h parce que j'ai besoin d'objets B dans la classe Z (non montré ici pour simplifié). La classe B hérite de la classe A :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#if !defined(B_H)
   #define B_H
 
 #include "A.h"
 
 class B : public A {
 public:  
     B() {}
     ~B() {}
 };
 
 #endif

Evidemment, je suis obligé d'inclure le fichier A.h ici. Les fichiers A.cpp, B.cpp et Z.cpp contiennent juste l'inclusion de leur header.

Donc en fait on tourne un peu en rond dans tout ça. Lors de la compilation j'obtiens le message : expected class-name before '{' token sur la ligne 6 de mon B.h.

Je voudrais juste savoir si c'est possible de faire ceci en C++ (et alors comment ?) ou bien il faut que je revois ma modélisation ?

Merci d'avance.

[Aurelien.Regat-Barrel]

Tu es une dépendance cyclique, il serait bon d'essayer de la supprimer. Mais sinon:
http://c.developpez.com/faq/cpp/?pag...erence_croisee

[ep31]

C'est normal, si B.cpp est ton premier fichier compilé, ça donne :
j'inclus A.h
A_H n'est pas défini donc j'y rentre.
J'inclus Z.h
Z_H n'est pas défini donc j'y rentre.
Et Z.h inclut B.h qui est a déjà défini B_H
donc il passe directement au prototype de classe qui ne trouve pas la classe A.

[Swoög]

utilise

Code :

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

#ifndef A_H

plutôt que

Code :

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

#if !defined (B_H)

c'est la première fois que je vois cette syntaxe, je ne suis pas sûr qu'elle soit valide

ensuite, pourquoi avoir mis le corps du constructeur et du destructeur de B dans son .h ? (même si je doute que le problème vienne de là)

enfin, tes .h s'incluent mutuellement...
il y a un problème...
A a besoin de Z qui a besoin de B qui a besoin de A (puisqu'il en hérite)

on se retrouve avec A a besoin de A... et surtout, très surprenant, A a besoin de B !!!

es-tu sûr de ne pas avoir fait d'erreur de modélisation ?

sinon essaie de rajouter

Code :

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

class A;

dans ton A.h (avant le #ifndef)
et

Code :

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

class B;

dans le B.h (avant le #ifndef) etc...

ça permettra de dire que les classes existent sans pour autant causer de déclaration multiples, mais je pense qu'on va tomber sur un "A non spécifiée complètement"...

[Jloox]

Hé ben en tout cas vous ne perdez pas de temps pour répondre !!

utilise

Code :

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

#ifndef A_H

plutôt que

Code :

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

#if !defined (B_H)

c'est la première fois que je vois cette syntaxe, je ne suis pas sûr qu'elle soit valide

La synthaxe est correcte. Je sais plus où je l'ai vu mais une bibliothèque l'utilise.

ensuite, pourquoi avoir mis le corps du constructeur et du destructeur de B dans son .h ? (même si je doute que le problème vienne de là)

Là c'était juste pour tout mettre dans le même fichier, histoire de simplifier mon post.

on se retrouve avec A a besoin de A... et surtout, très surprenant, A a besoin de B !!!

Euh ... non A n'a pas besoin de B.

es-tu sûr de ne pas avoir fait d'erreur de modélisation ?

Ben je crois que je vais revoir ça ...

sinon essaie de rajouter

Code :

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

class A;

dans ton A.h (avant le #ifndef)
et

Code :

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

class B;

dans le B.h (avant le #ifndef) etc...

ça permettra de dire que les classes existent sans pour autant causer de déclaration multiples, mais je pense qu'on va tomber sur un "A non spécifiée complètement"...

Non ça déconne : invalid use of undefined type `struct A' et forward declaration of `struct A'.

Bon merci bien. Je crois que je vais tout simplement revoir ma modélisation !!

[Swoög]

Euh ... non A n'a pas besoin de B.

A a besoin de Z qui a besoin de B !!
donc A a besoin de B... la dépendance est transitive...$

Non ça déconne : invalid use of undefined type `struct A' et forward declaration of `struct A'.

Bon merci bien. Je crois que je vais tout simplement revoir ma modélisation !!

oé me suis planté, je voulais dire ce dont a parlé Aurelien...