Problème d'inclusions imbriquées

Version imprimable

Bonsoir,

j'ai deux headers (pour simplifier...) définis comme ça (je n'ai mis que les parties intéressantes):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 #ifndef _LAYOUT_H #define _LAYOUT_H #include "point.h" #include "plane.h" #include "scene.h" typedef struct { Plane plane; Plane scrollable; Point refresh; void *memory; } Layout; void Layout_DrawScene(Layout *this, Scene *scene, Point *position); #endif /* _LAYOUT_H */
et le deuxième, qui inclut le premier:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #ifndef _SCENE_H #define _SCENE_H #include "layout.h" #include "character.h" #include "level.h" typedef struct { Layout screen; Level level; Character player; } Scene; #endif /* _SCENE_H */
Mais ça ne compile pas, car les deux headers s'incluent l'un l'autre (j'ai déja vu la réponse quelquepart mais je n'arrive pas à remettre la main dessus) :oops:

Sans l'avoir testé :

Code:

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
 
/* On n'utilise pas un '_' en premier caractère donc je change... */
#ifndef	H_LAYOUT
#define	H_LAYOUT
 
struct sLayout;
 
#include "point.h"
#include "plane.h"
#include "scene.h"
 
 
typedef struct sLayout
{
	Plane		plane;
	Plane		scrollable;
	Point		refresh;
	void		*memory;
} Layout;
 
 
void		Layout_DrawScene(Layout *this, Scene *scene, Point *position);
 
 
#endif	/* _LAYOUT_H */

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#ifndef	H_SCENE
#define	H_SCENE
 
struct sScene;
 
#include "layout.h"
#include "character.h"
#include "level.h"
 
 
typedef struct sScene
{
	Layout		screen;
	Level		level;
	Character	player;
} Scene;
 
 
#endif	/* _SCENE_H */

14/08/2006, 00h23
b4u

non, toujours Expected specifier-qualifier-list before 'Layout' au moment de compiler :(

finalement je m'en suis sorti en remplaçant:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include "point.h" #include "plane.h" #include "scene.h" typedef struct { Plane plane; Plane scrollable; Point refresh; void *memory; } Layout; void Layout_DrawScene(Layout *this, Scene *scene, Point *position);
par
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #include "point.h" #include "plane.h" struct sScene; typedef struct { Plane plane; Plane scrollable; Point refresh; void *memory; } Layout; void Layout_DrawScene(Layout *this, struct sScene *scene, Point *position);
Maintenant, est-ce qu'il existe un moyen pour que le prototype

Code:

void Layout_DrawScene(Layout *this, struct sScene *scene, Point *position);

devienne

Code:

void Layout_DrawScene(Layout *this, Scene *scene, Point *position);

?

14/08/2006, 10h28
DaZumba

Citation:

Envoyé par b4u

Maintenant, est-ce qu'il existe un moyen pour que le prototype

Code:

void Layout_DrawScene(Layout *this, struct sScene *scene, Point *position);

devienne

Code:

void Layout_DrawScene(Layout *this, Scene *scene, Point *position);

?

Ben, tu fais un typedef struct au lieu de struct. Tu as su le faire pour Layout et Point, non?
Sinon, pour ton probleme d'inclusion, je ferai un fichier entete par type exporte (Layout, Scence et Point) et j'incluerais chaque type dont on a besoin pour le prototype de la fonction.
14/08/2006, 12h39
b4u

Citation:

Ben, tu fais un typedef struct au lieu de struct

tu veux dire, je redéfinis le type Scene (déja défini dans le second header du premier post, scene.h)?
la "définition incomplète" (je ne connais pas le terme précis - forward definition je crois)

Code:

struct sScene;

n'est utile que pour remplacer un

Code:

#include "scene.h"

dans layout.h (qui lui meme inclut layout.h, et c'est bien ça le probleme), pour que le prototype de la fonction Layout_DrawScene() puisse etre compilé (vu que celle-ci prend en paramètre un pointeur vers Scene)... enfin, je n'ai peut-etre pas bien compris ce que tu me suggères.

Citation:

Sinon, pour ton probleme d'inclusion, je ferai un fichier entete par type exporte (Layout, Scence et Point) et j'incluerais chaque type dont on a besoin pour le prototype de la fonction.

je ne vois pas bien, tu pourrais me donner un exemple rapide s'il te plait?
merci :)

Citation:

Envoyé par b4u

tu veux dire, je redéfinis le type Scene (déja défini dans le second header du premier post, scene.h)?

Il faut que les deux types soient coherent. Si tu fais un

Code:

struct sScene { ... };

d'un cote, tu es oblige de faire un

Code:

struct sScene *Scene

de l'autre. Ce que je proposais, c'est de faire un

Code:

typedef struct sScene { ... } tScene;

dans scene.h histoire de ne pas avoir a trainer un struct partout.

Citation:

Envoyé par b4u

je ne vois pas bien, tu pourrais me donner un exemple rapide s'il te plait?

Je proposais d'isoler les definitions de type des prototypes de fonction:
Ton scene.h devient t_scene.h (pour indiquer que c'est une definition de type uniquement)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #ifndef H_T_SCENE #define H_T_SCENE #include "t_layout.h" #include "character.h" #include "level.h" typedef struct { Layout screen; Level level; Character player; } Scene; #endif
ton layout.h devient t_layout.h
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #ifndef H_T_LAYOUT #define H_T_LAYOUT #include "point.h" #include "plane.h" typedef struct sLayout { Plane plane; Plane scrollable; Point refresh; void *memory; } Layout; #endif
Remarque que t_layout.h ne depend plus de t_scene.h -> on est tranquille.
Et le prototype de ta fonction, dans layout.h
Code:

1 2 3 4 5 6 7 8 9 10 11 #ifndef H_LAYOUT #define H_LAYOUT #include "point.h" #include "t_layout.h" #include "t_scene.h" void Layout_DrawScene(Layout *this, Scene *scene, Point *position); #endif
Ensuite, si le contenu des typedefs est cense etre prive a la fonction Layout_DrawScene, alors tu peux definir des types incomplets, et completement oublier les inclusions:
Code:

1 2 3 4 5 6 7 8 9 10 11 #ifndef H_LAYOUT #define H_LAYOUT typedef struct sPoint Point; typedef struct sScene Scene; typedef struct sLayout Layout; void Layout_DrawScene(Layout *this, Scene *scene, Point *position); #endif
Mais la, l'utilisateur de la fontion Layout_DrawScene n'a aucun moyen de connaitre le contenu des structures. Du coup, il faut creer un 'constructeur' pour chaque structure, etc. C'est ce qu'on appelle un Abstract Data Type ).

14/08/2006, 14h11
b4u

ok merci beaucoup pour ta réponse détaillée, c'est exactement ce qu'il me fallait
je vais étudier ça en détail :)

Citation:

Du coup, il faut creer un 'constructeur' pour chaque structure, etc.

ça tombe bien, c'est justement ce que j'ai actuellement pour chacune de mes structures. En fait j'essaye d'avoir au maximum un code qui sépare implémentation et interface, comme pour un langage objet (pour qu'il soit facilement maintenable, réutilisable par module etc.), le tuto que tu m'a indiqué devrait bien m'aider