Discussion :

[johnny3]

Hello,

j'ai programmé le jeu du devin mais je me heurte à deux problèmes:

mon programme saute l'instruction getchar et exécute donc 2 fois:

Code :

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

1
2
3
printf("L'ordinateur propose %d.\n", nbOrdi);
printf("Ce nombre est-il trop petit, trop grand ou le même que le vôtre?\n");
printf("(p ou P pour \"trop petit\", g ou G pour \"trop grand\", t ou T pour le même)\n");

Mais la seconde fois, pas de problème, réponse est lue (et ce, à chaque fois)

J'ai aussi un problème avec compteur qui affiche une valeur complètement farfelue (du style 21 alors qu'il n'y a eu que 5 essais, par exemple)

Je vous mets ici mon code (par contre, le programme se déroule jusqu'au bout)

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
#include <stdio.h>
#include <stdlib.h>
#define N 1000
#define MIN 0
#define MAX 1000
int tab [N];
int nb_pense;
 
void creationTableau();
int nbPense();
void jeu();
 
 
int main(void)
{
    creationTableau();
    jeu();
    return 0;
}
 
void creationTableau(){
    int i;
    for (i = 1; i <= 1000; i++){
        tab[i] = i;
    }
}
 
int nbPense(){
    printf("A quel nombre pensez-vous?\n");
    scanf ("%d", &nb_pense);
    return nb_pense;
}
 
void jeu()
{
    nb_pense = nbPense();
    int m;
    int debut = MIN;
    int fin = MAX;
    int nbOrdi;
    char reponse = '!';
    int compteur = 1;
    do
    {
        m = (debut + fin) / 2;
        nbOrdi = tab[m];
        printf("L'ordinateur propose %d.\n", nbOrdi);
        printf("Ce nombre est-il trop petit, trop grand ou le même que le vôtre?\n");
        printf("(p ou P pour \"trop petit\", g ou G pour \"trop grand\", t ou T pour le même)\n");
        reponse = getchar();
        if (reponse != 't' || reponse != 'T') compteur ++;
        if (reponse == 'p' || reponse == 'P')
        {
            debut = m + 1;
        } else if (reponse == 'g' || reponse == 'G')
        {
            fin = m - 1;
        } else if (reponse == 't' || reponse == 'T')
        {
            printf("C'était le bon nombre!\n");
            printf("Le nombre a été trouvé en %d essais.\n", compteur);
            break;
        }
    } while (debut <= fin && (reponse != 't' || reponse != 'T'));
}

Voyez-vous d'où viennent ces erreurs?

En vous remerciant par avance,
Johnny

[m0ul3sh0t]

getchar va read caractere par caractere. Comme a chaque fois tu auras ton caractere + \n ...
sinon tu peux faire un read, et regarder uniquement le premier caractere


pour les valeurs du compteur, c'est normal vu que tu fais a chaque fois deux tour de boucles, il sera incremente deux fois

[johnny3]

Ca a fonctionné pour la première proposition, mais pour les autres, ça me remet tout en double.

J'ai tapé read (0, &reponse, 1)

Johnny

[m0ul3sh0t]

en mettant read (0, &reponse, 1), le probleme est exactement le meme parce que faire un read de 1 revient exactement a un getchar

si tu pars du principe que ton utilisateur ne va pas marquer n'importe quoi, tu peux faire qqch comme ca:
- declarer un buffer de lecture d'une taille suffisante pour lire le caractere + le \n
- faire une lecture de 2 caracteres en 2 caracteres

[johnny3]

pourrais-tu me donner un exemple, s'il te plait? Je ne pratique le C que depuis 2-3 mois et je ne vois pas comment faire.

Merci!

Johnny

[diogene]

C'est effectivement un problème courant parce qu'il peut rester en attente un caractère '\n' non lu dû à une lecture précédente. Si on fait alors un scanf %c ou %s ou un getchar, on récupère ce '\n' et il constitue la lecture.
Dans de tels cas, il faut prendre soin de vider la ligne en attente.

Plusieurs fonctions pouvant faire cette opération ont été publiées sur ce forum (cherche fclean() mais ne pas utiliser fflush()).

On peut aussi, à chaque lecture, vider la fin de la ligne qui ne nous interesse pas dans un scanf. C'est un peu ésotérique mais simple : il suffit de compléter le format utile par %*[^'\n']%*c.
Ansi, ta lecture dans la fonction nbPense devient :

Code :

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

    scanf ("%d%*[^'\n']%*c", &nb_pense);

et le getchar est remplacé par :

Code :

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

    scanf("%c%*[^'\n']%*c",&reponse);

Avec ce code, tu peut taper tazerty , il prendra le t et ignorera la suite de la ligne y compris le caractère de fin de ligne et tout sera propre pour l'entrée suivante.

L'interprétation est la suivante :
%*[^'\n'] lit une chaîne de caractères (format %[]) tant que les caractères sont différents (à cause du ^) de '\n' et ignorer (à cause de *) la chaine obtenue
%*c lit un caractère (c'est le '\n' sur lequel on est arrivé après le format précédent) et l'ignorer (à cause de *)