Exercice autour de Random et résultat incompréhensible.

Version imprimable

1 pièce(s) jointe(s)

Bonjour tout le monde :),

je suis en train d'apprendre les bases du C#, je débute avec ce langage comme en programmation en général (excepté un peu de Basic au collège).
Je me suis donné un exercice consistant à simuler des lancers de dé à six faces en utilisant Random et à vérifier la distribution des résultat de ces lancers.
Jusqu'ici tout va bien sauf que, lorsque je vérifie le total de pourcentage (normalement 100%) je tombe sur 96 ou 97 ou 98% :weird:
Sauriez vous expliquer si j'ai fais une erreur quelque part dans mes calculs ou si il y a une mécanique invisible qui est à l'œuvre ici?

Merci d'avance pour votre temps et vos éventuelles réponses.

Le code en question :

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
66
67
68
69
70
71
72
73
74
75
76
77
78
namespace ConsoleApp3
{
    class Testeur
    { 
 
        static void Main(string[] args)
 
        {
            int un = 0, deux = 0, trois = 0, quatre = 0, cinq = 0, six = 0, compte = 0, result = 0;
 
        begin:
 
            compte += 1;
            Random rnd = new Random();
            result = rnd.Next(1, 7);
            switch(result)
            {
                case 1:
                    un += 1;
                    break;
                case 2:
                    deux += 1;
                    break;
                case 3:
                    trois += 1;
                    break;
                case 4:
                    quatre += 1;
                    break;
                case 5:
                    cinq += 1;
                    break;
                case 6:
                    six += 1;
                    break;
            };
 
 
            if (compte == 100000)
            {
                int totalPerCent = 0;
 
                un = un / 1000;
                deux = deux / 1000;
                trois = trois / 1000;
                quatre = quatre / 1000;
                cinq = cinq / 1000;
                six = six / 1000;
                totalPerCent = un + deux + trois + quatre + cinq + six;
 
 
                Console.WriteLine("Il y a " + un + "% de lancers de valeur 1");
                Console.WriteLine("Il y a " + deux + "% de lancers de valeur 2");
                Console.WriteLine("Il y a " + trois + "% de lancers de valeur 3");
                Console.WriteLine("Il y a " + quatre + "% de lancers de valeur 4");
                Console.WriteLine("Il y a " + cinq + "% de lancers de valeur 5");
                Console.WriteLine("Il y a " + six + "% de lancers de valeur 6");
                Console.WriteLine("Ce qui donne " + totalPerCent + "% !!??!!");
                Console.ReadKey(true);
            }
 
            else
            goto begin;
 
 
 
        }
 
        static int RollDice(int result)
        {
            Random rnd = new Random();
 
            result = rnd.Next(1, 7);
 
            return result;
        }
    }
}

Le résultat :

Pièce jointe 351761

16/02/2018, 15h40
Julien698

Salut,

Essayes la même chose en remplaçant tes int par des doubles.
16/02/2018, 15h45
Jean-Philippe André

Salut,

si tes variables un, deux, trois... sont des entiers, que leur arrivent-elles à ton avis lorsque tu tentes de faire une division et que cela donne un nombre à virgule ?

Code:

un = un / 1000;

Réfléchis aux types de variables et à ton mode de calcul :)
16/02/2018, 15h51
PixelJuice

Bonjour,

comme l'a dit Julien, remplace des ints par des floats ou doubles. Les ints n'ont pas de précision (comprends par la que ce sont des entiers, donc pas de chiffres après la virgule). Tu auras toujours entre 0 et 6% qui manque dans ton total.

Par contre, autant prendre les bonnes habitudes dès le début, le vilain goto est a proscrire. Il est propice a du mauvais code, dit code spaghetti

Merci beaucoup pour vos réponse et conseil, erreur bête de ma part pour ce qui est de la définition des variable.
La majorité de mes variables sont passé de int à double et j'ai remplacé ce vilain goto par while ce qui fonctionne très bien :D

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
  static void Main(string[] args)
 
        {
            double un = 0, deux = 0, trois = 0, quatre = 0, cinq = 0, six = 0;
            int compte = 0, result = 0;
 
            Random rnd = new Random();
 
            while (compte != 100000)
            {
 
                compte ++;
 
                result = rnd.Next(1, 7);
                switch (result)
                {
                    case 1:
                        un ++;
                        break;
                    case 2:
                        deux ++;
                        break;
                    case 3:
                        trois ++;
                        break;
                    case 4:
                        quatre ++;
                        break;
                    case 5:
                        cinq ++;
                        break;
                    case 6:
                        six ++;
                        break;
                }
 
 
                if (compte == 100000)
                {
                    double totalPerCent = 0;
 
                    un = un / 1000;
                    deux = deux / 1000;
                    trois = trois / 1000;
                    quatre = quatre / 1000;
                    cinq = cinq / 1000;
                    six = six / 1000;
                    totalPerCent = un + deux + trois + quatre + cinq + six;
 
 
                    Console.WriteLine("Il y a " + un + "% de lancers de valeur 1");
                    Console.WriteLine("Il y a " + deux + "% de lancers de valeur 2");
                    Console.WriteLine("Il y a " + trois + "% de lancers de valeur 3");
                    Console.WriteLine("Il y a " + quatre + "% de lancers de valeur 4");
                    Console.WriteLine("Il y a " + cinq + "% de lancers de valeur 5");
                    Console.WriteLine("Il y a " + six + "% de lancers de valeur 6");
                    Console.WriteLine("Ce qui donne bien" + totalPerCent + "%");
                    Console.ReadKey(true);
                }
 
            }
 
 
        }

20/02/2018, 11h49
popo

Dans ce cas, passe la discussion à "Résolu" (bouton en bas à gauche, je crois)

Citation:

Envoyé par C tranchant

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
  static void Main(string[] args)
        {
            double un = 0, deux = 0, trois = 0, quatre = 0, cinq = 0, six = 0;
            int compte = 0, result = 0;
            Random rnd = new Random();
            while (compte != 100000)
            {
                 ...
            }
            if (compte == 100000)
            {
                   ...
            }
         }

Presque parfait. :)
Mais maintenant le "if" n'est plus nécessaire. 8-)