problème de boucle

Version imprimable

problème de boucle

Bonjour

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
              do
                {
                    scanf("%ld", &choixContinuer);
                    if (choixContinuer == "Oui")
                    {
                        continuer=1;
                    }
                    else if (choixContinuer == "Non")
                    {
                        continuer=0;
                    }
                    else
                    {
                        printf("Veuillez choisir entre Oui et Non.\n");
                    }
                }while ((choixContinuer != "Oui") && (choixContinuer != "Non"));

Le résultat c'est que quoi que je mette (Oui, Non ou gagougi), la console me répète indéfiniment "Veuillez choisir entre Oui et Non." sans revenir au début de la boucle.
Quelle est mon erreur svp ?

16/02/2007, 23h47
thewho

Bonjour,

Tu fais un scanf avec %ld avec ce qui es manifestement un pointeur sur une chaîne, d'après le code qui suit.

Et regarde ton test de contrôle de boucle
16/02/2007, 23h48
DaZumba

La comparaison de chaine se fait avec strcmp(). Le specificateur de format de scanf() pour une chaine n'est pas %ld, mais %s (et il vaut mieux utiliser fgets() de toute facon).
16/02/2007, 23h52
thewho

Bonjour,

On s'est croisés pendant que j'éditais mon post
16/02/2007, 23h53
Zak Blayde

Ahhhhhhh j'y avais pas pensé ! Je fais un tutoriel C et j'ai pas encore vu les chaines de caractères ^^. Bon j'essaie de bidouiller avec ce que vous m'avez dit. Merci.

edit : vous pouvez me le corriger ? J'apprendrais plus tard avec le tuto ^^.
16/02/2007, 23h57
Thierry Chappuis

Je ne sais pas de quel type est ta variable choixContinuer, mais si il s'agit d'une chaine de caractères, ton appel à scanf(), c'est n'importe quoi:

Code:

scanf("%ld", &choixContinuer);

Ceci laisserait supposer que choixContinuer est de type long int, ce qui est absolument incohérent avec:

Code:

if (choixContinuer == "Oui")

Par ailleurs, si on admet que choixContinuer est une chaîne de caractères, l'opérateur de comparaison == ne te permet pas de tester l'égalité de deux chaînes des caractères. Au lieu de cela, cette expression teste l'égalité de deux adresses.

Thierry
16/02/2007, 23h59
Zak Blayde

Oui, c'est n'importe quoi, mais j'apprends ;)
Vous pouvez me le corriger ?
17/02/2007, 00h09
Thierry Chappuis

Je vois déjà certain arriver en disant que l'apprentissage du C ne se fait pas en improvisant. C is a sharp... Commence donc par éliminer le mot bidouiller de ton esprit. Je ne sais pas si cela remplace un bon bouquin sur le C, mais j'en ai entendu des bons échos. Avoir un bon livre de référence est également un bonne idée. Lorsqu'on a un doute sur un aspect du langage, on retrouve parfois plus rapidement son chemin que dans un tutoriel. Pour l'heure, c'est excellent d'explorer par toi même, mais essaie de bien saisir les subtilités que tu rencontrera dans le tutoriel. Les C est rempli de pièges... N'hésite pas à poser des questions lorsque tu ne comprends pas.

En ce qui concerne ton code, je ne vois pas bien ce que tu veux faire avec ta boucle... Restez dans la boucle (qui ne fait rien d'autre que de reposer la question à chaque tour) tant que la réponse est non?

Thierry

C'est pas idéal, mais à cette heure ci...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(void)
{
    char s_buffer[10] = {0};
 
    while (strstr(s_buffer, "Oui") == NULL)
    {
        printf("Veuillez choisir entre Oui et Non: ");
        fflush(stdout);
 
        fgets(s_buffer, sizeof s_buffer, stdin);
    }
 
    return EXIT_SUCCESS;
}

Thierry

17/02/2007, 11h37
Zak Blayde

J'ai l'impression qu'il en manque la moitié. Bon, je vais faire sans chaines de caractères en attendant.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
            do
                {
                    scanf("%ld", &choixContinuer);
                    if (choixContinuer == "1")
                    {
                        continuer=1;
                    }
                    else if (choixContinuer == "2")
                    {
                        continuer=0;
                    }
                    else
                    {
                        printf("Veuillez choisir entre 1 et 2.\n");
                    }
                }while ((choixContinuer != "1") && (choixContinuer != "2"));

J'arrive toujours pas à sortir de la boucle en répondant 1 ou 2. Sûrement une erreur dans mon ET ?

17/02/2007, 11h48
thewho

Bonjour,

On te l'a déjà dit, on n'apprend pas la programmation en "bidouillant" jusqu'à ce que ça marche.

A ton avis, que fais-tu en écrivant

Code:

if (choixContinuer == "1")

C'est faux, quel que soit le type de choixContinuer.

Reprends ton cours ou tes docs, suit les exemples en t'efforçant de comprendre ils ont été écrits comme ils le sont, et pas seulement en les copiant pour voir si ça fonctionne.
17/02/2007, 11h56
Zak Blayde

ok les guillemets sont de trop. Encore une erreur malgré tout... Surement plus haut dans mon main. Je cherche.

Un autre problème maintenant

Code:

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
        do
        {
            printf("Quel est le nombre mystere ?\n");
            nombreCoups++;
            scanf("%ld", &nombre);
            if (nombre > nombreMystere)
            {
                printf("Le nombre mystere est plus petit !\n");
            }
            else if (nombre < nombreMystere)
            {
                printf("Le nombre mystere est plus grand !\n");
            }
            else if (nombre == nombreMystere)
            {
                printf("Congratulations, YOU WIN !\n");
                printf("Vous avez reussi en %ld coups !\n", nombreCoups);
                printf("1-Continuer\n2-Arreter\n");
                do
                {
                    scanf("%ld", &choixContinuer);
                    if (choixContinuer == 1)
                    {
                        continuer=1;
                    }
                    else if (choixContinuer == 2)
                    {
                        continuer=0;
                    }
                    else
                    {
                        printf("Veuillez choisir entre 1 et 2.\n");
                    }
                }while ((choixContinuer != 1) && (choixContinuer != 2));
            }
            else
            {
                printf("Ceci n'est pas une donnee valide");
            }
        }while(nombre != nombreMystere);

si l'utilisateur entre du texte et non un nombre, je n'entre pas dans le else, le programme plante.
Est-ce que c'est du au scanf qui ne supporte pas le texte ou une erreur de ma part ?

17/02/2007, 13h08
Thierry Chappuis

C'est dû à une erreur de ta part, parce que tu ne vérifie pas la valeur de retour de scanf(). Pour apprendre à bien utiliser cette fonction complexe: http://xrenault.developpez.com/tutoriels/c/scanf/. Pour ne pas avoir ce genre de surprises, tu dois sécuriser tes entrées utilisateurs.

Thierry
17/02/2007, 13h12
Thierry Chappuis

Citation:

Envoyé par Zak Blayde

J'ai l'impression qu'il en manque la moitié. Bon, je vais faire sans chaines de caractères en attendant.

Comment cela? Mon code fait la même chose que le code original que tu as posté, sauf qu'il n'utilise pas de variable continuer et que toute entrée autre que Oui est considérée comme Non. Faiblesse: fsgdOuin est considéré comme Oui.

Thierry
18/02/2007, 11h11
Emmanuel Delahaye
Citation:
Envoyé par Zak Blayde

Code:

1 2 3 scanf("%ld", &choixContinuer); if (choixContinuer == "1")
Quels sont les mots que tu ne comprends pas dans

Citation:

Envoyé par Dazumba

La comparaison de chaine se fait avec strcmp()

D'autre part, choixContinuer est un entier, donc il faut le comparer avec un entier et non une chaine...

Code:

if (choixContinuer == 1)

On ne peut pas apprendre le C en bricolant. Il suivre un cours sérieux et s'y tenir. Il y a des références de tutoriels et de livres sur mon site.