Variable qui s'initialise à 0 toute seule

Version imprimable

Salut Salut!
Voilà, je dev un petit programme pour m'amuser.
J'utilise c++ builder, et voici le code:

.cpp

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::FormCreate(TObject *Sender)
{
int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;
bool PouvoirNuc; //Variable Pouvoir Nucléaire
 
//Dimensions
Form1->Height=276;
Form1->Width=705;
Form1->Top=260;
Form1->Left=315;
 
Image4->Visible=false;  //Je cache l'icone nucléaire de J1 en début de partie
Image5->Visible=false;  //Je cache l'icone nucléaire de J2 en début de partie
 
Timer1->Enabled=true; //J'allume l'objet Timer
Timer1->Interval=400; //J'initialise le temps de répétition à 0.4 seconde
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Timer1Timer(TObject *Sender)
{
//Toutes les fonctions se trouvant ici seront répétées en fonction de l'interval du Timer1
 
Label1->Caption=AnsiString(ArgentJ1)+" $";  //Affichage de l'argent du joueur 1
Label2->Caption=AnsiString(ArgentJ2)+" $";  //Affichage de l'argent du joueur 2
 
ArgentJ1=ArgentJ1+1;    //J'incrémente l'argent de J1 de 1$
ArgentJ2=ArgentJ2+1;    //J'incrémente l'argent de J2 de 1$
 
        if(ArgentJ1<=10000)        //Si J1 à assez d'argent, j'active son pouvoir Nuc
        {
        Image4->Visible=true;   //L'image de l'icone nucléaire devient visible
        }
        else
        {
        Image4->Visible=false;
        }
 
        if(ArgentJ2<=10000)        //Si J2 à assez d'argent, j'active son pouvoir Nuc
        {
        Image5->Visible=true;   //L'image de l'icone nucléaire devient visible
        }
        else
        {
        Image5->Visible=false;
        }
}
//---------------------------------------------------------------------------

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
//---------------------------------------------------------------------------
 
#ifndef Unit1H
#define Unit1H
//---------------------------------------------------------------------------
#include <Classes.hpp>
#include <Controls.hpp>
#include <StdCtrls.hpp>
#include <Forms.hpp>
#include <ExtCtrls.hpp>
#include <jpeg.hpp>
#include <Graphics.hpp>
//---------------------------------------------------------------------------
class TForm1 : public TForm
{
__published:	// IDE-managed Components
        TImage *Image1;
        TImage *Image2;
        TImage *Image3;
        TTimer *Timer1;
        TLabel *Label1;
        TLabel *Label2;
        TLabel *Label3;
        TLabel *Label4;
        TImage *Image4;
        TImage *Image5;
        void __fastcall FormCreate(TObject *Sender);
        void __fastcall Timer1Timer(TObject *Sender);
private:	// User declarations
public:		// User declarations
        int ArgentJ1;
        int ArgentJ2;
        int VieJ1;
        int VieJ2;
        __fastcall TForm1(TComponent* Owner);
};
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif

J'utilise le Timer pour répéter l'action qui consiste à gagner 1$ toutes les 0.4 seconde.
Le problème c'est que j'initialise la valeur de l'argent à 2000$, mais lorsque j’exécute le programme elle reste 0.4 seconde sur 2000$,
s'initialise à 0 puis ensuite ca repart (0$,1$,2$...)

Moi, j'aimerai que ça parte directement de 2000$... Comment faire SVP :) Merci d'avance aux courageux !

19/05/2014, 22h31
foetus

C'est normal :mrgreen: ... sauf que ton compilateur devrait te dire que tu caches des attributs de classe.

Dans ta méthode FormCreate tu crées 4 entiers (+ PouvoirNuc) avec le même nom que tes attributs.
Mais tes attributs de classe ne sont pas changés (donc ils valent 0 grâce au constructeur).

De toute manière je pense fortement que les attributs de ta classe doivent être initialisés dans le constructeur et non pas dans cette méthode.
19/05/2014, 22h49
cedni

Salut

Deplaces :
int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;
en dessous de :
TForm1 *Form1;
19/05/2014, 23h01
Invité
Bonsoir,

Non ! Pas la peine de s'encombrer avec des variables globales alors qu'on a des attributs.
Comme l'a très bien fait remarquer foetus, tu crées de nouvelles variables locales au lieu d'utiliser les attributs existants.

Dans TForm1::FormCreate remplace :

Code:

int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;

par :
Code:

1 2 3 4 ArgentJ1 = 2000; ArgentJ2 = 2000; VieJ1 = 20; VieJ2 = 20;
(voir plutôt dans la liste d'initialisation du constructeur).
19/05/2014, 23h02
algerino77

Citation:

Envoyé par foetus

C'est normal :mrgreen: ... sauf que ton compilateur devrait te dire que tu caches des attributs de classe.

Dans ta méthode FormCreate tu crées 4 entiers (+ PouvoirNuc) avec le même nom que tes attributs.
Mais tes attributs de classe ne sont pas changés.

De toute manière je pense fortement que les attributs de ta classe doivent être initialisés dans le constructeur et non pas dans cette méthode.

Salut :)

Tu emplois beaucoup de mots que je connais, mais je t'avoues avoir du mal avec tout ce vocabulaire..
D’après ce que je comprends, c'est dans le .h que ca cafouille ?
Il faudrait donc que j'initialise mes variables dans TForm1 ?
19/05/2014, 23h06
algerino77
Citation:

Envoyé par cedni

Salut

Deplaces :
int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;
en dessous de :
TForm1 *Form1;

Salut, j'avais déja essayé, mais ca ne marche pas..
Citation:
Envoyé par Winjerome

Bonsoir,

Non ! Pas la peine de s'encombrer avec des variables globales alors qu'on a des attributs.
Comme l'a très bien fait remarquer foetus, tu crées de nouvelles variables locales au lieu d'utiliser les attributs existants.

Dans TForm1::FormCreate remplace :

Code:

int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;

par :

Code:

1 2 3 4 ArgentJ1 = 2000; ArgentJ2 = 2000; VieJ1 = 20; VieJ2 = 20;

(voir plutôt dans la liste d'initialisation du constructeur).
Génial! Ca marche !!
Mais.. Pourquoi?? Elles sont de quel type mes variables alors maintenant ?? Merci de m’éclairer..
19/05/2014, 23h13
Invité
Elles sont du type que tu as déclaré dans l'en-tête :
Code:

1 2 3 4 5 6 7 8 class TForm1 : public TForm { public: // User declarations int ArgentJ1; int ArgentJ2; int VieJ1; int VieJ2; };
Avec ta ligne

Code:

int ArgentJ1=2000, ArgentJ2=2000, VieJ1=20, VieJ2=20;

tu créais et initialisais de nouvelles variables du même nom qui n'avaient rien à voir, variables qui étaient détruites à la fin de la méthode.
Par la suite, dans Timer1Timer(), tu utilisais les variables membres qui elles n'avaient pas été initialisées.
19/05/2014, 23h15
foetus
Citation:

Envoyé par algerino77

Mais.. Pourquoi?? Elles sont de quel type mes variables alors maintenant ?? Merci de m’éclairer..

Regardes la déclaration TForm1::FormCreate: cela veut dire que tu implémentes la méthode FormCreate de la classe TForm1.

Et comme c'est une méthode tu peux utiliser tous les attributs (public ou private) que tu as défini dans ton fichier header .h
Code:

1 2 3 4 5 6 7 8 9 10 11 .... private: // User declarations public: // User declarations int ArgentJ1; int ArgentJ2; int VieJ1; int VieJ2; .... };
19/05/2014, 23h18
algerino77

C'est parfait! Merci beaucoup à tous :) Vraiment !

C++ permet également d'initialiser les variables dans le constructeur de ta classe
ton code sera alors le suivant et plus besoin d'utiliser la procédure FormCreate qui est très peu utilisée en C++ Builder

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner),   // liste d'initialisation
          ArgentJ1(2000),
          ArgentJ2(2000),
          VieJ1(20),
          VieJ2(20)
{
   Height=276;
   Width=705;
  Top=260;
   Left=315;
 
Image4->Visible=false;  //Je cache l'icone nucléaire de J1 en début de partie
Image5->Visible=false;  //Je cache l'icone nucléaire de J2 en début de partie
 
Timer1->Enabled=true; //J'allume l'objet Timer
Timer1->Interval=400; //J'initialise le temps de répétition à 0.4 seconde
}

cdlt