Comportement boucle WHILE très étrange

**czar1983** · 14/06/2013, 23h44

Salut tout le monde voila j'ai déjà posté mon sujet sur un autre site mais je n'ai pas encore eu de réponse ( enfin si mais pers n'a de réponse a mon problème )

voila je fait un TP du pendu qui est un des exercices proposer sur ce site et mon while ne boucle plus que sur la variable nbEssaie .

j 'utilise une variable que j'ai appelé "boolean" qui me sert de boolean afin de décrémenter ma variable "nbEssai" si la personne n'a pas trouver le bon caractère , sauf que dés que j utilise cette variable ma seconde condition qui vérifie si les chaines de caractères sont égal ne fonctionne plus .
la boucle devrai normalement stopper quand le nombre d’essai arrive a 0 ou que les chaines de caractères dans les deux tableaux sont identiques

pour résume mon while ne fonctionne que si le nombre d’essai arriver a 0 et continu de boucler même si les chaines de caractères sont identiques .

MAIS WHILE fonctionne dés que je retire cette variable "boolean" .

Voila mon code Merci pour votre aide

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void MyFlush(void);
int main()
{
    int i ;
    int nbEssai = 10;
    int longueurchaine = 10;
    int ChaineIdentique = 1;
    int boolean = 0;
    const char motSecret[10] = "elephant";
    char motSecretcopy[10];
    char lettre;
    char chaine1[longueurchaine] , chaine2[longueurchaine];
 
    /*******************************/
    /** longueurchaine est égal a la taille de la chaine "motSecret" .*/
 
    longueurchaine = strlen(motSecret);
 
    /*******************************/
    /** création d'un tableau d'étoile égale a la longueur de la chaine motSecret .*/
 
    for (i=0; i<longueurchaine; i++)
    {
        motSecretcopy[i]='*';
    }
 
 
    /*******************************/
    printf("\n");
    printf("Bienvenue dans le Pendu\n");
 
 
 
    /*******************************/
    /** tant que le nombre d’essai est inférieur à 0 ou que la chaine de caractère est différente on boucle */
 
    while (nbEssai > 0 && ChaineIdentique!=0  )
 
    {
        printf("\nIl vous reste %d essai \n\n",nbEssai);
        printf("\nEntrer une lettre ! ");
        scanf ("%c",&lettre);
        MyFlush();
 
        boolean=0;
 
        /** boucle qui parcours le tableau */
 
        for (i=0; i<longueurchaine; i++ )
 
        {
            /** Condition qui vérifie si la lettre entrer est égal au tableau a la position "i"
                si oui je copie la lettre entrer dans un autre tableau et je met la variable
                "boolean a a 1 afin de ne pas décrémenter la variable "essaie"   */
 
            if (lettre == motSecret[i])
 
            {
                motSecretcopy[i]=lettre;
                boolean=1;
 
            }
 
        }
        /*******************************/
        /** Boucle qui affiche le tableau */
 
        for (i=0; i<longueurchaine; i++)
        {
            printf("%c",motSecretcopy[i]);
        }
 
        /** La variable " ChaineIdentique " contient la fonction strcmp qui permet de verifier si deux chaine de caractére sont identiques .*/
 
        ChaineIdentique =  strcmp(motSecret, motSecretcopy) ;
 
        /** Condition qui verifie si ma variable " boolean " est égal a 0 si oui la variable " nbEssaie " est décrémenter de 1 */
 
        if ( boolean == 0)
        {
            nbEssai--;
        }
    }
 
 
    /*******************************/
    /** Affichage du tableau  a la fin du grace a une boucle */
 
    printf("\n\n");
    printf("\n");
    for (i=0; i<longueurchaine; i++)
    {
        printf("%c",motSecretcopy[i]);
    }
 
 
    return 0;
}
 
/*******************************/
/** Fonction qui permet de vider le scanf avant de le réutiliser dans une boucle afin d'éviter qui garde en mémoire un caractère */
 
void MyFlush(void)
{
    scanf("%*[^\n]");
    getchar();
}

**b-med** · 15/06/2013, 02h55

Salut, le problème j’espère dans strcmp, j'ai changé la méthode de comparaison, et quelques choses dans le code, tu vas voir le changement.
le code marche bien maintenant, je vais te donner deux solutions qui marchent bien, j'ai déjà essayé

première:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>
void MyFlush(void);
int main()
{
    int i ;
    int nbEssai = 10;
    int longueurchaine = 10;
    int ChaineIdentique = 0;//s'éatit 1, maintenant 0
    int boolean = 0;
    const char motSecret[10] = "pinom";
    char motSecretcopy[10] ;
    char lettre;
    char chaine1[longueurchaine] , chaine2[longueurchaine];
 
    /*******************************/
    /** longueurchaine est égal a la taille de la chaine "motSecret" .*/
 
    longueurchaine = strlen(motSecret);
 
    /*******************************/
    /** création d'un tableau d'étoile égale a la longueur de la chaine motSecret .*/
 
    for (i=0; i<longueurchaine; i++)
    {
        motSecretcopy[i]='*';
    }
 
 
    /*******************************/
    printf("\n");
    printf("Bienvenue dans le Pendu\n");
 
 
 
    /*******************************/
    /** tant que le nombre d’essai est inférieur à 0 ou que la chaine de caractère est différente on boucle */
 
    while ( nbEssai > 0 && ChaineIdentique != 1)//l'initialisation de ChaineIdentique est chnagé en haut
 
    {
        printf("\nIl vous reste %d essai \n\n",nbEssai);
        printf("\nEntrer une lettre ! ");
        scanf ("%c",&lettre);
        MyFlush();
        printf("\n\n");
        boolean=0;
 
        /** boucle qui parcours le tableau */
 
        for (i=0; i<longueurchaine; i++ )
 
        {
            /** Condition qui vérifie si la lettre entrer est égal au tableau a la position "i"
                si oui je copie la lettre entrer dans un autre tableau et je met la variable
                "boolean a a 1 afin de ne pas décrémenter la variable "essaie"   */
 
            if (lettre == motSecret[i])
 
            {
                motSecretcopy[i]=lettre;
                boolean=1;
 
            }
 
        }
 
        /*******************************/
        /** Boucle qui affiche le tableau */
 
        for (i=0; i<longueurchaine; i++)
        {
            printf("%c",motSecretcopy[i]);
        }
 
        /** La variable " ChaineIdentique " contient la fonction strcmp qui permet de verifier si deux chaine de caractére sont identiques .*/
        //ChaineIdentique =  strcmp(motSecret, motSecretcopy) ;
 
 
        //voila ce que j'ai ajouté
          ChaineIdentique = 1 ;
        for(int j = 0 ; j < longueurchaine ; j++)
        {
                if( motSecret[j] != motSecretcopy[j] )
                {
                 ChaineIdentique = 0 ;
                }
        }
 
        /*if( ChaineIdentique == 1 )
        {
                printf("\n\n____________________IDENTIQUE____________________\n\n");
                system("pause");
                exit(0);
        }*/
 
        //jusqu'ici
        /** Condition qui verifie si ma variable " boolean " est égal a 0 si oui la variable " nbEssaie " est décrémenter de 1 */
 
        if ( boolean == 0)
        {
            nbEssai--;
        }
    }
 
 
    /*******************************/
    /** Affichage du tableau  a la fin du grace a une boucle */
 
    printf("\n\n");
    printf("\n");
    for (i=0; i<longueurchaine; i++)
    {
        printf("%c",motSecretcopy[i]);
    }
 
 
    return 0;
}
 
/*******************************/
/** Fonction qui permet de vider le scanf avant de le réutiliser dans une boucle afin d'éviter qui garde en mémoire un caractère */
 
void MyFlush(void)
{
    scanf("%*[^\n]");
    getchar();
}

la deuxième

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>
void MyFlush(void);
int main()
{
    int i ;
    int nbEssai = 10;
    int longueurchaine = 10;
    int ChaineIdentique = 0;//s'éatit 1, maintenant 0
    int boolean = 0;
    const char motSecret[10] = "pinom";
    char motSecretcopy[10] ;
    char lettre;
    char chaine1[longueurchaine] , chaine2[longueurchaine];
 
    /*******************************/
    /** longueurchaine est égal a la taille de la chaine "motSecret" .*/
 
    longueurchaine = strlen(motSecret);
 
    /*******************************/
    /** création d'un tableau d'étoile égale a la longueur de la chaine motSecret .*/
 
    for (i=0; i<longueurchaine; i++)
    {
        motSecretcopy[i]='*';
    }
 
 
    /*******************************/
    printf("\n");
    printf("Bienvenue dans le Pendu\n");
 
 
 
    /*******************************/
    /** tant que le nombre d’essai est inférieur à 0 ou que la chaine de caractère est différente on boucle */
 
    while ( nbEssai > 0 )//&& ChaineIdentique != 1)//l'initialisation de ChaineIdentique est chnagé en haut
 
    {
        printf("\nIl vous reste %d essai \n\n",nbEssai);
        printf("\nEntrer une lettre ! ");
        scanf ("%c",&lettre);
        MyFlush();
        printf("\n\n");
        boolean=0;
 
        /** boucle qui parcours le tableau */
 
        for (i=0; i<longueurchaine; i++ )
 
        {
            /** Condition qui vérifie si la lettre entrer est égal au tableau a la position "i"
                si oui je copie la lettre entrer dans un autre tableau et je met la variable
                "boolean a a 1 afin de ne pas décrémenter la variable "essaie"   */
 
            if (lettre == motSecret[i])
 
            {
                motSecretcopy[i]=lettre;
                boolean=1;
 
            }
 
        }
 
        /*******************************/
        /** Boucle qui affiche le tableau */
 
        for (i=0; i<longueurchaine; i++)
        {
            printf("%c",motSecretcopy[i]);
        }
 
        /** La variable " ChaineIdentique " contient la fonction strcmp qui permet de verifier si deux chaine de caractére sont identiques .*/
        //ChaineIdentique =  strcmp(motSecret, motSecretcopy) ;
 
 
        //voila ce que j'ai ajouté
          ChaineIdentique = 1 ;
        for(int j = 0 ; j < longueurchaine ; j++)
        {
                if( motSecret[j] != motSecretcopy[j] )
                {
                 ChaineIdentique = 0 ;
                }
        }
 
        if( ChaineIdentique == 1 )
        {
                printf("\n\n____________________IDENTIQUE____________________\n\n");
                system("pause");
                exit(0);
        }
 
        //jusqu'ici
        /** Condition qui verifie si ma variable " boolean " est égal a 0 si oui la variable " nbEssaie " est décrémenter de 1 */
 
        if ( boolean == 0)
        {
            nbEssai--;
        }
    }
 
 
    /*******************************/
    /** Affichage du tableau  a la fin du grace a une boucle */
 
    printf("\n\n");
    printf("\n");
    for (i=0; i<longueurchaine; i++)
    {
        printf("%c",motSecretcopy[i]);
    }
 
 
    return 0;
}
 
/*******************************/
/** Fonction qui permet de vider le scanf avant de le réutiliser dans une boucle afin d'éviter qui garde en mémoire un caractère */
 
void MyFlush(void)
{
    scanf("%*[^\n]");
    getchar();
}

j'ai juste commenté les parties différentes, et le mot secret == "pinom" pas "elephant"

**czar1983** · 16/06/2013, 00h18

Déjà merci beaucoup pour ton aide

bon la première chose que j'ai faite c'est de tester ton code et .... il me dit que ton for suivant

Code :

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

1
2
3
4
5
6
7
8
       ChaineIdentique = 1 ;
        for(int j = 0 ; j < longueurchaine ; j++)
        {
                if( motSecret[j] != motSecretcopy[j] )
                {
                 ChaineIdentique = 0 ;
                }
        }

est simplement allowed en mode c99 et moi je suis en c89 un truc comme ça lol

**b-med** · 16/06/2013, 00h24

J'ai pas compris

c'est quoi c89 et c99, explique bien

**b-med** · 16/06/2013, 00h35

Ok oK, c'est la normalisation, je sais pas les différences

**czar1983** · 16/06/2013, 00h38

en fait tu sait la ligne suivante :

Code :

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

        for(int j = 0 ; j < longueurchaine ; j++)

n'est pas reconnu en c89 voila ce que l on dit sur un site

" Le langage C existe en plusieurs versions.
Une version récente, appelée le C99, autorise la création de tableaux à taille dynamique, c'est-à-dire de tableaux dont la taille est définie par une variable

Or cela n'est pas forcément reconnu par tous les compilateurs, certains planteront sur la seconde ligne. Le langage C que je vous enseigne depuis le début (appelé le C89) n'autorise pas ce genre de choses. Nous considèrerons donc que faire cela est interdit. "

Donc en fait ton code fonctionne A MERVEILLEEEEE lol j'ai juste retirer le int dans le for et déclarer le int j; en debut de programme , car le mode c89 n'autorise pas la déclaration d'une variable dans le for ( contrairement au c99 ou au JAVA )

Code :

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

 for( j = 0 ; j < longueurchaine ; j++)

je vois que tu a retirer la fonction STRCMP pour en créer une toi même qui fait exactement la même chose .
sait tu pourquoi la fonction STRCMP fonctionne quand je ne déclare pas la variable boolean qui décrémente? je trouve ça étrange comme comportement .

de plus je vois de nouvelle chose grâce a ton code comme le exit ou le system pause .

**czar1983** · 16/06/2013, 00h56

voila j'ai modifier ceci

Code :

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

1
2
3
4
5
6
/**************************************/
  /** ton code  while ( nbEssai > 0 ) */
 
     /**************************************/
  /** MON CODE  */
    while ( nbEssai > 0 && ChaineIdentique!=1)

et retirer ceci .

Code :

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

1
2
3
4
5
6
7
8
9
10
 
  /**************************************/
  /** ton code  */
 
//        if( ChaineIdentique == 1 )
//        {
//                printf("\n\n____________________IDENTIQUE____________________\n\n");
//                system("pause");
//                exit(0);
//        }

et garder ton for qui boucle pour vérifier le tableau et si le tableau est égal ta variable chaineIdentique est égal a 1

Code :

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

1
2
3
4
5
6
7
8
     ChaineIdentique = 1 ;
        for(j = 0 ; j < longueurchaine ; j++)
        {
                if( motSecret[j] != motSecretcopy[j] )
                {
                 ChaineIdentique = 0 ;
                }
        }

**moins1** · 16/06/2013, 08h59

Tu devrais faire en sorte que motSecretcopy soit une "vrai" chaîne de caractère avec le '\0' à la fin quoi. Ça te faciliterais la tâche un peu. Pour l'affichage tu pourrais utiliser printf() ou puts(). Pour vérifier si le mot est trouvé tu pourrais utiliser strchr() ou strcmp().

Par exemple pour savoir si le mot est trouvé avec strchr():

Code :

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

1
2
3
4
5
 
if((strchr(motSecretcopy, '*') == NULL)
     // Mot est trouvé, y'a plus de '*'
else
    // Pas trouvé.

Sinon pour placer les '*' y'a memset();

Code :

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

1
2
 
memset(motSecretcopy, '*', longueurchaine);

Enfin c'est juste de petites idées comme ça.

**gangsoleil** · 17/06/2013, 10h38

Bonjour,

Tes chaines ne sont simplement pas identiques, car tu n'initialises pas la fin de la chaine de motSecretcopy.

Voici un affichage des deux chaines :

Code :

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