Discussion :

[Zak Blayde]

Bonjour

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
              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 ?

[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

[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 :

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

scanf("%ld", &choixContinuer);

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

Code :

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

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

[Zak Blayde]

Oui, c'est n'importe quoi, mais j'apprends
Vous pouvez me le corriger ?

[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).

[thewho]

Bonjour,

On s'est croisés pendant que j'éditais mon post

[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 ^^.

[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

[Thierry Chappuis]

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

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
#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

[Zak Blayde]

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

[Zak Blayde]

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
 
            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 ?

[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 :

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

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.

[Zak Blayde]

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

[Emmanuel Delahaye]

Code :

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

1
2
3
 
                    scanf("%ld", &choixContinuer);
                    if (choixContinuer == "1")

Quels sont les mots que tu ne comprends pas dans

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 :

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

                    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.

[Zak Blayde]

Un autre problème maintenant

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
        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 ?

[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