Discussion :

[b4u]

Bonsoir,

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

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

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

[fearyourself]

Sans l'avoir testé :

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

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
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 */

Jc

[b4u]

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

[b4u]

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

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

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

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

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

devienne

Code :

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

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

?

[DaZumba]

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

Code :

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

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

devienne

Code :

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

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.

[b4u]

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 :

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

struct sScene;

n'est utile que pour remplacer un

Code :

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

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

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

[DaZumba]

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 :

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

struct sScene { ... };

d'un cote, tu es oblige de faire un

Code :

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

struct sScene *Scene

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

Code :

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

typedef struct sScene { ... } tScene;

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

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 :

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

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

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

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 :

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

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

[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

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