Discussion :

[trans13]

Bonjour,

Le probleme est le suivant :

Je lance pendu.exe je rentre les lettres pour marron, on me demande une

nouvelle Partie je met oui la partie redemarre, jusque la pas de probleme.

Je rerentre une lettre pour trouver le mot mystere et celle ci n'est pas

prise en compte (exemple si je tape la lettre m pour marron mon coups va

diminuer comme si m ne se trouver pas dans marron)

Et si je rerentre m une seconde fois la mon coups ne diminue pas et la lettre m

s'affiche dans le mot mystere crypte.

Voici mon code :

Main.c :

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
#include <stdio.h>
#include <stdlib.h>
#include "jeux.h"
 
int main()
{
    int nouvellePartie = 0, a = 1;
 
    do
    {
 
        char motMystere[] = "MARRON", motMystereCrypte[7] = {0};
        char lettreEntre = 0;
        long coups = 10;
        int sortieBoucle = 1;
 
        cryptageMotMystere (motMystere, motMystereCrypte);
 
        printf("\nBienvenue dans le jeux du pendu !\n\n\n");
 
        do
        {
 
            printf ("\nIl vous reste %ld coups a jouer\n", coups);
            printf ("Quel est le mot secret ? %s\n", motMystereCrypte);
            printf ("Proposez une lettre : ");
 
            lettreEntre = lireCaractere();
 
            verificationLettreEntre (motMystere, motMystereCrypte, lettreEntre, &coups);
            gagne (motMystere, motMystereCrypte, coups, &nouvellePartie, &sortieBoucle);
 
        }while (sortieBoucle);
 
    }while (nouvellePartie);
 
    return 0;
}

Jeux.c :

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
char lireCaractere()
{
    char caractere = 0;
 
    caractere = getchar(); // On lit le premier caractère
    caractere = toupper(caractere); // On met la lettre en majuscule si elle ne l'est pas déjà
 
    // On lit les autres caractères mémorisés un à un jusqu'à l'\n (pour les effacer)
    while (getchar() != '\n') ;
 
    return caractere; // On retourne le premier caractère qu'on a lu
}
 
void cryptageMotMystere (char motMystere[], char motMystereCrypte[])
{
    long nombreDeLettreMotMystere = 0, i;
 
    nombreDeLettreMotMystere = strlen(motMystere);
 
    for (i = 0; i < nombreDeLettreMotMystere; i++)
    {
        motMystereCrypte[i] = '*';
    }
}
 
void verificationLettreEntre (char motMystere[], char motMystereCrypte[], char lettreEntre, long *coups)
{
    long nombreDeLettreMotMystere = 0, i;
    char copieMotMystereCrypte[7] = {0};
 
    strcpy (copieMotMystereCrypte, motMystereCrypte);
 
    nombreDeLettreMotMystere = strlen(motMystere);
 
    for (i = 0; i < nombreDeLettreMotMystere; i++)
    {
        if (lettreEntre == motMystere[i])
        {
            motMystereCrypte[i] = lettreEntre;
        }
    }
 
    if (strcmp (copieMotMystereCrypte, motMystereCrypte) == 0)
    {
        *coups-= 1;
    }
}
 
void gagne (char motMystere[], char motMystereCrypte[], long coups, int *nouvellePartie, int *sortieBoucle)
{
 
    if (coups == 0)
    {
        *sortieBoucle = 0;
 
        printf ("\nPerdu, le mot mystere etait : %s\n\n", motMystere);
        printf ("Une nouvellePartie ?\n\n");
        printf ("1. Oui\n");
        printf ("2. Non\n\n");
 
        scanf ("%ld", nouvellePartie);
 
    }
    else if (strcmp (motMystere, motMystereCrypte) == 0)
    {
        *sortieBoucle = 0;
 
        printf ("\nGagne ! Le mot mystere etait bien : %s\n\n", motMystere);
        printf ("Une nouvellePartie ?\n\n");
        printf ("1. Oui\n");
        printf ("2. Non\n\n");
 
        scanf ("%ld", nouvellePartie);
 
    }
 
    if (*nouvellePartie != 1)
    {
        *nouvellePartie = 0;
    }
}

Jeux.h :

Code :

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

1
2
3
4
void cryptageMotMystere (char motMystere[], char motMystereCrypte[]);
char lireCaractere();
void verificationLettreEntre (char motMystere[], char motMystereCrypte[], char lettreEntre, long *coups);
void gagne (char motMystere[], char motMystereCrypte[], long coups, int *nouvellePartie, int *sortieBoucle);

Cordialement,

[homeostasie]

T'as omis de purger le buffer clavier dans la fonction gagne, suite à ton scanf. D'ailleurs il est préférable d'utiliser fgets.

A la place de cela:

Code :

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

1
2
 
scanf ("%ld", nouvellePartie)

je ferais ceci:

Code :

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

1
2
3
4
5
6
7
8
 
char newGame[2] = {0};
int c;
...        
fgets(newGame,2,stdin);
*nouvellePartie = strtol(newGame,NULL,10);
while ((c = getchar()) != '\n' && c != EOF);
...

[Emmanuel Delahaye]

je ferais ceci:

Code :

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

1
2
3
4
5
6
7
8
 
char newGame[2] = {0};
int c;
...        
fgets(newGame,2,stdin);
*nouvellePartie = strtol(newGame,NULL,10);
while ((c = getchar()) != '\n' && c != EOF);
...

Ce n'est pas comme ça qu'on purge après un fgets().

Code :

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

1
2
3
4
5
6
7
 
rechercher '\n'
si on l'a trouvé, 
 le supprimmer
sinon
 purger les caractères non lus
fin si

J'ai pour habitude de faire une fonction

Code :

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

void clean (char *s);

qui fait ce travail après chaque fgets() systématiquement.
On récupère une chaine propre et un tuyau débouché...

[homeostasie]

Ce n'est pas comme ça qu'on purge après un fgets().

Oui c'est vrai, il faut d'abord détecter le '\n' pour savoir s'il ne reste pas des données à réceptionner (selon le choix du programmeur) dû au fait que le buffer de réception peut avoir une dimension inférieure à la chaîne saisie.

Voici la fonction à utiliser après un fgets, présente dans plusieurs conversations:

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
 
static void PurgeFile (char *s, FILE *fp)
{
   /* search ... */
   char *p = strchr (s, '\n');
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      /* purge */
      int c;
      while ((c = fgetc(fp)) != '\n' && c != EOF)
      {
      }
   }
}

[trans13]

Re,

J'ai 14 ans, je debute j'ai suivie les cours d'un autre site qui n'utiliser que des

scanf, si j'ai bien compris il faut utiliser fgetc car scanf

La fonction scanf() est une fonction destinée, comme le f dans son nom l'indique, aux saisies formatées et s'avère ainsi très peu adaptée aux saisies humaines qui peuvent présentées des erreurs de format, ces erreurs de formats pouvant entrainerr des débordements de tableau, des boucles infinies, des perturbations sur les saisies suivantes, etc.

Mais je ne comprends pas cette histoire de vider le buffer, si l'on reecrit par

dessus sa ne fonctionne pas?

Et sinon pour utiliser fgetc c'est ce code si pour moi?

Code :

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

fgets(nouvellePartie, sizeof nouvellePartie, stdin);

Cordialement,

[Emmanuel Delahaye]

scanf, si j'ai bien compris il faut utiliser fgetc

fgets(), sinon, oui c'est le bon code si le premier paramètre est bien un tableau de char. Ensuite, il faut nettoyer comme indiqué ci-dessus...

Mais je ne comprends pas cette histoire de vider le buffer, si l'on reecrit par

dessus sa ne fonctionne pas?

Qui est-ce qui ecrit 'par-dessus' un flux ?

[homeostasie]

Et sinon pour utiliser fgetc c'est ce code si pour moi?

C'est celui pour fgets que tu as fait.

Pour fgetc, ca serait ainsi (fgetc me semble pas mal pour le pendu):

Code :

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

1
2
3
 
int x = fgetc(stdin);
printf ("x = %c\n", x);

Par contre, faudra toujours purger stdin. Sinon il y aura risque au prochain passage que l'exécution ne soit pas suspendue et une lecture du caractère suivant le plus ancien dans le flux.
En effet, fgetc retourne le caractère saisi le plus ancien après validation de la saisie ('\n').

scanf, si j'ai bien compris il faut utiliser fgetc car scanf
Citation:
La fonction scanf() est une fonction destinée, comme le f dans son nom l'indique, aux saisies formatées et s'avère ainsi très peu adaptée aux saisies humaines qui peuvent présentées des erreurs de format, ces erreurs de formats pouvant entrainerr des débordements de tableau, des boucles infinies, des perturbations sur les saisies suivantes, etc.

Il est toujours possible d'utiliser scanf mais il faut bien comprendre ses particularités pour l'utiliser convenablement.
Voir ce lien:
http://xrenault.developpez.com/tutoriels/c/scanf/