Discussion :

[FITRI]

Salut, j'ai encodé ce code qui permettre de faire affichage sur le tableau de façon aléatoire. Il permettre d'afficher sur le 3 labels le résultat de cet aleatoire et chaque fois que les résultats de ces 3 label sont les mêmes on ajout 25 dans textBoxGagneForm2. Le problème c'est que il ajoute 25 dans textBoxGagneForm2 qu'une fois et le reste il ne veut plus additionner le résultat précédent avec le 25 quand les 3 label sont à nouveaux les mêmes valeurs.
Voici le code que j'ai fais:

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
 
private void b1DemarrerForm2_Click(object sender, EventArgs e)
        {
            /* DECLARATION DES VARIABLES */
            int iJeton, iJetonDep, iArgentGagner = 0;
 
            /* DECALARATION D'UN TABLEAU */
            string[] sTab = { "LIVRES", "FRUITS", "ANIMAUX", "GAZ", "BANQUE" };
 
            Random rndm1, rndm2, rndm3;
            rndm1 = new Random(unchecked((int)DateTime.Now.Ticks));
            /*******************************************************************************
             * Le nbr. aléatoires générés par Next ou NextDouble appliqués à rndm1 et rndm2*
             * seront maintenant différents (grâce à l'opérateur ~ d'inversion de bits qui *
             * donnera deux valeurs de base différents.                                    *
             *******************************************************************************/
            rndm2 = new Random(~unchecked((int)DateTime.Now.Ticks));
            rndm3 = new Random(unchecked((int)DateTime.Now.Millisecond));
 
            /* RENVOIE UN NOMBRE ALÉATOIRE SUP. OU ÉGALE À MINVAL ET INFÉRIEUR À MAXVAL */
            int iRand1 = rndm1.Next(0, 5);
            int iRand2 = rndm2.Next(0, 5);
            int iRand3 = rndm3.Next(0, 5);
 
            iJeton = Int32.Parse(tBox4ChangeJetonForm2.Text); //Convertir le type tBox4ChangeJetonForm2 en Int.
 
            if (iJeton > 0)
            {
                iJetonDep = iJeton - 1;
                tBox4ChangeJetonForm2.Text = iJetonDep.ToString();
 
                label4Form2.Text = sTab[iRand1];
                label5Form2.Text = sTab[iRand2];
                label6Form2.Text = sTab[iRand3];
 
                if ((iRand1 == iRand2)&& (iRand2 == iRand3))//Réalisé si condition1 et condition2 sont vraies
                {
                    iArgentGagner = iArgentGagner + 25;
                    tBox3GagneForm2.Text = iArgentGagner.ToString();
                }
            }
 
            else
            {
                MessageBox.Show("Vous n'avez plus de Jeton. Echanger votre argent!!");
            }  
        }

Pouriez-vous m'aider s.v.p. Merci d'avance

[Louis-Guillaume Morand]

parce que t'es étourdi
tu réinitialise argentgagner à zero à chaque fois que tu cliques sur le bouton, donc 0+25 puis plus tard 0+25, ca fera toujours 25 au final

revoit ton code

[FITRI]

parce que t'es étourdi
tu réinitialise argentgagner à zero à chaque fois que tu cliques sur le bouton, donc 0+25 puis plus tard 0+25, ca fera toujours 25 au final

revoit ton code

Mm... est-ce-que tu pourras me donner un peu idée comment, car je n'ai pas une bonne connaissance de C#. Merci

[Louis-Guillaume Morand]

je viens de te le dire, ton erreur c'est de remettre à Zero la variable iArgentGagner (en passant, perds l'habitude de mettre le type de la variable. c'est du fortement typé le C# donc tu mets le nom explicite et c'est tout).

donc dans ton clic, plutot que mettre
iArgentGagner = 0, tu dois recuperer la valeur de la textbox puis ajouter 25.

c'est juste une question de logique, pas de C# ou autre

[tomlev]

(en passant, perds l'habitude de mettre le type de la variable. c'est du fortement typé le C# donc tu mets le nom explicite et c'est tout).

Euh... tu voulais dire "prends l'habitude" plutôt, non ? Mais de toutes façons ça ne change rien, tu peux déclarer plusieurs variables de même type en spécifiant le type une seule fois. Personnellement j'aime pas trop, ça n'arrange pas la lisibilité, mais bon, c'est une question de préférence personnelle...

@FITRI : Simple curiosité de ma part... pourquoi utilises-tu "unchecked" pour initialiser les générateurs de nombre aléatoire ? Et pourquoi en utiliser plusieurs ? Tu peux très bien utiliser le même pour générer les 3 nombres aléatoires...

[StormimOn]

Euh... tu voulais dire "prends l'habitude" plutôt, non ? Mais de toutes façons ça ne change rien, tu peux déclarer plusieurs variables de même type en spécifiant le type une seule fois. Personnellement j'aime pas trop, ça n'arrange pas la lisibilité, mais bon, c'est une question de préférence personnelle...

Ce n'est pas ce qu'il a voulu dire je pense, mais il manque quelques mots dans sa phrase à ce moment.

En fait, tu ne dois pas indiquer le type de la variable dans le nom de celle-ci (iMonEntier pour indiquer que la variable est un entier par exemple), c'est une mauvaise habitude. Le nom d'une variable être lié à ce qu'elle représente et non au type associé

@FITRI : Simple curiosité de ma part... pourquoi utilises-tu "unchecked" pour initialiser les générateurs de nombre aléatoire ? Et pourquoi en utiliser plusieurs ? Tu peux très bien utiliser le même pour générer les 3 nombres aléatoires...

Il peut en utiliser un seul et préciser le seed ne sert à rien (99% du temps en tout cas) car par défaut on se base alors sur les ticks.

Par contre, si jamais tu dois utiliser plusieurs générateurs (instanciations multiples de Random) il faudra à ce moment préciser un seed différent pour chaque générateur si jamais l'instanciation est faite au même moment, car sinon les ticks n'auront probablement pas le temps de changer