Discussion :

[snake264]

Bonjour à tous
Voila mon problème est le suivant j'ai fais un mastermind pour un tp que l'on doit faire et lorsque j'execute mon code, il me genere 5 fois le même nombre dans la combinaison que l'on doit trouvée et lorsque le résultat s'affiche pour dire qu'on a gagné ou perdu il affiche la combinaison à trouvé mais avec +1 pour le dernié chiffre. (ex: il me met comme combinaison à trouvé 4 | 4 | 4 | 4 | 4 et quand il le réaffiche pour donné la solution il me met 4 | 4 | 4 | 4 | 5 ).
Puis lorsqu'il y a un double dans la solution par exemple 3 | 6 | 7 | 3 | 5 et que je rentre 3 | 9 | 3 | 8 | 2 il me met qu'il y a 2 mal placé alors qu'il devrait y en avoir un seul.
Ensuite lorsqu'il y a un double dans la réponse qui n'est pas dans la solution par exmple 3 | 4 | 5 | 6 | 2 et que je rentre 3 | 3 | 9 | 9 | 9 il me met qu'il y a deux mal placé alors qu'il y en a que un qui est bien placé et un qui n'y est pas du tout. (ces deux derniés sont des problèmes que j'avais remarquer lorsque je rentrait la combinaison à la main avant que je fasse une combinaison aléatoire)
En espérant que vous puissiez m'aider. Merci.

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
#include<stdio.h>
#include<time.h>
#define taille 5
 
int genereNombre(int);
int init(int []);
int saisie(int []);
int mal_place(int [],int[]);
int bien_place(int [],int []);         
 
int main ()
{
	int tab[taille],tab2[4], i,reussi,perdu,essai=0;
 
	printf (" ***********************\n");
    printf (" * MASTERMIND *\n");
    printf (" ***********************\n\n");
    printf ("Bienvenue sur le jeu de Mastermind\n");
    init (tab);
    for (i=0; i < taille; i++)
    {
		printf (" %d | ",tab[i]);
    }
    printf ("\t");
    printf (" \nLes chiffres ont été générés aléatoirement\n\n");
    printf (" Début du jeu : \n\n");
    while (essai != 5)
    {
		saisie (tab2);
        reussi = bien_place (tab,tab2);
        mal_place (tab,tab2);
        essai++;
        for (i=0; i < taille; i++)
        {
			printf (" %d | ",tab2[i]);
        }
        if (reussi == taille)
        {
			essai = 5;
            printf ("Félicitation vous avez gagné!!!\n");
        }
        else if ((essai == 5) && (reussi !=4))
        {
            printf ("\n\nVous avez perdu!!!\n");
        }
    }
}
 
int init(int tab[])
{
	int i,n ;
 
    printf ("Jusqu'à quel valeur voulez vous initialiser le jeu??\t");
    scanf ("%d",&n);
    for (i=0; i< taille; i++)
    {
		tab[i] = genereNombre(n);
    }
}
 
int genereNombre(int MAX)
{
	int MIN=1;
	srand(time(NULL));
	return (rand() % (MAX - MIN + 1)) + MIN;
}
 
int saisie (int tab2 [])
{
	int i;
        int ch;
 
    printf ("\n\n");
    for (i=0; i < taille; i++)
    {
		printf ("chiffre %d : ",i+1);
                scanf ("%d",&ch);
                tab2[i] = ch;
    }
}
 
int mal_place (int tab[taille] , int tab2[taille])
{
        int i,a=0,m=0;
        int tab3[4];
 
    for (i = 0; i< taille; i++)
    {
		if (tab[i]!=tab2[i])
		{
	                 for (a = 0; a< taille; a++)
                         {
				if (tab2[a] == tab[i])
                                {
					m++;
                                }
                         }
                 }
    }
    printf ("\nIl y a %d chiffres mal placés\n\n",m);
    return m;
}
 
int bien_place (int tab[taille] , int tab2[taille])
{
	int i,mal=0;
 
	for (i = 0; i< taille; i++)
       {
		if (tab[i]==tab2[i])
		{
			mal++;
                }
        }
        printf ("\nIl y a %d chiffres bien placés\n\n",mal);
        return mal;
}

[diogene]

Code :

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

1
2
3
4
5
6
#define taille 5
...       
int main ()
{
	int tab[taille],tab2[4] ...
...

Dans cette fonction (et dans les autres), tu modifies ou accèdes à tab2[taille-1] (soit tab2[4]) qui n'est pas un élément du tableau tab2

[snake264]

A oui je J'avais oublier de le changer car avant je n'avais pas mis de define et j'avais défini le nombre de chiffre pour la combinaison à 4.

Pour le problème ou il me donne 5 fois les mêmes chiffres est ce que c'est à cause du fait que je demande la fonction genereNombre() à chaque fois jusqu'à la fin de la boucle ?

Par contre pour les deux autres je ne vois vraiment pas

[Emmanuel Delahaye]

Voila mon problème est le suivant j'ai fais un mastermind pour un tp que l'on doit faire et lorsque j'execute mon code,

Un compilateur bien réglé montre que ce code est largement améliorable :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:12: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:13: warning: unused variable `perdu'
main.c:47: warning: control reaches end of non-void function
main.c: In function `init':
main.c:59: warning: control reaches end of non-void function
main.c: In function `genereNombre':
main.c:64: warning: implicit declaration of function `srand'
main.c:65: warning: implicit declaration of function `rand'
main.c: In function `saisie':
main.c:80: warning: control reaches end of non-void function
main.c: In function `mal_place':
main.c:85: warning: unused variable `tab3'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 5 seconds)
0 errors, 8 warnings

Code :

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

1
2
3
4
5
6
7
 
int genereNombre(int MAX)
{
	int MIN=1;
	srand(time(NULL));
	return (rand() % (MAX - MIN + 1)) + MIN;
}

srand() est mal placé : http://emmanuel-delahaye.developpez.com/notes.htm#rand
Je propose ça :

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
 
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
 
#define NB_ELEM(a) (sizeof(a)/sizeof*(a))
 
static void purge (void)
{
   int c;
   while ((c = getchar ()) != '\n' && c != EOF)
   {
   }
}
 
static int genereNombre (int MAX)
{
   int MIN = 1;
   return (rand () % (MAX - MIN + 1)) + MIN;
}
 
static void init (int tab[], size_t taille)
{
   int n;
 
   printf ("Jusqu'a quel valeur voulez vous initialiser le jeu ? ");
   scanf ("%d", &n);
   purge ();
   {
      size_t i;
      for (i = 0; i < taille; i++)
      {
         tab[i] = genereNombre (n);
      }
   }
}
 
static void aff (int tab[], size_t taille)
{
   size_t i;
   for (i = 0; i < taille; i++)
   {
      printf (" %d | ", tab[i]);
   }
}
 
static void saisie (int tab[], size_t taille)
{
   size_t i;
 
   printf ("\n\n");
   for (i = 0; i < taille; i++)
   {
      int ch;
      printf ("chiffre %d : ", i + 1);
      scanf ("%d", &ch);
      purge ();
      tab[i] = ch;
   }
}
 
static int mal_place (int tab[], int tab2[], size_t taille)
{
   int m = 0;
   size_t i;
 
   for (i = 0; i < taille; i++)
   {
      if (tab[i] != tab2[i])
      {
         size_t a;
         for (a = 0; a < taille; a++)
         {
            if (tab2[a] == tab[i])
            {
               m++;
            }
         }
      }
   }
   printf ("\nIl y a %d chiffres mal places\n\n", m);
   return m;
}
 
static int bien_place (int tab[], int tab2[], size_t taille)
{
   int mal = 0;
   size_t i;
 
   for (i = 0; i < taille; i++)
   {
      if (tab[i] == tab2[i])
      {
         mal++;
      }
   }
   printf ("\nIl y a %d chiffres bien placés\n\n", mal);
   return mal;
}
 
int main (void)
{
   srand (time (NULL));
 
   printf (" ***********************\n");
   printf (" * MASTERMIND *\n");
   printf (" ***********************\n\n");
   printf ("Bienvenue sur le jeu de Mastermind\n");
 
   {
      int tab[5];
      int essai = 0;
 
      init (tab, NB_ELEM (tab));
      aff (tab, NB_ELEM (tab));
 
      printf ("\t");
      printf (" \nLes chiffres ont été générés aléatoirement\n\n");
      printf (" Début du jeu : \n\n");
 
      do
      {
         int tab2[NB_ELEM (tab)];
         saisie (tab2, NB_ELEM (tab));
         {
            int reussi = bien_place (tab, tab2, NB_ELEM (tab));
 
            mal_place (tab, tab2, NB_ELEM (tab));
            essai++;
 
            aff (tab2, NB_ELEM (tab2));
 
            if (reussi == NB_ELEM (tab))
            {
               essai = 5;
               printf ("Félicitation vous avez gagné!!!\n");
            }
            else if ((essai == 5) && (reussi != NB_ELEM (tab)))
            {
               printf ("\n\nVous avez perdu!!!\n");
            }
         }
      }
      while (essai != 5);
   }
   return 0;
}

[snake264]

Arf je m'en doutait bien car j'ai beaucoup hésité sur ou la mettre et pourtant j'ai esayer de la placer en tt début de programme, toujours pareil puis aussi en tout début de la fonction init() et idem, pourtant je ne l'appelle qu'une seule fois :/

[Emmanuel Delahaye]

Arf je m'en doutait bien car j'ai beaucoup hésité sur ou la mettre et pourtant j'ai esayer de la placer en tt début de programme, toujours pareil puis aussi en tout début de la fonction init() et idem, pourtant je ne l'appelle qu'une seule fois :/

D'abord, tu l'appelles 5 fois. Ensuite, pour éviter tout risque, il faut placer srand() au début de l'application horse de toute boucle. C'est le bon sens même non ?

[snake264]

Je c'est bien mais quand je la met au début de la fonction init sa me met toujours :

Code :

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

1
2
3
4
essai2.c: Dans la fonction «init» :
essai2.c:54: attention : déclaration implicite de la fonction « «srand» »
essai2.c: Dans la fonction «genereNombre» :
essai2.c:68: attention : déclaration implicite de la fonction « «rand» »

Même si sa marche, c'est sa que je comprend pas Mais c'est pas grave du moment que sa marche je m'occuperais de cela plus tard maintenant faut que je trouve une solution à mes deux autres petits soucis qui commencent à me faire craker^^
Voici le code modifié depuis tout à leur:

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
#include<stdio.h>
#include<time.h>
#define taille 5
 
int genereNombre(int MAX);
int init(int tab[]);
int saisie(int tab2[]);
int mal_place(int tab[taille],int tab2[taille]);
int bien_place(int tab[taille],int tab2[taille]);         
 
int main(void)
{
    int tab[taille],tab2[taille], i,reussi,essai=0;
 
    printf (" **************\n");
    printf (" * MASTERMIND *\n");
    printf (" **************\n\n");
    printf ("Bienvenue sur le jeu de Mastermind\n");
    init (tab);
    for (i=0; i < taille; i++)
    {
		printf (" %d | ",tab[i]);
    }
    printf ("\t");
    printf (" \nLes chiffres ont été générés aléatoirement\n\n");
    printf (" Début du jeu : \n\n");
    while (essai != 5)
    {
	saisie (tab2);
        reussi = bien_place (tab,tab2);
        mal_place (tab,tab2);
        essai++;
        for (i=0; i < taille; i++)
        {
			printf (" %d | ",tab2[i]);
        }
        if (reussi == taille)
        {
			essai = 5;
            printf ("Félicitation vous avez gagné!!!\n");
        }
        else if ((essai == 5) && (reussi !=4))
        {
            printf ("\n\nVous avez perdu!!!\n");
        }
    }
    return 0;
}
 
int init(int tab[])
{
    int i,n ;
 
    srand(time(NULL));
    printf ("Jusqu'à quel valeur voulez vous initialiser le jeu??\t");
    scanf ("%d",&n);
    for (i=0; i< taille; i++)
    {
		tab[i] = genereNombre(n);
    }
    return tab[i];
}
 
int genereNombre(int MAX)
{
	int MIN=1;
 
	return (rand() % (MAX - MIN + 1)) + MIN;
}
 
int saisie(int tab2[])
{
    int i;
    int ch;
 
    printf ("\n\n");
    for (i=0; i < taille; i++)
    {
	printf ("chiffre %d : ",i+1);
        scanf ("%d",&ch);
        tab2[i] = ch;
    }
    return tab2[i];
}
 
int mal_place(int tab[taille] , int tab2[taille])
{
    int i,a=0,m=0;
 
    for (i = 0; i< taille; i++)
    {
		if (tab[i]!=tab2[i])
		{
			for (a = 0; a< taille; a++)
                        {
				if (tab2[a] == tab[i])
                                {
					m++;
                                }
                        }
               }
    }
    printf ("\nIl y a %d chiffres mal placés\n\n",m);
    return m;
}
 
int bien_place(int tab[taille] , int tab2[taille])
{
	int i,mal=0;
 
	for (i = 0; i< taille; i++)
        {
		if (tab[i]==tab2[i])
		{
			mal++;
                }
        }
        printf ("\nIl y a %d chiffres bien placés\n\n",mal);
        return mal;
}

[diogene]

Je c'est bien mais quand je la met au début de la fonction init sa me met toujours :

Code :

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

1
2
3
4
essai2.c: Dans la fonction «init» :
essai2.c:54: attention : déclaration implicite de la fonction « «srand» »
essai2.c: Dans la fonction «genereNombre» :
essai2.c:68: attention : déclaration implicite de la fonction « «rand» »

Ajoute un #include <stdlib.h> en tête du fichier qui contient la définition de init