[C/C++]: Header che si includono a vicenda

[cadue]

salve! io ho 2 header con ognuno una classe
così:

//file game.h
#include "gun.h"

class game{
public:
Gun* gun;
}

//file gun.h
#include "game.h"

class gun{
public:
gun(Game* game);
}


se provo a compilare questo codice il programma va in loop perchè ovviamente avviene un ciclo chiuso, un header include un header che a sua volta re-include l'header di partenza etc...

ma la classe "game" ha bisogno della classe "gun" per avere un puntatore a questa classe tra i membri, mentre la classe "gun" ha bisogno della classe "game" per il costruttore.

come faccio?

[andbin]

Nei file di include è consuetudine mettere un qualcosa del tipo:

#ifndef _NOMEHEADER_H
#define _NOMEHEADER_H

   tutto il resto (include, definizioni, ecc...)

#endif

Dove NOMEHEADER ovviamente è il nome del tuo header. Il motivo è semplice: evitare che le definizioni vengano incluse più volte (cosa che oltretutto causerebbe errori!).

[cadue]

uffa...non va. ma questo serve per fare in modo che l'header venga incluso una sola volta in ogni file dove è incluso?
Nel mio progetto ci sono questi header che oltre a includersi a vicenda vengono anche inclusi in due .cpp differenti. Praticamente con il codice che mi hai dato i file si includono nei cpp ma non si includono a vicenda

[MItaly]

Non vorrei dire una boiata, ma forse inserendo un #pragma once (se supportato dal compilatore) in ogni file di intestazione si potrebbe risolvere il problema.

[XWolverineX]

Originariamente inviato da MItaly
Non vorrei dire una boiata, ma forse inserendo un #pragma once (se supportato dal compilatore) in ogni file di intestazione si potrebbe risolvere il problema.

Non è una boiata.

[cadue]

dando un occhiata su altri header ho notato che risolvono questo problema come dite voi...ma a me non funziona ancora. gli header si includono ai cpp ma non a vicenda. Vi posto il codice

//file game.h
#ifndef _GAME_H_
#define _GAME_H_

#include "Gun.h"
#include <irrlicht.h>
using namespace irr;
using namespace core;
using namespace video;
using namespace scene;
using namespace gui;

class Game : public IEventReceiver
{
public:

	Game(bool fullscreen, bool music, bool shadows, bool additive, bool vsync, video::E_DRIVER_TYPE driver);

	~Game();

	void run();

	virtual bool OnEvent(SEvent event);

//funzioni
	void loadSceneData();
        void prepare();
        void setCamera();


	bool fullscreen;
	bool music;
	bool shadows;
	bool additive;
	bool vsync;

	ICameraSceneNode* camera;
	E_DRIVER_TYPE driverType;

//solite robe
    IrrlichtDevice *device;
    IVideoDriver* driver;
    ISceneManager* smgr;
    IGUIEnvironment* guienv;

//objects

    Gun* gun;//<------------------------------ERRORE!!!
};

#endif

//file gun.h

#ifndef _GUN_H_
#define _GUN_H_

#include <irrlicht.h>
#include "game.h"


using namespace irr;
using namespace core;
using namespace video;
using namespace scene;
using namespace gui;

class Gun
{
    private:
        //solite robe
        IrrlichtDevice *device;
        IVideoDriver* driver;
        ISceneManager* smgr;
        IGUIEnvironment* guienv;

        //camera
        ICameraSceneNode* camera;

        //Gun node & mesh
        IAnimatedMesh* gun_mesh;
        IAnimatedMeshSceneNode* gun_node;
    
    public:
    Gun(Game* game);//<-------------------ERRORE!!!
    ~Gun();

};

#endif

[XWolverineX]

E' normale che ti dia quegli errori poichè hai sbagliato.
Non puoi mettere in un file .h questo

Gun(Game* game);//<-------------------ERRORE!!!

]
Poichè tu dichiari e definisci la classe game proprio nel file .cpp!
Come può quindi sapere il file .h cosa sia game?
Per risolvere il problema devi fare la dichiarazione anticipata. Praticamente nel file gun.h dopo i soliti ifndef e define, aggiugi

class Game; //dichiarazione anticipata

]
In questo modo dovrebbe compilare bene

[cadue]

WoW! non lo sapevo!

[cadue]

Grazie mille XWolverineX