Variables globales statiques accessibles par plusieurs fichiers

**Chewbi** · 23/11/2006, 16h02

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 :

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

**Neo41** · 23/11/2006, 16h15

Salut,

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

**Chewbi** · 23/11/2006, 16h17

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

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

**Neo41** · 23/11/2006, 16h33

Euuhh c'est dans fichier cpp ça

Code :

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

short Global::posRoi=plateau.roiB;

?
Tu redéclares la variable là non.

**Chewbi** · 23/11/2006, 16h44

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

**coca25** · 23/11/2006, 17h17

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

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

**Chewbi** · 23/11/2006, 17h27

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

**loufoque** · 23/11/2006, 17h50

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 :

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

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.

**Neo41** · 23/11/2006, 18h22

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 :

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

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

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

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 :

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

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.

**Chewbi** · 23/11/2006, 18h33

@ 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 !

**Chewbi** · 23/11/2006, 18h47

@ 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 !

**loufoque** · 23/11/2006, 19h27

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

**Chewbi** · 23/11/2006, 20h53

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

**Chewbi** · 23/11/2006, 21h06

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

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
 
 
/* Global.h */
 
#ifndef _Global
#define _Global
 
class Global {
 
public:
 
   static short posRoi;
 
};
 
#endif

Code :

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

1
2
3
4
5
6
7
 
 
/* Global.cpp */
 
#include "Global.h"
 
short Global::posRoi=3;

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
 
 
/* test.h */
 
#ifndef _test
#define _test
 
class test {
 
public:
 
   test();
   void affiche();
 
};
 
#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
 
 
/* 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 :

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

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

**coca25** · 23/11/2006, 21h21

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 :

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

**loufoque** · 23/11/2006, 21h34

peux-tu me donner ton code ?

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

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
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;
}

**coca25** · 23/11/2006, 21h36

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

**loufoque** · 23/11/2006, 21h38

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

**JolyLoic** · 23/11/2006, 21h41

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.

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.

**Chewbi** · 23/11/2006, 21h48

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

Variables globales statiques accessibles par plusieurs fichiers

C++

Discussions similaires

Partager

Partager