variable globale string

Version imprimable

25/09/2010, 10h56
parabole33

variable globale string

bonjour ,

G un projet avec 2 cpp et 2 .h .

Je souhaiterai déclarer et initialiser 1 variable string gloable (les 2 cpp ).

G essayé plusieurs truc mais bof....:roll:

si je déclare string str ; erreur dans la déclaration

merci beaucoup par avance :ccool:
25/09/2010, 11h40
Astraya

Bonjour,

As-tu déclaré l'utilisation du namespace standard avant?

Citation:

using namespace std;
// ou
using std::string;

Afin d'éviter les variables globales de ce type ( pas bien! ).
Tu à plusieurs autres approches plus propres et respectant la philosophie OO.

1- Déclarer la string avant de passer la référence au constructeur des deux classes. Attention à la durée de vie de la string

2- Encapsuler la string dans une classe, prévoir les services offert par ta classe et les implémenter. Instancier la classe 1 fois et passer la référence sur au constructeur des deux classes. Attention à la durée de vie de la classe

La deuxième solution est de loin préférable si tu travail en multithread, tu pourras ainsi pouvoir mettre en œuvre des mécanismes de synchronisation correctement dans les services donnés.
25/09/2010, 12h11
parabole33

G bien
using namespace std;
// ou
using std::string;

dans les 2 Cpp .

Pourquoi le type string pose un tel problème ?

si je veux découper un projet en plusieurs cpp pour une meilleure clarté sans faire de nouvelle classe , je le déclare comment ?

En C# on met partial class

merci beaucoup
25/09/2010, 13h16
Astraya

As tu #include <string>
Quel est le message d'erreur?
N'as tu pas une autre variable global nommé str?
25/09/2010, 14h08
parabole33

g cet include sur les 2 cpp .

Si je déclare string str -> erreur dans la déclaration
Si je déclare String str -> OK

Sauf que les membres entre string et String ne sont tout à fait les même .. :?

8O
si je veux découper un projet en plusieurs cpp pour une meilleure clarté sans faire de nouvelle classe , je le déclare comment ?

En C# on met partial class

merci
25/09/2010, 18h38
ptyxs

Pour utiliser une variable définie dans un fichier a dans un fichier b tu dois la faire précéder du mot clé extern dans b.
dans a

Code:

int truc = 0;

dans b

Code:

extern int truc;

Mais, comme on te l'a dit l'utilisation des variables globales est unanimement déconseillée.
26/09/2010, 09h17
3DArchi

Salut,
Peux-tu nous montré le code qui te pose problème ?

P.S.: s/G/j'ai ;)

merci par avance de vôtre aide . C du Borland c++ 2010

Voila les déclarations premier fichier cpp ( programme principal ).

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
#include <vcl.h>
#pragma hdrstop
 
#include "IP2Check_V2.h"
#include "About.h"
#include "SNMP.h"
#include "Telnet.h"
#include "relais.h"
#include <string.h>
#include <string>
#include <iostream.h>
#include <sstream>
#include <algorithm> // remove(), erase()
#include <utilcls.h> //excel !
#include <sstream>
#include <vector>
 
 
using namespace std;
using std::string ;
 
TSDIAppForm *SDIAppForm;
 
SNMP snmp ; // déclaration de la class SNMP
Telnet telnet ; // class Telnet
 
string  Oid_recu;  // déclaration --> OK

Son IP2Check.h

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
#ifndef IP2Check_V2H
#define IP2Check_V2H
//----------------------------------------------------------------------------
#include <ComCtrls.hpp>
#include <ExtCtrls.hpp>
#include <Buttons.hpp>
#include <StdCtrls.hpp>
#include <Dialogs.hpp>
#include <Menus.hpp>
#include <Controls.hpp>
#include <Forms.hpp>
#include <Graphics.hpp>
#include <Classes.hpp>
#include <Windows.hpp>
#include <System.hpp>
#include <ActnList.hpp>
#include <ImgList.hpp>
#include <StdActns.hpp>
#include <ToolWin.hpp>
#include <Grids.hpp>
#include <ValEdit.hpp>
 
 
//----------------------------------------------------------------------------
class TSDIAppForm : public TForm
{
__published:
 
 
//---------------------------------------------------------------------

le 2 iéme fichier cpp relais.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
pragma hdrstop
#include "global.h"
#include "relais.h"
#include "IP2Check_V2.h"
#include "About.h"
#include "SNMP.h"
#include "Telnet.h"
#include <string.h>
#include <string>
#include <iostream.h>
#include <sstream>
#include <algorithm>
//---------------------------------------------------------------------------
using namespace std;
using std::string;
 
#pragma package(smart_init)
 
//------------------------------

relais.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#ifndef relaisH
#define relaisH
 
#include "IP2Check_V2.h"
#include "About.h"
#include "SNMP.h"
#include "Telnet.h"
#include "relais.h"
#include <string.h>
#include <string>
#include <iostream.h>
#include <sstream>
//---------------------------------------------------------------------------
#endif

la variable Oid_recu ( 1 iér cpp ) n'est pas visible à l'intérieur de relais.cpp .
C surement normal = encapsulation ? Pourtant les deux fichiers sont normalement pour la même class !!! C ce que j'essaye de faire :mouarf:

26/09/2010, 15h51
Lucien63

As tu lu ce que t'as dis ptyxs
26/09/2010, 17h31
parabole33

Et oui :cry:

on va faire avec !:(

g au moins mieux compris la mécanique du extern ... :ccool:

merci
28/09/2010, 13h44
dragonjoker59

Si tes 2 fichiers sont 'pour la même classe'. Pourquoi ne déclares-tu pas ton 'Oid_recu' en tant que variable membre de ta classe ?
Sinon, si tu le veux en variable globale, effectivement 'extern' est la meilleure solution.
29/09/2010, 11h20
Astraya

Citation:

effectivement 'extern' est la meilleure solution

La plus simple! pas la meilleur...
29/09/2010, 20h05
parabole33

il y a peut être un peu trop de include ...:cry:

Mais c le début des débutants :roll:
G pas recopier les codes en entier mais grosse inspiration grâce au forum comme Développez.net :ccool::ccool::ccool::ccool:

C pour cela qu'il y a en a un peu trop :mouarf:
Je ferais le ménage quand je cernerai mieux le C++ .

Pour dragonjoker59 :)
Si tes 2 fichiers sont 'pour la même classe'. Pourquoi ne déclares-tu pas ton 'Oid_recu' en tant que variable membre de ta classe ?

Pourrais tu préciser ?

merci à tous
30/09/2010, 08h45
dragonjoker59

Et bien tu dis que tes 2 .cpp sont pour la même classe. Ca laisse supposer que tu as dans un .h la déclaration d'une classe et que pour une raison quelconque tu as séparé son implémentation dans 2 .cpp séparés.
Si c'est bien le cas de figure dans lequel tu te trouves, pourquoi ne mets tu donc pas ton Oid_recu en variable membre de la classe déclarée (TSDIAppForm je pense) dans ton .h ?
06/10/2010, 23h13
parabole33

Citation:

Envoyé par dragonjoker59

Et bien tu dis que tes 2 .cpp sont pour la même classe. Ca laisse supposer que tu as dans un .h la déclaration d'une classe et que pour une raison quelconque tu as séparé son implémentation dans 2 .cpp séparés.
Si c'est bien le cas de figure dans lequel tu te trouves, pourquoi ne mets tu donc pas ton Oid_recu en variable membre de la classe déclarée (TSDIAppForm je pense) dans ton .h ?

C bien ce cas . En C# , on peut pour plus de clarté, découper un code en plusieurs .cs . L'ensemble étant pour la même classe .On met " partial class " à chaque déclaration .

G placé dans le .h la déclaration suivante :

extern TSDIAppForm *SDIAppForm;
//----------------------------------------------------------------------------

char data_recu[20]; // pour les valeu num

string Oid_recu; // <-----
bool blocage_snmp=false ;

#endif

G une erreur : E2141 Erreur de syntaxe de déclaration

merci pour ton aide :ccool:
07/10/2010, 09h37
dragonjoker59
Euh... Je n'aurais pas fait comme ça.
Si ta variable Oid_recu dépend de ta classe TSDIAppForm, alors mets le en membre de cette classe, de la manière suivante :
Code:

1 2 3 4 5 6 class TSDIAppForm : public TForm { private: std::string Oid_recu; };
Sinon (si Oid_recu n'a strictement rien à voir avec cette classe, ni avec aucune autre d'ailleurs), le mieux est que tu laisses ton code comme il était, avec le 'extern'.

Bonjour,
Au risque d'avoir un second avertissement, la classe TSDIAppForm
est crée et déclarée automatiquement par Borland, ainsi
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 class TSDIAppForm : public TForm { __published: TOpenDialog *OpenDialog; TSaveDialog *SaveDialog; TToolBar *ToolBar1; ... ... private: public: virtual __fastcall TSDIAppForm(TComponent *AOwner); }; //---------------------------------------------------------------------------- extern TSDIAppForm *SDIAppForm; // Fin du .h Le .cpp Le constructeur //--------------------------------------------------------------------- __fastcall TSDIAppForm::TSDIAppForm(TComponent *AOwner) : TForm(AOwner) { }
C'est cela qui doit être considéré comme le main pour un utilisateur pas très expérimenté.
Il n'y a aucun inconvénient à avoir plusieurs .cpp pour une même classe, main naturellement les #include headers.h correspondants doivent figurer dans chacun des .cpp

Si, comme je le pense, l'autre .cpp ressemble à cela
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 //--------------------------------------------------------------------- #include <vcl.h> #pragma hdrstop //--------------------------------------------------------------------- USEFORM("sdimain.cpp", SDIAppForm); //--------------------------------------------------------------------------- WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR, int){ Application->Initialize(); Application->HelpFile = "Jeux_enfants"; Application->Title = "Jeux d'enfants"; Application->CreateForm(__classid(TSDIAppForm), &SDIAppForm); Application->Run(); return 0; } //---------------------------------------------------------------------
Alors, c'est le vrai main(), mais avant se savoir exactement ce qu'on fait, il ne vaut mieux pas y toucher.

Ces 3 lignes sont à la fin du .h
Code:

1 2 3 4 char data_recu[20]; // pour les valeu num string Oid_recu; // <----- bool blocage_snmp=false ;
dans votre code.
Vous faites une initialisation dans un .h ce qui est interdit.
Je ne suis pas sûr qu'il y ait un #include <string>

Citation:
Envoyé par Pierre Dolez
Bonjour,
Au risque d'avoir un second avertissement, la classe TSDIAppForm
est crée et déclarée automatiquement par Borland, ainsi
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 class TSDIAppForm : public TForm { __published: TOpenDialog *OpenDialog; TSaveDialog *SaveDialog; TToolBar *ToolBar1; ... ... private: public: virtual __fastcall TSDIAppForm(TComponent *AOwner); }; //---------------------------------------------------------------------------- extern TSDIAppForm *SDIAppForm; // Fin du .h Le .cpp Le constructeur //--------------------------------------------------------------------- __fastcall TSDIAppForm::TSDIAppForm(TComponent *AOwner) : TForm(AOwner) { }
C'est cela qui doit être considéré comme le main pour un utilisateur pas très expérimenté.
Il n'y a aucun inconvénient à avoir plusieurs .cpp pour une même classe, main naturellement les #include headers.h correspondants doivent figurer dans chacun des .cpp

Si, comme je le pense, l'autre .cpp ressemble à cela
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 //--------------------------------------------------------------------- #include <vcl.h> #pragma hdrstop //--------------------------------------------------------------------- USEFORM("sdimain.cpp", SDIAppForm); //--------------------------------------------------------------------------- WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR, int){ Application->Initialize(); Application->HelpFile = "Jeux_enfants"; Application->Title = "Jeux d'enfants"; Application->CreateForm(__classid(TSDIAppForm), &SDIAppForm); Application->Run(); return 0; } //---------------------------------------------------------------------
Alors, c'est le vrai main(), mais avant se savoir exactement ce qu'on fait, il ne vaut mieux pas y toucher.

Ces 3 lignes sont à la fin du .h
Code:

1 2 3 4 char data_recu[20]; // pour les valeu num string Oid_recu; // <----- bool blocage_snmp=false ;
dans votre code.
Vous faites une initialisation dans un .h ce qui est interdit.
Je ne suis pas sûr qu'il y ait un #include <string>
je pense que #include <string> n'est pas reconnu .

merci à tous !