Discussion :

[Roud9]

Bonjour,

Je suis entrain d'écrire un petit programme et je suis bloqué lorsque je demande à l'utilisateur d'entrer un nombre qui est compris entre deux bornes. Mon idée, c'était de vérifier si l'utilisateur a bien entré un nombre en plus des bornes, mais lorsque j'entre des lettres, j'ai une boucle infinie. Voici le code problématique :

Code :

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

1
2
3
4
5
 do {
      printf("Entrez un nombre entre 1 et 26 : \n");
      scanf("%d", &decalage);
      printf("%d\n" , decalage);
   } while ((decalage < 1 || decalage > 26) && isdigit(decalage) == 0);

La vérification du nombre compris entre 1 et 26 fonctionne bien, mais le problème c'est lorsque j'entre autre chose que des chiffres. Pourrie-Vous m'aider?

Merci

[imperio]

Quel est l'intérêt de faire un isdigit sur un int ? O.o

Le mieux serait de faire un fgets puis un atoi (ou fonction plus sécurisée telle que strtol). Si tu utilises strtol, inutile de faire ceci :

Code :

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

1
2
3
4
5
6
7
8
if (fget(s, 10, stdin) != NULL) {
    int i = 0;
 
    while (s[i] && i < 10 && isdigit(s[i]))
        i++;
    if (i >= 10 || s[i])
       continue;
}

[Invité]

Tu devrais t'intéresser sur le retour de la fonction scanf().

Scanf() est une fonction attendant une saisie formaté. Il lit dans le stdin TANT QUE ce qu'il lit respecte le format indiqué. Et, accessoirement, retourne le nombre de caractère ayant été lu.

A ton avis, que va t'elle renvoyer comme valeur si elle lit directement un caractère alors que l'on demande un entier ?

[Roud9]

Merci de vos réponses.

J'ai finalement résolu mon problème en utilisant scanf tel que vous l'aviez suggéré. La boucle infinie venait du '\n' qui est compté comme un caractètre entré à nouveau si l'entrée est invalide.

[Sve@r]

J'ai finalement résolu mon problème en utilisant scanf tel que vous l'aviez suggéré. La boucle infinie venait du '\n' qui est compté comme un caractètre entré à nouveau si l'entrée est invalide.

Bonjour

Il est vrai que tu as résolu ce problème à court terme. Mais si tu t'obstines à vouloir utiliser scanf() tu vas au devant de gros ennuis à moyen et long terme. Car comme tu l'as dit, le caractère non récupéré par scanf() reste dans stdin ce qui peut ensuite te pourrir tes saisies suivantes. Et ne parlons pas du cas où tu vas ensuite tenter de coupler ton programme à un flux extérieur via un pipe.

Si vraiment tu veux faire une saisie "propre", alors suis les conseils d'imperio => un fgets() pour récupérer tout le stdin (ce qui le nettoie) puis une analyse de la chaine récupérée...