Variables globales statiques accessibles par plusieurs fichiers

Bonjour,

Je programme un jeu d'échec qui devient de plus en plus volumineux, et je me suis dit qu'au lieu de passer un objet plateau par référence à 15 fonctions différentes dans quasiment autant de fichiers, il serait plus intéressant de le mettre dans une classe (un namespace? un fichier? bref je sais pas trop) spéciale "Global.h" qui contiendrait les variables utilisées par les fonctions éparpillées dans les fichiers.

Jusque là pas de problème, j'ai réussi ^^ ..Mais chaque fonction qui utilise les variables globales s'en fait ses propres copies réinitialisées à zéro, ce qui n'est pas du tout le comportement souhaité :-/

Finalement je suis tombé sur une solution qui règle ce dernier problème:

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 26 27 28

#ifndef _Global #define _Global   /* variables globales utiles en de nombreux endroits */   #include "Coup.h" #include "Plateau.h" #include <vector>   class Global {   public:   static Plateau plateau;   static short posRoi; /* la position du roi, dependamment de la couleur */   static std::vector<Coup> setCoups; /* l'ensemble des coups possibles */   };   Plateau Global::plateau=Plateau();   short Global::posRoi=plateau.roiB;   #endif

---

Mais ainsi les variables semblent n'être utilisables que dans les headers: j'ai créé une classe bidon avec .h et .cpp (en plus du main.cpp, en supprimant tous les autres fichiers du projet) en faisant un #include "Global.h", et le compilateur me dit qu'il y a une définition multiple des variables plateau..etc lorsqu'elles sont utilisées dans le .cpp, alors qu'elles peuvent être utilisées dans le .h sans problèmes. J'utilise aussi ces variables dans main.cpp et bizarrement ça fonctionne.

Que puis-je faire pour accéder et modifier ces variables dans un fichier .cpp?

Merci d'avance,
Benjamin.

Salut,

essaye de rajouter un #ifndef dans ton fichier .h afin qu'il ne soit pas inclu plusieurs fois.

c'est déjà fait :-/ (cf mon code plus haut)

L'erreur doit être plus subtile, hélas

Euuhh c'est dans fichier cpp ça

Code:

---

short Global::posRoi=plateau.roiB;

---

?
Tu redéclares la variable là non.

hmm non c'est toujours le fichier .h, avant le #endif d'ailleurs.

J'ai utilisé exactement la même structure que dans le livre "thinking in C++" dans l'exemple d'utilisation de variables de classe statiques, où ces dernières sont initialisées ainsi en dehors de la classe.

Pourquoi ça marche chez lui et pas chez moi, je n'en sais rien.

Tu as peut-être raison, mais alors que faire? Je ne vais pas écrire deux fois #ifndef au cas où le compilo, mal réveillé après une nuit difficile, ne verrait pas le premier ^^

Citation:

---

Envoyé par Chewbi

Mais chaque fonction qui utilise les variables globales s'en fait ses propres copies réinitialisées à zéro, ce qui n'est pas du tout le comportement souhaité :-/

---

faudrait un exemple de fonctions dans un meme programme qui reinitialise les variables...

Citation:

---

Envoyé par Chewbi

Mais ainsi les variables semblent n'être utilisables que dans les headers: j'ai créé une classe bidon avec .h et .cpp (en plus du main.cpp, en supprimant tous les autres fichiers du projet) en faisant un #include "Global.h", et le compilateur me dit qu'il y a une définition multiple des variables plateau..etc lorsqu'elles sont utilisées dans le .cpp, alors qu'elles peuvent être utilisées dans le .h sans problèmes. J'utilise aussi ces variables dans main.cpp et bizarrement ça fonctionne.

Que puis-je faire pour accéder et modifier ces variables dans un fichier .cpp?

Merci d'avance,
Benjamin.

---

comment ca accessible seulement depuis le header, ca se compile pas?
poste un exemple d'erreur de compilation

Bon, j'essayerai d'être plus clair tout à l'heure (je dois partir là).

le compilo dit "variables déjà déclarées", mais je te donnerai le message précis si tu veux.

Des exemples de fonctions aussi, mais bon.. je ne pense pas que ça aidera beaucoup (tu peux créer un fichier toto.cpp et toto.h, puis tester dans une fonction bidon qui affiche les variables, tu risques d'avoir le même problème que moi :-/)

L'initialisation des variables statiques non intégrales doit se faire dans le fichier .cpp.
Dans ton cas ta variable intégrale dépend d'une autre, donc les deux doivent être dans le .cpp

Code:

---

1 2 3

Plateau Global::plateau; short Global::posRoi = Global::plateau.roiB; std::vector<Coup> Global::setCoups;

---

Attention, on dit souvent qu'on ne connait pas l'ordre d'initialisations des variables statiques.
Je ne suis donc pas certain que cela fonctionne, étant donné que Global::posRoi dépend de Global::plateau.

Je doute néanmoins de la pertinence de ton design, moi j'aurais plutôt fait une instance globale de Global.
Et puis je vois pas pourquoi la position du roi est dans Global.

Effectivement dans le "thinking in C++" c'est ce qui est écrit.

Et sinon j'ai essayé avec un projet vierge et chez moi un cas simple avec uniquement un short ca marche bien :

Header1.h

Code:

---

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

#ifndef _Global #define _Global   /* variables globales utiles en de nombreux endroits */   class Global {   public: static short posRoi; /* la position du roi, dependamment de la couleur */   };   short Global::posRoi=2;   #endif

---

test.cpp

Code:

---

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

#include "stdafx.h" #include "test.h" #include "Header1.h" #include <iostream>   using namespace std;   test::test() { Global::posRoi=5; }   void test::affiche() { cout << Global::posRoi; }

---

test.h

Code:

---

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

#ifndef _test #define _test   class test {   public: test(); void affiche(); };   #endif

---

Et enfin le main:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include "stdafx.h" #include "test.h"   int _tmain(int argc, _TCHAR* argv[]) { test t; t.affiche(); // 5 getchar();   return 0; }

---

Donc faudrait peut être que tu nous montres un peu de code pour essayer de comprendre ce qui se passe chez toi.

@ loufoque :
la position du roi est utile pour tester le statut du roi, et ça m'évite ainsi de réallouer une case mémoire à chacun des 35623468 appels effectués ;-) J'envisage de stocker les positions des autres pièces aussi pour gagner une boucle.

Sinon, pour l'ordre des variables et la définition dans le .cpp que tu mentionnes, le "thinking in C++" (encore lui ;o)) présente une classe similaire à Global avec deux variables x et y, y dépendant de x; peut-être cela dépend-il du compilateur alors?

@neo :
OK je regarde ton exemple et je vais essayer de calquer sur le mien pour voir ce qui cloche. Merci !

@ neo toujours :

Ton programme fonctionne car tu n'inclue le .h qu'à un seul endroit et n'utilise les variables globales que dans un fichier (je viens de vérifier, malgré le #include "test.h", Global::posRoi n'est pas accessible depuis le main). Mon but étant de les utiliser dans plusieurs fichiers, j'ai essayé de faire un #include "Global.h" dans le main afin d'y utiliser posRoi, voici le résultat:

test.o:(.data+0x0): définitions multiples de « Global::posRoi »
main.o:(.data+0x0): défini pour la première fois ici

Inutile donc de poster un bout de mon code, car.. c'est exactement mon problème ^^ Ceci dit j'ai le sentiment que la solution n'est pas bien compliquée, mais je la vois pas :-/ heeeeeeelp !

La solution, je te l'ai donnée.
J'ai testé et cela fonctionne parfaitement.

@ loufoque :

oui et non.
Je n'ai pas testé l'initialisation dans un fichier Global.cpp, et ça fonctionnerait peut-être (j'en doute mais j'essayerai). En revanche en utilisant uniquement posRoi (short, donc type intégral), tu peux lire le problème ci-dessus :-/ ..et celui-ci tu ne l'as pas résolu ;-)

ça marche pas, loufoque :

main.o : Dans la fonction "main":main.cpp:(.text+0x1f): référence indéfinie vers « Global::posRoi »
test.o : Dans la fonction "test::test()":test.cpp:(.text+0x6): référence indéfinie vers « Global::posRoi »
test.o : Dans la fonction "test::test()":test.cpp:(.text+0x14): référence indéfinie vers « Global::posRoi »
test.o : Dans la fonction "test::affiche()":test.cpp:(.text+0x25): référence indéfinie vers « Global::posRoi »
collect2: ld a retourné 1 code d'état d'exécution
make: *** [main] Erreur 1

voici mon code copié de celui de neo et s'inspirant de ce que tu m'as dit :

Code:

---

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

/* Global.h */   #ifndef _Global #define _Global   class Global {   public:   static short posRoi;   };   #endif

---

Code:

---

1 2 3 4 5 6 7

/* Global.cpp */   #include "Global.h"   short Global::posRoi=3;

---

Code:

---

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

/* test.h */   #ifndef _test #define _test   class test {   public:   test(); void affiche();   };   #endif

---

Code:

---

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

/* test.cpp */   #include "test.h" #include "Global.h" #include <iostream>   using namespace std;   test::test() { Global::posRoi=5; }   void test::affiche() { cout << Global::posRoi << endl; }

---

Code:

---

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

/* main.cpp */   #include "Global.h" #include "test.h" #include <iostream>   using namespace std;   int main(int argc, char* argv[]) { Global::posRoi=2; test t; t.affiche();   return 0; }

---

Code:

---

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

/* Makefile */   CXX=g++ CXXFLAGS=-Wall LDFLAGS=-Wall EXEC=main   all: $(EXEC)   main: main.o test.o Global.o $(CXX) -o $@ $^ $(LDFLAGS)   %.o: %.cpp $(CXX) -o $@ $^ $(CXXFLAGS)   clean: rm -f *.o rm -f $(EXEC)

---

peux-tu me donner ton code ?

avec quoi tu compiles?

chez moi, ton code marche parfaitement


EDIT: j'avais pas vu le make file, je regarde d'abord :)


OK ton problème vient du makefile, tu compiles chaque programme tout seul.
tu dois changer le Makefile en:

Code:

---

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

CXX=g++ CXXFLAGS=-Wall LDFLAGS=-Wall EXEC=main   all: $(EXEC)   main: main.o test.o Global.o $(CXX) -o $@ $^ $(LDFLAGS)   %.o: %.cpp $(CXX) -c $^ $(CXXFLAGS)   clean: rm -f *.o rm -f $(EXEC)

---

Citation:

---

peux-tu me donner ton code ?

---

Je n'ai bien sûr pas perdu mon temps à faire des fichiers multiples et un makefile.

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 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47

#ifndef _Global #define _Global   /* variables globales utiles en de nombreux endroits */   struct Coup { };   struct Plateau { short roiB; Plateau() : roiB(5) { } };   #include <vector>   class Global {   public:   static Plateau plateau;   static short posRoi; /* la position du roi, dependamment de la couleur */   static std::vector<Coup> setCoups; /* l'ensemble des coups possibles */   };   #endif   #include <vector>   Plateau Global::plateau; short Global::posRoi = Global::plateau.roiB; std::vector<Coup> setCoups;   #include <iostream>   int main() { std::cout << Global::posRoi << std::endl; }

---

le problème vient du makefile, j'ai édité mon post d'avant

Y'a pas des règles implicites pour les makefiles normalement ?

Citation:

---

Envoyé par loufoque

L'initialisation des variables statiques non intégrales doit se faire dans le fichier .cpp.

---

En fait, dans tous les cas, la variable statique doit être définie en dehors de la classe, et dans un Cpp pour éviter les doublons, comme une définition classique. La seule différence avec les types entiers constants, c'est que la variable peut être initialisée dans la classe, mais elle doit quand même être définie en dehors.

Citation:

---

Envoyé par loufoque

Attention, on dit souvent qu'on ne connait pas l'ordre d'initialisations des variables statiques.
Je ne suis donc pas certain que cela fonctionne, étant donné que Global::posRoi dépend de Global::plateau.

---

On ne connais pas cet ordre quand les variables sont dans des unités de compilation différentes. A l'intérieur d'une même unité de compilation, l'ordre est connu, c'est celui des définitions, quand les objets ne sont pas initialisés avec des expressions constantes (ce qui est le cas ici). Donc, ici, pas de problèmes.

EDIT: oops je répondais à coca25, pas vu la page 2. Je regarde !

EDIT2: si tu passes par là coca25, je crois que tu t'es trompé de Makefile, c'est le mien que tu as donné ici. Si le problème vient de là et que tu peux me donner le bon Makefile ce serait génial :-)

J'ai copié collé ton Makefile, ça marche pas mieux (même erreur) :-(

Je commence à désespérer un peu :-/ Tu es sûr que tu as exactement le dernier code que j'ai écrit?

Merci d'essayer :-)

@ loufoque :

c'est bien ce que je craignais, tu n'as utilisé Global.h qu'en un seul endroit, et donc ça marche.

au-delà de deux fichiers .cpp utilisant les variables déclarées dans Global, ça coince (cf messages d'erreur plus haut).
Et c'est là tout le problème :-/

Personne sait ce qu'il faut faire?

non non, c'est pas le même makefile.
dans le tien, tu compile et tu lie -o les fichier un par un

je l'ai modifie pour que ca compile juste:

Code:

---

1 2 3

%.o: %.cpp $(CXX) -c $^ $(CXXFLAGS)

---

et ensuite ca lie les 3 .o entre eux

le make doit faire ceci

Code:

---

1 2 3

g++ -c main.cpp test.cpp Global.cpp g++ -o main main.o test.o Global.o

---

Yes ça marche !!!!!!!!!!!!
Merci bcp :)
En fait j'utilisais toujours le même Makefile depuis +1 an, sans trop savoir comment il fonctionne, les ^ $ @ me faisant un peu peur..

Chewbi, heureux, qui va enfin pouvoir optimiser son programme efficacement sans avoir des fichiers de 7000 lignes :p