Discussion :

[Chewbi]

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]

Salut,

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

[Chewbi]

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

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

[Neo41]

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]

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]

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

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]

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]

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]

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]

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

[JolyLoic]

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.

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.