Problème de variables globales

Bonjour à tous,

voilà j'ai un petit problème de définition avec une variable globale; voici la configuration de mon projet :

un fichier GraphicalTemplates.h (avec prototypes de fcts et la déclaration d'une variable globale floor1)
un fichier graphicalTemplates.cpp (qui va entre autres donner une valeur à floor1)

un fichier Object3D.h
un fichier Object3D.cpp (qui va utiliser floor1)


J'importe bien dans Object3D.cpp son header ainsi que GraphicalTemplates.h mais à la compilation ça plante il me met :

multiples definitoons of 'floor1'
first defined here


Et pourtant c'est bien le seul endroit où j'utilise floor1...

Salut mets dans un seul .c

Code:

---

1 2	type floor1;

---

et dans le .h

Code:

---

1 2 3

extern type floor1;

---

Peux tu nous fournir ton code ? Sans savoir comment tu fais les choses, c'est impossible de t'aider correctement.
Ensuite les déclaration 'extern' , ca fait plutôt "C" (et non C++).

Dans GraphicalTemplates.h :

Code:

---

1 2 3 4 5 6 7 8 9 10

#include<...>   GLuint floor1;   ...   void loadTextures();   ...

---

GraphicalTemplates.cpp :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#include "GraphicalTemplates.h"   ...   void loadTextures() { //ici je donne une valeur à floor1 }   ...

---

dans Object3D.h

Code:

---

1 2 3 4

... void drawCube(); ...

---

dans Object3D.cpp

Code:

---

1 2 3 4 5 6 7 8 9

#include<Object3D.h>   ... void drawCube() { //ici j'utilise la valeur donnée à floor1 } ...

---

en gros dans ma main je fais à un moment loadTextures() et ensuite drawCube().

Mais voilà même en mettant extern dans Graphicaltemplates.h ça compile pas...

pour ton .h il faut peu être mettre un

Code:

---

1 2 3 4 5 6

#ifndef _GraphicalTemplates_ #define _GraphicalTemplates_   // tout ton code   #endif

---

car là si tu lis plusieurs fois ce fichier .h dans d'autres fichier tu peux avoir ce genre de soucis je pense...

sinon la solution de "alskaar" devrait aussi marcher :)

Tu la déclare dans le cpp et pas de le header

Code:

---

GLuint floor1;

---

et dans le header tu la déclare extern

Code:

---

extern GLuint floor1;

---

Citation:

---

Envoyé par G-OnZa

Peux tu nous fournir ton code ? Sans savoir comment tu fais les choses, c'est impossible de t'aider correctement.
Ensuite les déclaration 'extern' , ca fait plutôt "C" (et non C++).

---

désolé si extern ca fait C mais c'est le mot clef!
par contre tu peux proposer une methode pour eviter les globales ;)

Citation:

---

par contre tu peux proposer une methode pour eviter les globales

---

Si vraiment tu veux une variable accessible par tout le monde mais que tu souhaites éviter de déclarer comme ça, tu peux faire :

Code:

---

1 2 3 4 5

/*Fichier .h*/   GLuint getFloor(); void setFloor(GLuint);

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

/* Fichier cpp*/ #include "lefichierhpp"   static GLuint floor1 = 0;     GLuint getFloor() { return floor1; }   void setFloor(GLuint machin) { floor1 = machin; }

---

Mais bon, ce n'est qu'une astuce (une pseudo-méthode on va dire car ça reste équivalent à avoir une variable globale). Retirer totalement le fait d'avoir une variable globale ne pourra se faire que si je comprend exactement pourquoi tu en as besoin ;)

Citation:

---

Envoyé par Bob.Killer

pour ton .h il faut peu être mettre un

Code:

---

1 2 3 4 5 6

#ifndef _GraphicalTemplates_ #define _GraphicalTemplates_   // tout ton code   #endif

---

car là si tu lis plusieurs fois ce fichier .h dans d'autres fichier tu peux avoir ce genre de soucis je pense...

---

l'utilisation des ifdef ... ne permet pas de resoudre le pb
un include correspond a recopier le code du .h au debut de ton fichier
c'est l'une des tache de la précompilation

le fait de mettre une definition d'une variable ou d'une fonction dans un .h
amenera forcement a un conflit lors du link des N fichiers .o qui correspondent au N fichiers de codes (enfin, l'utilisation de static peut etre une solution mais ne repondra pas a son besoin ).

et pour une variable la seul maniere de differencier une definition d'une declaration est d'utiliser extern