mastermind en C

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : mastermind en C

fia123

Bonjour à tous,

Pour mon projet de programmation, je dois réaliser le mastermind en C, je l'ai terminé et il n'y a pas d'erreur de compilation, seulement c'est impossible de gagner, quelque soit la combinaison que l'on rentre, le programme n'indique aucun point blanc ni noir, et je ne comprend pas d'où vient l'erreur .. Si quelqu'un pouvait m'éclairer et me dire quelle partie du code est incorrecte, ce serait vraiment sympa

Code :

#include <stdio.h> // Fonctions getchar, printf()
#include <stdlib.h> // Fonctions system()
#include <string.h>
#include <time.h>
int menu();
void jeu();
void regles();
// ---------------------------------------------------------------------------------------------------------------------------------------------
void initCode(int *codeSecret);
int main ()
{
int continuer = 1, choix = 0;
int i;
int *codeSecret[4];
for(i=0; i<4; i++)
printf("%c", codeSecret[i]);
while (continuer) //Tant que continuer vaut vrai (1)
{
choix = menu();
switch (choix)
{
case 1:
jeu();
break;
// ----------------------------------
case 2:
regles();
break;
// ----------------------------------
case 3:
// Quitter
continuer = 0;
exit(-1);
break;
// ----------------------------------
default:
printf("Choix errone\n" );
}
// ----------------------------------
} // while
return 0;
}
// ------------------------------------------------------------------------------------------------------------------------------------------------
int menu()
{
int choix = 0;
system("cls" ); // Nettoyer la console
printf("======== MENU ========\n\n" );
printf("(1) : Jouer\n" );
printf("(2) : Regles du jeu\n" );
printf("(3) : Quitter\n\n" );
printf("======================\n\n\n" );
printf("Que voulez vous faire ?\n\n" );
scanf("%d", &choix);
return choix;
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void regles()
{
int choix = 0;
system("cls" ); //Nettoyer la console
//Affichage des règles du Mastermind
printf("***********************\n" );
printf("*REGLES DU MASTERMIND:*\n" );
printf("***********************\n" );
printf("\nLe programme choisit une combinaison de 4 pions de couleurs au hasard\n" );
printf("parmi 6 couleurs disponibles.\n" );
printf("Cette combinaison de pions peut contenir plusieurs fois la meme couleur.\n" );
printf("L utilisateur essaie de la deviner en proposant au maximum dix combinaisons.\n" );
printf("Pour chaque tentative le programme indique le nombre de couleurs bien placees\n" );
printf("(a l aide de pion noir) et le nombre de couleurs mal placees\n" );
printf("(a l aide de pion blanc). Si le nombre maximal d essais est atteint,\n" );
printf("le programme indique au joueur qu il a perdu et donne la combinaison.\n\n" );
printf("------------------------------------------------------------------------------\n\n" );
printf("Entrez (0) pour revenir au menu principale\n" );
scanf("%d", &choix);
if (choix == 0)
{
menu();
}
else
{
regles();
}
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void jeu()
{
int essais=1,choix=0, ligne=9,continuer=1,verif=0,a; // "entrer" dans la boucle jeu
char tab[10][8]={
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46}
};
char comb[4];
system("cls" ); // Nettoyer la console
// --------------------------------------------------------------------------------------------------------
// Générer une combinaison aléatoire
void initCode(int *codeSecret)
{
int i;
srand (time(NULL));
/* initialiser le code secret avec un choix de 6 couleurs */
for (i = 0; i < 4 ; i++)
{
switch (rand() % 6)
{
case 0: codeSecret[i] = 'r'; break;
case 1: codeSecret[i] = 'b'; break;
case 2: codeSecret[i] = 'v'; break;
case 3: codeSecret[i] = 'j'; break;
case 4: codeSecret[i] = 'o'; break;
case 5: codeSecret[i] = 'm'; break;
}
}
}
// -----------------------------------------------------------------------------------------------------------------------------------------------------
// Jeu du MASTERMIND
while (essais <= 10)
{
int i, j, t, test=0, L=10, verif=0; // L : numéroter la ligne du tableau à partir de 10. ligne -> monter d'une ligne lorsque l'on entre une combinaison
char rep[4]={'\0'};
int a,b; // variable pour la vérification des combinaisons entrées
system("cls" ); // Nettoyer la console
printf("\nCouleurs disponibles : \n" );
printf("=========================================================\n" );
printf("r=Rouge - v=Vert - b=Bleu - j=Jaune - o=Orange - m=Marron \n" );
printf("=========================================================\n\n" );
// initialisation du tableau
for (i=0;i<=9;i++)
{
printf ("[%3d]-[",L);
for (j=0;j<=7;j++)
{
printf ("(%c)",tab[i][j]);
}
printf (" ]\n" );
L--;
}
//printf("\n comb : %s",comb);
// --------------------------------------------------------------------------------------------------------
// Entrer une combinaison
while (verif != 4)
{
verif=0;
printf("\n\nEssai n# %d: \n",essais);
scanf("%s",&rep);
fflush(stdin);
// vérification de la combinaison entrée
for (a=0;a<=3;a++)
{
if (rep[a]=='r' || rep[a]=='v' || rep[a]=='b' || rep[a]=='j' || rep[a]=='o' || rep[a]=='m' )
{
verif++;
}
else
{
verif--;
}
}
if (verif !=4)
{
printf("\nCombinaison impossible. Recommencez !" );
}
}
// -----------------------------------------------------------------------------------------------------------
// Ajout dans le combinaison dans le tableau
test = strcmp(rep,comb); // on compare les 2 chaine de caractère, test sera négatif si s1 est < à s2, nul si s1 est == à s2 ou positif si s1 est > à s2
if (test==0)
{
essais=11; // le joueur a trouvé la combinaison, on quitte la boucle jeu
}
else
{
for (j=0;j<=3;j++) // se déplacer de cellule en cellule
{
for (t=0;t<=3;t++) // permet de se déplacer dans la chaine de caractère comb, afin d'analyser chaque lettre
{
if (j == t) // si j == t, on est à la même position dans la chaine
{
if (rep[j]==comb[t]) // Si le caractère n°x est égale au caractère n°x de la combinaison
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=78; // on affiche N (ascii 78)
}
}
else
{
if (rep[j]==comb[t]) // Si le caractère n°x n'est pas égale au caractère n°x, mais qu'il figure dans la combinaison
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=66; // on affiche B (ascii 66)
}
else // Sinon on affiche un point
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=46; // on affiche un point (ascii 46)
}
}
}
}
}
fflush(stdin);
essais++;
ligne--;
}
// ---------------------------------------------------------------------------------------
// Jeu terminé
if (essais == 12) // si essais == 12, alors le joueurs a trouvé la combinaison
{
printf("Jeu termine" );
printf("\n\n ****** B R A V O *****\n" );
printf("\n **!!! VOUS AVEZ TROUVE LA COMBINAISON !!!**\n" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
else
{
system("cls" ); // Nettoyer la console
printf("Jeu termine" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
// ---------------------------------------------------------------------------------------
if (choix == 0)
{
menu();
}
else
{
jeu();
}
}

Message cité 1 fois

Publicité

Sve@r

fia123 a écrit :

Bonjour à tous,

Pour mon projet de programmation, je dois réaliser le mastermind en C, je l'ai terminé et il n'y a pas d'erreur de compilation, seulement c'est impossible de gagner, quelle que soit la combinaison que l'on rentre, le programme n'indique aucun point blanc ni noir, et je ne comprends pas d'où vient l'erreur .. Si quelqu'un pouvait m'éclairer et me dire quelle partie du code est incorrecte, ce serait vraiment sympa

Salut
Bravo pour ton orthographe presque irréprochable. Comparé à ce que certains niaiseux écrivent ici, ça fait déjà plaisir à lire.

Bon maintenant voyons ce code...

fia123 a écrit :

Code :

#include <stdio.h> // Fonctions getchar, printf()
#include <stdlib.h> // Fonctions system()
#include <string.h>
#include <time.h>
int menu();
void jeu();
void regles();
// ---------------------------------------------------------------------------------------------------------------------------------------------
void initCode(int *codeSecret);
int main ()
{
int continuer = 1, choix = 0;
int i;
int *codeSecret[4]; // Pourquoi codeSecret est-il un tableau de 4 pointeurs ???
for(i=0; i<4; i++)
printf("%c", codeSecret[i]);
while (continuer) //Tant que continuer vaut vrai (1) hum... des commentaires oui mais des commentaires intelligents !!! Expliquer par exemple quelle va être l'action du bout de code. Traduire en français une instruction C n'est pas très utile...
{
choix = menu();
switch (choix)
{
case 1:
jeu();
break;
// ----------------------------------
case 2:
regles();
break;
// ----------------------------------
case 3:
// Quitter
continuer = 0;
exit(-1); // Inutile ici (et de façon générale mettre un exit (en dehors d'un debug) est souvent synonyme de mauvaise conception
break;
// ----------------------------------
default:
printf("Choix errone\n" );
}
// ----------------------------------
} // while
return 0;
}
// ------------------------------------------------------------------------------------------------------------------------------------------------
int menu()
{
int choix = 0;
system("cls" ); // Nettoyer la console
printf("======== MENU ========\n\n" );
printf("(1) : Jouer\n" );
printf("(2) : Regles du jeu\n" );
printf("(3) : Quitter\n\n" );
printf("======================\n\n\n" );
printf("Que voulez vous faire ?\n\n" );
scanf("%d", &choix);
return choix;
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void regles()
{
int choix = 0;
system("cls" ); //Nettoyer la console
//Affichage des règles du Mastermind
printf("***********************\n" );
printf("*REGLES DU MASTERMIND:*\n" );
printf("***********************\n" );
printf("\nLe programme choisit une combinaison de 4 pions de couleurs au hasard\n" );
printf("parmi 6 couleurs disponibles.\n" );
printf("Cette combinaison de pions peut contenir plusieurs fois la meme couleur.\n" );
printf("L utilisateur essaie de la deviner en proposant au maximum dix combinaisons.\n" );
printf("Pour chaque tentative le programme indique le nombre de couleurs bien placees\n" );
printf("(a l aide de pion noir) et le nombre de couleurs mal placees\n" );
printf("(a l aide de pion blanc). Si le nombre maximal d essais est atteint,\n" );
printf("le programme indique au joueur qu il a perdu et donne la combinaison.\n\n" );
printf("------------------------------------------------------------------------------\n\n" );
printf("Entrez (0) pour revenir au menu principale\n" );
scanf("%d", &choix);
if (choix == 0)
{
menu();
}
else
{
regles(); // OUCH !!! Tu n'as rien trouvé de mieux que de la récursivité pour programmer une boucle ??? Tu n'as pas l'impression que "récursivité" implique quelque part "mécanisme d'empilement des couches successives" ???
}
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void jeu()
{
int essais=1,choix=0, ligne=9,continuer=1,verif=0,a; // "entrer" dans la boucle jeu
char tab[10][8]={
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46}
}; //Pourquoi ces valeurs 42 et 46
char comb[4];
system("cls" ); // Nettoyer la console
// --------------------------------------------------------------------------------------------------------
// Générer une combinaison aléatoire
// J'ai dû rater un truc. On est bien dans la fonction jeu() non ???
void initCode(int *codeSecret)
{
int i;
srand (time(NULL)); //Le srand() est un outil d'initialisation. Comme tel, il ne doit être exécuté qu'une seule fois dans le programme et non à chaque fois que la fonction est appelée.
/* initialiser le code secret avec un choix de 6 couleurs */
for (i = 0; i < 4 ; i++)
{
switch (rand() % 6)
{
case 0: codeSecret[i] = 'r'; break;
case 1: codeSecret[i] = 'b'; break;
case 2: codeSecret[i] = 'v'; break;
case 3: codeSecret[i] = 'j'; break;
case 4: codeSecret[i] = 'o'; break;
case 5: codeSecret[i] = 'm'; break;
}
}
}
// -----------------------------------------------------------------------------------------------------------------------------------------------------
// Jeu du MASTERMIND
while (essais <= 10)
{
int i, j, t, test=0, L=10, verif=0; // L : numéroter la ligne du tableau à partir de 10. ligne -> monter d'une ligne lorsque l'on entre une combinaison
char rep[4]={'\0'}; // char rep[4]=""
int a,b; // variable pour la vérification des combinaisons entrées
system("cls" ); // Nettoyer la console
printf("\nCouleurs disponibles : \n" );
printf("=========================================================\n" );
printf("r=Rouge - v=Vert - b=Bleu - j=Jaune - o=Orange - m=Marron \n" );
printf("=========================================================\n\n" );
// initialisation du tableau
for (i=0;i<=9;i++)
{
printf ("[%3d]-[",L);
for (j=0;j<=7;j++)
{
printf ("(%c)",tab[i][j]);
}
printf (" ]\n" );
L--;
}
//printf("\n comb : %s",comb);
// --------------------------------------------------------------------------------------------------------
// Entrer une combinaison
while (verif != 4)
{
verif=0;
printf("\n\nEssai n# %d: \n",essais);
scanf("%s",&rep); //rep est déjà un pointeur !!!
fflush(stdin);
// vérification de la combinaison entrée
for (a=0;a<=3;a++) // for (a=0; a < 4; a++) => la lisibilité d'un code ce n'est pas seulement mettre des commentaires ; c'est aussi garder une certaine cohérence. Puisque rep est défini comme tab[4] alors autant réutiliser cette valeur "4"
{
if (rep[a]=='r' || rep[a]=='v' || rep[a]=='b' || rep[a]=='j' || rep[a]=='o' || rep[a]=='m' )
{
verif++;
}
else
{
verif--;
}
}
if (verif !=4)
{
printf("\nCombinaison impossible. Recommencez !" );
}
}
// -----------------------------------------------------------------------------------------------------------
// Ajout dans le combinaison dans le tableau
test = strcmp(rep,comb); // on compare les 2 chaine de caractère, test sera négatif si s1 est < à s2, nul si s1 est == à s2 ou positif si s1 est > à s2 Et où a été rempli comb ??? D'ailleurs je ne vois plus trop ce qu'est devenu ce fameux tableau de 4 pointeurs "codeSecret"...?
if (test==0)
{
essais=11; // le joueur a trouvé la combinaison, on quitte la boucle jeu
}
else
{
for (j=0;j<=3;j++) // se déplacer de cellule en cellule
{
for (t=0;t<=3;t++) // permet de se déplacer dans la chaine de caractère comb, afin d'analyser chaque lettre
{
if (j == t) // si j == t, on est à la même position dans la chaine
{
if (rep[j]==comb[t]) // Si le caractère n°x est égale au caractère n°x de la combinaison
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=78; // on affiche N (ascii 78) ????? tab[ligne][j+4]='N' !!!
}
}
else
{
if (rep[j]==comb[t]) // Si le caractère n°x n'est pas égale au caractère n°x, mais qu'il figure dans la combinaison
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=66; // on affiche B (ascii 66) oui oui
}
else // Sinon on affiche un point
{
tab[ligne][j]=rep[j];
tab[ligne][j+4]=46; // on affiche un point (ascii 46) oui oui
}
}
}
}
}
fflush(stdin); // NON !!! fflush() est interdit sur stdin - Si ton buffer clavier a des caractères en trop c'est dû aux scanf() mal employés (ou employés à tort)
essais++;
ligne--;
}
// ---------------------------------------------------------------------------------------
// Jeu terminé
if (essais == 12) // si essais == 12, alors le joueurs a trouvé la combinaison
{
printf("Jeu termine" );
printf("\n\n ****** B R A V O *****\n" );
printf("\n **!!! VOUS AVEZ TROUVE LA COMBINAISON !!!**\n" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
else
{
system("cls" ); // Nettoyer la console
printf("Jeu termine" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
// ---------------------------------------------------------------------------------------
if (choix == 0)
{
menu();
}
else
{
jeu(); //Oui, quand une solution fonctionne alors autant l'employer de partout !!!
}
}

Bon ben j'ai mis en rouge tout ce qui ne va pas dans ton code. Certaines erreurs ne sont pas la cause de ce que ça ne marche pas mais toutes sont préjudiciables à plus ou moins long terme.

Ensuite c'est bien de découper en fonctions mais pourquoi ne faire que la moitié du travail ? Faut essayer de repérer les phases importantes et les attribuer à des fonctions dédiées. Par exemple je verrais bien une fonction dédiée à la phase de comparaison du jeu et de la solution...

Message édité par Sve@r le 07-01-2012 à 10:44:59

fia123

Sve@r,
Merci pour ta réponse très claire et détaillée, seulement depuis la publication de mon message j'ai considérablement changé mon code et maintenant il fonctionne bien. Le voici :

Code :

#include <stdio.h> // Fonctions getchar, printf()
#include <stdlib.h> // Fonctions system()
#include <string.h>
#include <time.h>
#define OK 1
int menu();
void jeu();
void regles();
// ---------------------------------------------------------------------------------------------------------------------------------------------
void initCode(char *codeSecret);
void analyserCombinaison(char *comb, char *rep, char *resultat);
int main ()
{
int continuer = 1, choix = 0;
while (continuer) // Boucle permettant de choisir ce que l'on veut faire dans le programme
{
choix = menu();
switch (choix)
{
case 1:
jeu();
break;
// ----------------------------------
case 2:
regles();
break;
// ----------------------------------
case 3:
// Quitter
continuer = 0;
break;
// ----------------------------------
default:
printf("Choix errone\n" );
}
// ----------------------------------
} // while
return 0;
}
// ------------------------------------------------------------------------------------------------------------------------------------------------
int menu()
{
int choix = 0;
system("cls" ); // Nettoyer la console
printf("======== MENU ========\n\n" );
printf("(1) : Jouer\n" );
printf("(2) : Regles du jeu\n" );
printf("(3) : Quitter\n\n" );
printf("======================\n\n\n" );
printf("Que voulez vous faire ?\n\n" );
scanf("%d", &choix);
return choix;
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void regles()
{
int choix = 0;
system("cls" ); //Nettoyer la console
//Affichage des règles du Mastermind
printf("***********************\n" );
printf("*REGLES DU MASTERMIND:*\n" );
printf("***********************\n" );
printf("\nLe programme choisit une combinaison de 4 pions de couleurs au hasard\n" );
printf("parmi 6 couleurs disponibles.\n" );
printf("Cette combinaison de pions peut contenir plusieurs fois la meme couleur.\n" );
printf("L utilisateur essaie de la deviner en proposant au maximum dix combinaisons.\n" );
printf("Pour chaque tentative le programme indique le nombre de couleurs bien placees\n" );
printf("(a l aide de pion noir) et le nombre de couleurs mal placees\n" );
printf("(a l aide de pion blanc). Si le nombre maximal d essais est atteint,\n" );
printf("le programme indique au joueur qu il a perdu et donne la combinaison.\n\n" );
printf("------------------------------------------------------------------------------\n\n" );
printf("Entrez (0) pour revenir au menu principale\n" );
scanf("%d", &choix);
if (choix == 0)
{
menu();
}
else
{
regles(); // Sve@r, j'avoue ne pas avoir très bien compris ton commentaire pour cette ligne de code .. je ne vois pas le problème.
}
}
void verifierCombi(char *comb, char *rep, char *resultat) //Permet de tester la combinaison entrée par le joueur pour voir si elle est sous la bonne forme.
{
char repMinuscule[4];
int a;
for(a=0;a<4;a++)
{
repMinuscule[a]=tolower(rep[a]); //Permet de transformer une majuscule en minuscule, ce qui permettra d'enter à la fois des minuscules et des majuscules comme combinaison
}
int controle[4] = {0,0,0,0};//sert à vérifier s'il n'y a pas un N à cette position
int j=0;
int i;
for( i = 0;i<4;i++)//Sert à détecter les pions noirs
{
if (repMinuscule[i]==comb[i])
{
resultat[j] = 'N';
j++;
controle[i] = OK;
}
}
for(i = 0;i<4;i++)//Sert à détecter les pions blancs
{
if(controle[i]!=OK)
{
int k;
for(k=0;k<4;k++)
{
if (repMinuscule[k]==comb[i] && controle[i]!=OK)
{
resultat[j]= 'B';
j++;
controle[i] = OK;
}
}
}
}
}
void initCode(char *codeSecret) //Fonction permettant de générer un code de 4 couleurs avec les initiales des 6 couleurs proposées.
{
int i;
srand (time(NULL));
/* initialiser le code secret avec un choix de 6 couleurs */
for (i = 0; i < 4 ; i++)
{
switch (rand() % 6)
{
case 0: codeSecret[i] = 'r'; break;
case 1: codeSecret[i] = 'b'; break;
case 2: codeSecret[i] = 'v'; break;
case 3: codeSecret[i] = 'j'; break;
case 4: codeSecret[i] = 'o'; break;
case 5: codeSecret[i] = 'm'; break;
}
}
}
// ------------------------------------------------------------------------------------------------------------------------------------------
void jeu()
{
int essais=0,choix=0, ligne=9,continuer=1,verif=0,a; // "entrer" dans la boucle jeu
char tab[10][8]={
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46},//Sv@r, 42:* 46:. j'initialise les valeurs de mon tableau avec ces 2 éléments
{42,42,42,42,46,46,46,46},
{42,42,42,42,46,46,46,46}
};
char comb[5];
initCode(comb);
system("cls" ); // Nettoyer la console
// -----------------------------------------------------------------------------------------------------------------------------------------------------
// Jeu du MASTERMIND
char rep[5]={'\0'};
while (essais <= 10)
{
char resultat[4]={46,46,46,46};
int i, j, t, test=0, L=1, verif=0; // L : numéroter la ligne du tableau à partir de 10. ligne -> monter d'une ligne lorsque l'on entre une combinaison
int a,b; // variable pour la vérification des combinaisons entrées
system("cls" ); // Nettoyer la console
printf("\nCouleurs disponibles : \n" );
printf("=========================================================\n" );
printf("r=Rouge - v=Vert - b=Bleu - j=Jaune - o=Orange - m=Marron \n" );
printf("=========================================================\n\n" );
// affichage du tableau
for (i=0;i<=9;i++)
{
printf ("[%3d]-[",L);
for (j=0;j<=7;j++)
{
printf ("(%c)",tab[i][j]);
}
printf (" ]\n" );
L++;
}
//printf("\n Les caracteres saisit sont: %c%c%c%c", rep[0],rep[1],rep[2],rep[3]);
//printf("\n comb : %c%c%c%c",comb[0],comb[1],comb[2],comb[3]);
// --------------------------------------------------------------------------------------------------------
// Entrer une combinaison
while (verif != 4)
{
verif=0;
printf("\n\nEssai n# %d: \n",essais);
scanf("%s",&rep);
fflush(stdin); /*j'ai essayé en utilisant a la place while(fgetc(stdin)!=EOF); mais le problème est qu'alors la combinaison que je rentre ne s'introduit plus dans le tableau .. */
// vérification de la combinaison entrée
for (a=0;a<=3;a++)
{
if (rep[a]=='r' || rep[a]=='v' || rep[a]=='b' || rep[a]=='j' || rep[a]=='o' || rep[a]=='m' || rep[a]=='R' || rep[a]=='V' || rep[a]=='B' || rep[a]=='J' || rep[a]=='O' || rep[a]=='M' )
{
verif++;
}
else
{
verif--;
}
}
if (verif !=4)
{
printf("\nCombinaison impossible. Recommencez !" );
}
}
verifierCombi(comb,rep,resultat);
tab[essais][0]=rep[0];
tab[essais][1]=rep[1];
tab[essais][2]=rep[2];
tab[essais][3]=rep[3];
tab[essais][4]=resultat[0];
tab[essais][5]=resultat[1];
tab[essais][6]=resultat[2];
tab[essais][7]=resultat[3];
if (resultat[0]=='N' && resultat[1]=='N' && resultat[2]=='N' && resultat[3]=='N')// Si 4 'N' alors le joueur a trouvé le code secret
{
essais=99;
}
else{
essais++;
}
}
// ---------------------------------------------------------------------------------------
// Jeu terminé
if (essais == 99) // si essais == 99, alors le joueurs a trouvé la combinaison
{
printf("Jeu termine" );
printf("\n\n ****** B R A V O *****\n" );
printf("\n **!!! VOUS AVEZ TROUVE LA COMBINAISON !!!**\n" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
else
{
system("cls" ); // Nettoyer la console
printf("Jeu termine" );
printf("\n\n*********************************************************\n" );
printf("---- Entrez (0) pour revenir au menu principal ou (1) pour rejouer : " );
scanf ("%d",&choix);
}
// ---------------------------------------------------------------------------------------
if (choix == 0)
{
menu();
}
else
{
jeu();
}
}

Si tu pouvais me donner ton avis et me dire si il est préférable que je change certaines partie de mon code, ce serait vraiment super (je tiens à re-préciser qu'il fonctionne correctement mais il n'est certainement pas parfait).

Message cité 1 fois
Message édité par fia123 le 08-01-2012 à 01:31:38

Sve@r

fia123 a écrit :

Sve@r,
Merci pour ta réponse très claire et détaillée, seulement depuis la publication de mon message j'ai considérablement changé mon code et maintenant il fonctionne bien. Si tu pouvais me donner ton avis et me dire si il est préférable que je change certaines partie de mon code, ce serait vraiment super (je tiens à re-préciser qu'il fonctionne correctement mais il n'est certainement pas parfait).

Il fonctionne oui. Mais "correctement"...

1) Concernant ton commentaire à propos de ma remarque: ta boucle principale affiche un menu. De ce menu on peut afficher les règles. Et les règles réaffichent les règles. Et les règles réaffichent les règles. Et etc etc. Pourtant tu as appris le principe d'imbrication des fonctions. Si une fonction A appelle une fonction B qui appelle une fonction C alors ensuite la C doit remonter vers la B qui remonte vers la A. Donc tu dis "tu ne vois pas ..." mais tu n'as rien fait pour "voir".
Rajoute printf("Fin de fonction\n" ) à la dernière ligne de la fonction "regles()" puis lance ton programme. Tape "2" pour avoir les règles puis, de là, tape encore cinq ou six fois "2" avant de taper "0". Là ca réaffichera le menu (bien inutilement d'ailleurs puisque ensuite la fonction se terminera et la boucle du main() réaffichera le menu) mais j'espère que là tu "verras" ce qui ne va pas...

2) Concernant les valeurs 42 et 46 ta réponse est vraiment crétine. Je te demande pourquoi tu initialises avec 42 et 46 et tu me réponds "j'initialise avec 42 et 46" !!!

3) pour vérifier que la réponse est acceptable c'est quoi tout ce micmac avec verif++ et verif-- ? Considère qu'elle l'est par défaut puis vérifie avec une boucle s'il y a un élément qui ne va pas. A la fin de la boucle, si ton verif est resté dans le même état qu'au début de la boucle ça veut dire que la réponse est acceptable.

4) je ne vois pas trop à quoi sert le tableau "controle". Si tu veux vérifier qu'il n'y a pas de "N" dans la case "resultat[x]" ben va regarder la valeur de cette case !!!

5) t'as rien fait de mes remarques dans les lignes 100, 144, 171 et surtout 204 de mon premier post

6) concernant le fflush(stdin) tu le supprimes. Mais chaque fois que tu utilises scanf() pour faire saisir un nombre (%d ou %f), un caractère (%x) alors tu rajoutes un fgetc(stdin) juste après. Mais si tu fais saisir une chaine (%s) alors surtout pas. Ce fgetc() sert à supprimer le '\n' représentant l'appui sur la touche <entrée> qui n'est pas vidé de stdin dans les 3 cas précités. Car quoi qu'il en soit tu appuies toujours sur cette touche à chaque saisie mais si scanf() attend un nombre ou un caractère alors cette touche appuyée reste dans stdin. Et si scanf() attend une chaine alors cette touche est intégrée à la chaine saisie.

Et si tu veux remplacer scanf (qui signifie "saisie formatée" et que ce que tape un humain est tout sauf formaté) par un outil de saisie plus robuste, alors essaye de tester et de t'inspirer de ce petit bout de code

Code :

#include <stdio.h>
int main()
{
int x, y;
char chaine[50];
while(1)
{
printf("Entrez deux nombres séparés par un (ou plusieurs) espace\n" );
fgets(chaine, 50, stdin);
if (sscanf(chaine, "%d %d", &x, &y) == 2) break;
printf("Erreur sur l'un des deux nombres\n" );
}
printf("Vous avez entré %d et %d\n", x, y);
}

Sinon pour le reste ben c'est encore brouillon mais ça peut attendre...

Message édité par Sve@r le 08-01-2012 à 03:15:40

fia123

Sve@r, dans la table ascii, 42 correspond à "*" et 46 à "." je l'ai mis dans mon commentaire si tu regardes bien, j'initialise mon tableau avec des "*" et des ".".
Les "*" sont remplacés par les combinaisons que rentre le joueur, et les "." sont remplacés par les pions noirs ou blancs. Donc ma réponse n'étais pas "crétine". Pour ce qui est du reste, je te remercie toujours autant pour tes réponses détaillées, je vais modifier mon code ce soir. Merci.

Message cité 1 fois

Sve@r

fia123 a écrit :

Mais si ta réponse était crétine parce que tu ne sais pas lire entre les lignes. Je cherchais par cette motivation faite à ton amour propre que tu déduises de mes autres commentaires ce que j'aurais aimé que tu comprennes toute seule (me démontrant ainsi que tu savais aussi te servir de ta tête). Je sais bien que 42 et 46 représentent les caractères '*' et '.' mais pourquoi n'utilises-tu justement pas cette notation standard '*' et '.' ? Je t'avais pourtant signalé la chose en ligne 244 dans mon premier post !!!

En plus aujourd'hui tu travailles en ascii mais que se passera-t-il si demain tu travailles en utf-8 ? Tu crois que 42 et 46 auront la même signification ?

FORUM HardWare.fr

Programmation

mastermind en C

Sujets relatifs
[C#/C] Dllimport et pointeur de fonction	[C]Gérer le env -i ?
[Resolu] probleme en C : comparaison de chaine de caractere	DE C++ EN C
[C]Comment récupérer une saisie clavier ?	[C Multitâche] Récupérer un string sur stdin
Les fichiers en C	[C#] Cacher des méthodes d'une interface
[C][OSX][LibSDL] Compilation OK, mais problème d'exécution...	Mastermind en algo puis en C
Plus de sujets relatifs à : mastermind en C

Page générée en 0.344 secondes