Discussion :

[C tranchant]

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%
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 :

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
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 :

[Julien698]

Salut,

Essayes la même chose en remplaçant tes int par des doubles.

[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 :

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

un = un / 1000;

Réfléchis aux types de variables et à ton mode de calcul

[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

[C tranchant]

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

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
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);
                }
 
            }
 
 
        }

[popo]

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

[MicBru]

Code :

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

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.