[Class] Problème avec définitions multiples

**Leo_** · 01/07/2005, 18h13

Bonjour.

J'ai deux classes, CInit (qui contient la fonction InitSDL) et CBMP (qui contient la fonction ShowBMP), ces deux class font appels à screen qui est déini de la manière suivante (dans cinit.cpp) :
SDL_Surface *screen;
Mon problème : comment rendre screen utilisable par les deux fonctions ?

J'ai utilisé un fichier .h intermédiaire, mais j'ai eu un problème de multiple définition de screen (malgrès l'utilisation de #ifndef, etc.).

Merci.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef CINIT_H
#define CINIT_H
 
class CInit
{
	public:
		CInit();
		virtual ~CInit(void);
        void InitSDL(int,int,int,int);
};
 
#endif

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
26
27
28
29
30
31
32
33
34
#include "include.h"
#include "cinit.h"
 
CInit::CInit()
{
}
 
CInit::~CInit()
{
}
 
SDL_Surface *screen;
 
// Fonction d'initialisation de SDL
extern "C" __declspec(dllexport) void InitSDL(int x,int y,int bpp,int fullscreen)
{
    Uint32 flagsinit=SDL_HWSURFACE|SDL_DOUBLEBUF;
	if (fullscreen)
		flagsinit|=SDL_FULLSCREEN;
	SDL_Init(SDL_INIT_VIDEO|SDL_INIT_TIMER);
    if(SDL_Init(SDL_INIT_VIDEO) == -1)
    {
        printf("Impossible d'initialiser SDL :  %s\n", SDL_GetError());
        exit(1);
    }
    atexit(SDL_Quit);
	screen=SDL_SetVideoMode(x,y,bpp,flagsinit);  
    if(screen == NULL)
    {
        printf("Imposible de régler les paramètres vidéos : %s\n", SDL_GetError());
        exit(1);
    }   
	SDL_ShowCursor(0);
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef CBMP_H
#define CBMP_H
 
class CBMP
{
	public:
		CBMP();
		virtual ~CBMP(void);
		void ShowBMP(const char*);
};
 
#endif

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
#include "include.h"
#include "cbmp.h"
 
CBMP::CBMP()
{
}
 
CBMP::~CBMP()
{
}
 
SDL_Surface *image;
 
extern "C" __declspec(dllexport) void ShowBMP(const char* NomImage)
{
    image = SDL_LoadBMP(NomImage);
    if(image == NULL)
    {
        printf("Impossible de charger l'image : %s\n", SDL_GetError());
        exit(1);
    }
    SDL_BlitSurface(image, NULL, screen, NULL);
    SDL_FreeSurface(image);
    SDL_UpdateRect(screen, 0, 0, 0, 0);
}

PS: j'utilise Dev-c++ 4.9.9.0

**Selenite** · 01/07/2005, 18h50

Tu te demande donc comment plusieurs classes peuvent accèder à une variable globale.

Dans un .cpp, tu définit la variable:
T global;

Dans un .h, tu la déclare ainsi:
extern T global;

**Leo_** · 01/07/2005, 19h25

Merci pour l'aide

Mais je viens de laisser tomber les variables globales, d'après ce que j'ai lu, c'est pas terrible :s

Par contre j'ai un autre petit problème que je ne comprend vraiment pas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef CINIT_H
#define CINIT_H
 
class CInit
{
	public:
		CInit();
		virtual ~CInit(void);
		static SDL_Surface *screen;
        void InitSDL(int,int,int,int);
};
 
#endif

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
26
27
28
29
30
31
32
#include "include.h"
#include "cinit.h"
 
CInit::CInit()
{
}
 
CInit::~CInit()
{
}
 
// Fonction d'initialisation de SDL
extern "C" __declspec(dllexport) void CInit::InitSDL(int x,int y,int bpp,int fullscreen)
{
    Uint32 flagsinit=SDL_HWSURFACE|SDL_DOUBLEBUF;
	if (fullscreen)
		flagsinit|=SDL_FULLSCREEN;
	SDL_Init(SDL_INIT_VIDEO|SDL_INIT_TIMER);
    if(SDL_Init(SDL_INIT_VIDEO) == -1)
    {
        printf("Impossible d'initialiser SDL :  %s\n", SDL_GetError());
        exit(1);
    }
    atexit(SDL_Quit);
	CInit::screen=SDL_SetVideoMode(x,y,bpp,flagsinit);  
    if(CInit::screen == NULL)
    {
        printf("Imposible de régler les paramètres vidéos : %s\n", SDL_GetError());
        exit(1);
    }   
	SDL_ShowCursor(0);
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef CBMP_H
#define CBMP_H
 
class CBMP : public CInit
{
	public:
		CBMP();
		virtual ~CBMP(void);
		void ShowBMP(const char*);
};
 
#endif

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
26
#include "include.h"
#include "cinit.h"
#include "cbmp.h"
 
CBMP::CBMP()
{
}
 
CBMP::~CBMP()
{
}
 
SDL_Surface *image;
 
extern "C" __declspec(dllexport) void CBMP::ShowBMP(const char* NomImage)
{
    image = SDL_LoadBMP(NomImage);
    if(image == NULL)
    {
        printf("Impossible de charger l'image : %s\n", SDL_GetError());
        exit(1);
    }
    SDL_BlitSurface(image, NULL, CInit::screen, NULL);
    SDL_FreeSurface(image);
    SDL_UpdateRect(CInit::screen, 0, 0, 0, 0);
}

A présent, ma classe CBMP hérite de CInit, screen est bien un membre public.
Pourtant, voici les erreurs que j'ai à la compilation :

Intermediaire/cinit.o(.text+0x197):cinit.cpp: undefined reference to `CInit::screen'
Intermediaire/cinit.o(.text+0x19d):cinit.cpp: undefined reference to `CInit::screen'
Intermediaire/cbmp.o(.text+0x165):cbmp.cpp: undefined reference to `CInit::screen'

Intermediaire/cbmp.o(.text+0x1b0):cbmp.cpp: undefined reference to `CInit::screen'

Je ne comprend vraiment pas d'où viennent ces erreurs :s

**michhh** · 01/07/2005, 20h04

si je dis pas de betises, ta surface, qui est en static, doit etre definie, or tu defini une surface "image" mais pas screen, il faudrait mettre :

EDIT:

SDL_Surface* CInit::screen = SDL_SetVideoMode(...);

**Leo_** · 01/07/2005, 20h16

cinit.cpp: In member function `void CInit::InitSDL(int, int, int, int)':
cinit.cpp:25: error: redeclaration of `SDL_Surface*CInit::screen'
cinit.h:9: error: `SDL_Surface*CInit::screen' previously declared here

screen est bien définie dans cinit.h:

Code :

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

1
2
3
4
5
6
7
8
9
class CInit
{
	public:
		CInit();
		virtual ~CInit(void);
		static SDL_Surface *screen;
	private:
        void InitSDL(int,int,int,int);
};

**michhh** · 01/07/2005, 20h24

non, screen est declaré mais pas defini, il faut definir les variables statiques comme si tu declarait une fonction... c'est logique d'ailleurs, parce que, imagine un compteur d'instances d'une classes, il faut bien dire qu'il est égal à 0 quelque part... et comme tu ne peux pas le dire dans un constucteur, puisque c'est du static, et que tu ne peux pas affecter une valeur à une variable dans la declaration de la classe (à part les constantes), il faut bien le dire quelque part

**Leo_** · 01/07/2005, 20h26

Ha ok :s

Mais dans ce cas, le compilateur ne devrait pas me dire :

Code :

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

1
2
cinit.cpp:25: error: redeclaration of `SDL_Surface*CInit::screen'
cinit.h:9: error: `SDL_Surface*CInit::screen' previously declared here

**michhh** · 01/07/2005, 20h28

en effet... heu, me serais-je trompé? attend j'essaie

**Leo_** · 01/07/2005, 20h36

Oké merci bien.

(Le projet à besoin de SDL : http://www.libsdl.org)

**michhh** · 01/07/2005, 20h37

euh, j'ai pas d'erreurs... j'ai simplifié grangement la chose mais j'ai pas d'erreurs... je te donne ce que j'ai :

head.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
#ifndef __HEAD__
#define __HEAD__
 
#include <windows.h>
#include <SDL.h>
#pragma comment(lib, "SDL.lib")
#pragma comment(lib, "SDLmain.lib")
 
class CInit {
public: 
	CInit(); 
	virtual ~CInit(void); 
	static SDL_Surface *screen; 
private: 
	void InitSDL(int,int,int,int);
};
 
#endif

definitions.cpp :

Code :

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

1
2
3
#include "head.h"
 
SDL_Surface* CInit::screen = SDL_SetVideoMode (800, 600, 32, SDL_HWSURFACE | SDL_DOUBLEBUF | SDL_FULLSCREEN);

main.cpp :

Code :

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

1
2
3
4
5
6
7
#include "head.h"
 
 
 
int main (int argc, char **argv) {
	return 0;
};

**michhh** · 01/07/2005, 20h40

le truc, c'est que je comprend pas trop tes extern "C" et donc j'ai pas trop envie de me risquer à tester la suite, mais une chose est sure, l'erreur ne vient pas de screen

**Leo_** · 01/07/2005, 20h41

Arg, c'est vraiment zarb :s

Tu peux essayer sans simplifier ?

Edit : Pour les extern "C" __declspec(dllexport) mafonction

C'est parceque le projet généré est une dll

Edit 2 : Vla les sources complétes http://www.megaupload.com/?d=23UZKY3F

**michhh** · 01/07/2005, 20h48

bon je teste mais sans les extern, j'espere que ca changera rien

**michhh** · 01/07/2005, 20h56

CInit.cpp :

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
#include "CInit.h"
 
SDL_Surface* CInit::screen = SDL_SetVideoMode (800, 600, 32, SDL_HWSURFACE | SDL_DOUBLEBUF | SDL_FULLSCREEN);
 
CInit::CInit() {};
CInit::~CInit() {};
 
void CInit::InitSDL(int x, int y, int z, int full) {
	SDL_Init(SDL_INIT_VIDEO|SDL_INIT_TIMER); 
    if(SDL_Init(SDL_INIT_VIDEO) == -1) { 
        printf("Impossible d'initialiser SDL :  %s\n", SDL_GetError()); 
        exit(1);
    }
 
	atexit(SDL_Quit);
	SDL_FreeSurface(this->screen); /*bah oui, on l'a declaree + haut, ca serait peut etre optimisable mais bon pour gagner qqs millisecondes au demmarage...*/
 
	Uint32 flagsinit = (full) ? SDL_HWSURFACE | SDL_DOUBLEBUF | SDL_FULLSCREEN : SDL_HWSURFACE | SDL_DOUBLEBUF;
	CInit::screen=SDL_SetVideoMode(x, y, z, flagsinit);
 
	if(CInit::screen == NULL) { 
        printf("Imposible de régler les paramètres vidéos : %s\n", SDL_GetError()); 
        exit(1); 
    }
}

Voila j'ai un peu modifié à ma sauce, en tout cas ca passe, essaie et si ca marche tjs pas on verra avec l'autre classe

EDIT : ca n'a aucun rapport, mais faudrait penser à mettre un SDL_FreeSurface dans le destructeur
EDIT2 : enfin non faudrait le mettre a la fin de main en fait
EDIT3 : j'arrive pas a compiler ton programme avec dev (faut dire que d'hab j'utilise VC++) il me dit "Could not create executable output directory : "Release". Please check your settings." je sais pas trop ce qu'il faut faire la...

**Leo_** · 01/07/2005, 21h20

Bon, ça me vire le "redeclaration of `SDL_Surface*CInit::screen'" mais j'ai toujours les erreurs du type " [Linker error] undefined reference to `CInit::screen'".

Pour l'erreur que tu as avec dev, vas dasn Projet, Options du projet, onglet Construction, et changes les répertoires

**michhh** · 01/07/2005, 21h27

ok, bon je regarde actuellement ta classe CBMP, et je me demande a quoi sert la surface image exactement?

**Leo_** · 01/07/2005, 21h43

La surface image sert simplement à stocker l'image.

Je viens d'essayer de compiler avec VC, et ça marche :s (mis à part les quelques erreurs du aux fonctions de création de dll spécifiques à dev-c++). Franchement, je ne vois vraiment pas ce qui coince avec ce projet :s

**michhh** · 01/07/2005, 21h53

donc en fait le probleme viendrait de dev?

**Leo_** · 01/07/2005, 22h40

Faut croire :s

C'est bizarre quand même :s

**Leo_** · 02/07/2005, 00h15

Hum, j'ai rien dit en fait, problème sous VC6 :

Code :

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

Init.obj : error LNK2001: unresolved external symbol "public: static struct SDL_Surface * CInit::screen" (?screen@CInit@@2PAUSDL_Surface@@A)

[Class] Problème avec définitions multiples

C++

Vue hybride

Discussions similaires

Partager

Partager