Discussion :

[freedom144]

Bonjour,

Comment déclarer une variable globale utilisable dans plusieurs classes ?

j'ai un projet d'IHM ou j'ai besoin de variable globale (je sais c'est pas très beau)..

voici se que j'ai fait mais qui ne marche pas hihih!


main.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
#include <cstdlib>
#include <iostream>
#include "variable.h"
#include "AfficheExterne.h"
 
using namespace std;
 
int main()
{
 
  int variableG=100;
 
  CAffExterne Call;
  Call.View();
 
   system("PAUSE");
   return EXIT_SUCCESS;
}

variable.h
-------------------------

Code :

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

1
2
3
4
5
6
7
#if !defined(VARIABLE_H)
#define VARIABLE_H
#pragma once
 
extern int variableG=0;
 
#endif

AfficheExterne.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
#if !defined(CAFFEXTERNE_H)
#define CAFFEXTERNE_H
#pragma once
 
#include <iostream> 
#include "Variable.h"
 
 
class CAffExterne
{
 
public:
 void View();
 
};
#endif

AfficheExterne.cpp
---------------------

Code :

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

1
2
3
4
5
6
#include "AfficheExterne.h"
 
void CAffExterne::View()
{
   std::cout<<"Variable Globale : " <<variableG <<std::endl;
}

Enfin c'est une version simplifié.

Le probleme DevCpp me marque plusieurs erreurs :
-Variable.h : [Warning] `variableG' initialized and declared `extern'
-Variable.h : multiple definition of `variableG'
first defined here
- [Linker error] undefined reference to `variableG'

Auriez vous des idées ? par avance merci.

[fregolo52]

Je ne suis pas expert des extern mais les messages me semblent explicites !!!

Et si tu n'initialisais pas ta variable externe ? :

Code :

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

1
2
3
4
5
6
7
#if !defined(VARIABLE_H)
#define VARIABLE_H
#pragma once
 
extern int variableG;
 
#endif

[3DArchi]

Salut,

Comment déclarer une variable globale utilisable dans plusieurs classes ?

en retravaillant ton design car c'est toi qui le dit :

j'ai besoin de variable globale (je sais c'est pas très beau)..

Mais surtout les variables globales ça couple fortement des modules qui ne devaient pas l'être, ça rend impure une fonction (c'est bien que c'est sale ), ça se marie mal avec du mutlithread, ça peut casser la réentrance d'une fonction, c'est des sources à bug, ça a de durée de vie disproportionnée, on sait (parfois) où ça commence mais on sait pas où s'est détruit etc...

Menfin, si tu veux quand même te tirer une balle dans le pied et pouvoir tirer plusieurs fois avec, voilà une possibilité :

Code maVariable.h :

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

TYPE& GetMaVariableQuiRendMonCodePasBeau();

Code maVariable.cpp :

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

1
2
3
4
5
6
TYPE& GetMaVariableQuiRendMonCodePasBeau()
{
   static TYPE maVariable;
 
   return maVariable
}

Code AfficheExterne.cpp :

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

1
2
3
4
5
6
7
8
 
#include "AfficheExterne.h"
#include "maVariable.h"
 
void CAffExterne::View() const
{
   std::cout<<"Variable Globale : "<<GetMaVariableQuiRendMonCodePasBeau()<<std::endl;
}

Code main.cpp :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include "variable.h"
#include "AfficheExterne.h"
 
int main()
{
 
  GetMaVariableQuiRendMonCodePasBeau()=100;
 
  CAffExterne Call;
  Call.View();
 
   return 0;
}

Et s'il te plait : tu me vire fissa les lignes #include <iostream> et #include "Variable.h" de AfficheExterne.h pour ne les mettre que dans le .cpp
On n'inclue dans un fichier que strictement ce dont il a besoin pour compiler. Et c'est AfficheExterne.cpp qui a besoin de iostream et variable.h pas AfficheExterne.h

using namespace std; ??? Ton clavier s'use si vite que ça ?

Et on n'oublie pas un petit tour par la F.A.Q. :
A quoi sert "using namespace std;" ?
Organisation du code source
Comment faire une pause (attendre que l'utilisateur tape une touche) ?

[tut]

...et le design pattern singleton, pour au moins encapsuler cela ?

[3DArchi]

...et le design pattern singleton, pour au moins encapsuler cela ?

Le pattern singleton ? Pour se donner un (fausse) bonne conscience objet
Etes-vous atteint de Singletonite ?