Discussion :

[maBs12]

Bonjour à tous,

Je débute en C et j'ai un petit problème dans mon code.

C'est un jeu ou on génère un nombre aléatoire. Le joueur est invité à entrer un nombre, suivant le nombre on lui dit si c'est + ou -.
Mon problème ce trouve dans ma boucle, la décrémentation se passe correctement mais la ou il faudrait sortir de la boucle si le nombre d'essai arrive à 0 le compteur continue en partant dans les négatifs.
Je pense que c'est une simple erreur mais je ne le vois pas.

Merci d'avance pour votre aide

code :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
int main()
{
    srand(time(NULL));
    const int MAX = 100, MIN = 1;
    int nombreMystere = (rand() % (MAX - MIN + 1)) + MIN;
    int nombreChoisi = 0;
    int rejouer = 1;
 
    // Boucle pour rejouer
    while (rejouer == 1)
    {
    printf ("***PLUS ou MOINS***\n");
    printf ("-------------------\n");
    printf ("Trouver le nombre mystere(uniquement des nombres entiers compris entre 1 et 100)\n");
    printf ("Vous avez 10 essais\n");
 
        // Boucle tant que pour ne pas dépasser 10 et tant que le nombre choisi n'est pas égale au nombre mystère
        int essai = 0;
        printf("Votre nombre :\n");
        scanf("%d",&nombreChoisi);
 
        while (essai < 0 || nombreChoisi != nombreMystere)
        {
            if (nombreChoisi > nombreMystere)
            {
                printf("C'est moins\n");
                essai--;
                printf("Il vous reste %d\n",essai);
                printf("Votre nombre :\n");
                scanf("%d",&nombreChoisi);
            }
            else if (nombreChoisi < nombreMystere)
            {
                printf("c'est plus\n");
                essai--;
                printf("Il vous reste %d\n",essai);
                printf("Votre nombre :\n");
                scanf("%d",&nombreChoisi);
            }
            else
            {
                printf("BRAVO !!!\n");
            }
 
        }
 
    printf("******************\n");
    printf("Voulez-vous rejouer\n");
    printf("1. Oui\n");
    printf("2. Non\n");
    scanf("%d",&rejouer);
        if (rejouer == 1)
        {
            int rejouer = 1;
            int essai = 10;
        }
        else
        {
            printf("Merci d'avoir joue\n");
            int rejouer = 0;
        }
    }
    return 0;
}

[Kannagi]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
        if (rejouer == 1)
        {
            int rejouer = 1;
            int essai = 10;
        }
        else
        {
            printf("Merci d'avoir joue\n");
            int rejouer = 0;
        }

Je suis étonné que cela compile , vu que tu as déjà déclarer ces variables , ou alors il considérer que c'est des nouvelles variables mais du coup leur porté est limité aux {}.
Mais dans tout les cas il faut vraiment enlever ces int...

[maBs12]

D'accord, mais même en retirant cette partie du code le problème reste le même

Code :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
int main()
{
    srand(time(NULL));
    const int MAX = 100, MIN = 1;
    int nombreMystere = (rand() % (MAX - MIN + 1)) + MIN;
    int nombreChoisi = 0;
 
 
    printf ("***PLUS ou MOINS***\n");
    printf ("-------------------\n");
    printf ("Trouver le nombre mystere(uniquement des nombres entiers compris entre 1 et 100)\n");
    printf ("Vous avez 10 essais\n");
 
        // Boucle tant que pour ne pas dépasser 10 et tant que le nombre choisi n'est pas égale au nombre mystère
        int essai = 10;
        printf("Votre nombre :\n");
        scanf("%d",&nombreChoisi);
 
        while (essai < 0 || nombreChoisi != nombreMystere)
        {
            if (nombreChoisi > nombreMystere)
            {
                printf("C'est moins\n");
                essai--;
                printf("Il vous reste %d\n",essai);
                printf("Votre nombre :\n");
                scanf("%d",&nombreChoisi);
            }
            else if (nombreChoisi < nombreMystere)
            {
                printf("c'est plus\n");
                essai--;
                printf("Il vous reste %d\n",essai);
                printf("Votre nombre :\n");
                scanf("%d",&nombreChoisi);
            }
            else
            {
                printf("BRAVO !!!\n");
            }
 
        }
    return 0;
}

Merci

[sambia39]

Bonsoir,
C'est normal que le programme se comporte de la sorte puisque dès le départ, la variable d'essai est initialisée avec la valeur zéro. Puis, dans la boucle "tant que" celle-ci est décrémentée, il serait préférable de fournir le nombre d'essais avant de pouvoir commencer à jouer.
Plus bas dans votre code, il faudrait restaurer le nombre d'essais dans la variable "essai", après que le joueur ait choisi de rejouer une partie. Il faudrait également ajouter l'évaluation de la variable essai, car c'est t'elle qui arrête le jeu et affiche échec de la partie quand celui-ci arrive a zéro. ( game over); et également tenir compte des remarques de @Kannagi

À bientôt

[maBs12]

Merci pour ta réponse Sambia,

J'ai rectifier le 1er code en fonction de la réponse de Kannagi, dans la réponse j'ai bien initialisé la variable essai à 10 directement. Regarde p-e le code que j'ai posté dans ma réponse qui est moins développé mais pour lequel le problème se pose toujours.

Que veux-tu dire par cette phrase ? "il serait préférable de fournir le nombre d'essais avant de pouvoir commencer à jouer"

et également par celle ci :

"Il faudrait également ajouter l'évaluation de la variable essaie" ?

Merci d'avance.

[sambia39]

Que veux-tu dire par cette phrase ? "il serait préférable de fournir le nombre d'essais avant de pouvoir commencer à jouer"

Exemple unsigned short essai = 5; (Chose que vous avez rectifiée dans votre précédent code en mettant la valeur 10)

"Il faudrait également ajouter l'évaluation de la variable essaie" ?

Exemple:

Code C :

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

1
2
3
4
5
6
7
 
 /*
     * Si le nombre d'essais est atteint
     */
     if( 5 == essai )
        (void)puts("game over");
    .............

À bientôt

[Sve@r]

Bonjour

mais la ou il faudrait sortir de la boucle si le nombre d'essai arrive à 0 le compteur continue en partant dans les négatifs.

Ben oui, tu pars avec "essai" à 0, ta boucle demande while (essai < 0 || pas trouvé) (rappel, un while c'est un "tant que" et non un "jusqu'à") donc comme le cas "pas trouvé" est vrai la boucle démarre puis tu demandes essai--. Donc c'est parti pour ne plus d'arrêter (ou plus exactement tu t'arrêteras quand "essai" atteindra -2147483648 et que le décrément de plus le passera à 2147483647).

J'ai rectifier le 1er code en fonction de la réponse de Kannagi, dans la réponse j'ai bien initialisé la variable essai à 10 directement. Regarde p-e le code que j'ai posté dans ma réponse qui est moins développé mais pour lequel le problème se pose toujours.

Quelle que soit la valeur de départ de "essai", le souci se pose parce que demandes "tant que essai est négatif ou nombre pas trouvé". Donc ta boucle passe par deux phases

• la première partie où "essai" est positif mais le nombre n'est pas trouvé => condition vraie
• la seconde partie où "essai" est négatif => condition vraie que le nombre soit trouvé ou pas

ou alors il considérer que c'est des nouvelles variables mais du coup leur porté est limité aux {}.

Exactement ça. Tout nouveau bloc (même intérieur) a le droit de déclarer des variables de son choix qui prendront le pas sur les variables éventuellement déjà existantes du bloc supérieur mais toute variable située dans un bloc est limitée à son bloc.

[maBs12]

Bonjour Sve@r,

Merci pour ta réponse.

Si je comprends bien je dois simplement changer la condition dans ma boucle au niveau d'essai comme ci-dessous :

Code :

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

while (nombreChoisi != nombreMystere || essai > 0)

Si je lis bien la, ca donne "tant que nombre choisi est différent du nombre mystère ou essai est plus grand que 0"
Je viens d'essayer à nouveau et le problème reste le même les essais passent en négatif, alors que normalement dés que essai arrive à 0 une des confitions n'est plus vrai donc on devrait sortir de la boucle non ?

[Sve@r]

Si je lis bien la, ca donne "tant que nombre choisi est différent du nombre mystère ou essai est plus grand que 0"

Tout à fait exact.

Je viens d'essayer à nouveau et le problème reste le même les essais passent en négatif, alors que normalement dés que essai arrive à 0 une des confitions n'est plus vrai donc on devrait sortir de la boucle non ?

Hé non, là tu réfléchis à l'envers. On ne demande pas à sortir sur l'expression mais à rester sur l'expression !!! Et le "ou" valide l'expression si l'une des deux conditions est vraie. Donc essai passe en négatif ok mais comme le nombre mystère n'est pas trouvé, alors la condition reste vraie et la boucle continue.

[maBs12]

D'accord, je dois simplement changer mon "Ou" par un "Et" et ca fonctionne.

Merci

[Sve@r]

D'accord, je dois simplement changer mon "Ou" par un "Et" et ca fonctionne.

Exactement. Un mathématicien nommé "De Morgan" a établi les lois qui permettent d'inverser une condition (dites lois de "De Morgan"). Elles ne sont pas très compliquées et peuvent d'ailleurs se déduire des tables de vérité de base (OU, ET et NON).
Elles sont:

• NON(A OU B) = NON(A) ET NON(B)
• NON(A ET B) = NON(A) OU NON(B)

Elles peuvent aussi se retrouver grâce à la réflexion. Si par exemple je veux un nombre compris entre 10 et 20 (donc >= 10 ET <= 20), alors le contraire c'est un nombre qui n'est pas compris entre 10 et 20 (donc < 10 OU > 20). Et si je veux un nombre divisible par 3 OU par 5 alors le contraire sera un nombre non divisible par 3 ET non divisible par 5.
Donc adapté à ton problème: si tu veux sortir sur "A OU B", alors il te faut demander à rester sur "non(A OU B)" soit "non(A) ET non(B)".