Controle de saisie !

**A_B** · 12/04/2007, 18h34

Salut :
Je travaille sur un programme qui comporte un menu de 1 a 5 (les choix) !
Le probleme c'est que lorsque l'utilisateur saisie un choix invalide (un char au lieu de int ) par exemple le programme se bloque (boucle infini) alors je veut controler la saisie d'une façon simple et efficace !
MERCI

**Gruik** · 12/04/2007, 18h44

Salut

Et tu utilises quoi pour lire le choix?

**A_B** · 12/04/2007, 18h48

Code :

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

1
2
int choix;
scanf("%d",&choix);

**Gruik** · 12/04/2007, 18h55

Pour la boucle infinie c'est un probleme dans l'algo et on pourra pas t'aider sans le code ,)
Une des choses à faire lorsqu'on fait des lectures sur l'entree standard est de nettoyer l'entrée standard apres lecture (voir FAQ)

**A_B** · 12/04/2007, 19h05

Vous m'avez mal compris :
Le programme attend la saisie d'un entier, alors que l'utilisateur a saisie un caractere, il existe des fonctions de teste comme ISDIGIT ou ISALPHA ...etc..
Mais ce que j'attend c'est la meilleur façon de les implementer car j'ai essayé mais ca ne donne pas grand chose ,..
Par exemple ces fonctions de tests prennent comme parametre un CHAR alors j'ai declaré choix comme CHAR et non pas INT, apres le teste , si ça va et que choix est un INT comment je convertit un char en un int ..?

**Gruik** · 12/04/2007, 19h14

Ah oui pardon
Bein scanf() doit retourner une erreur indiquant que le format n'est pas valide (on a saisi un caractere au lieu d'un chiffre), mais je connais pas scanf.. alors man 3 scanf
Cela dit, on est plutot partisans ici du couple fgets(buff, stdin) + strtoul() avec controle d'erreur suite à strtoul

**IDE** · 12/04/2007, 19h20

Bonjour, Voici un menu que j'ai réalisé avec l'aide de certaines personnes de ce site, si sa peut t'aider

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
 
tr_e Menu (void)
{  
   int c;
   char buffer [4];                                      /* Déclaration d'un buffer temporaire */
 
   gotoxy (1, 7);
   printf ("\t\t1. Ajout d'une fiche\n"                  /* Affichage du menu à l'écran */
           "\t\t2. Chercher une fiche\n"
           "\t\t3. Afficher une fiche\n"
           "\t\t4. Suppression d'une fiche\n"
	   "\t\t5. Quitter\n\n" "\t\tEntrez votre choix : "); 
 
  if ( fgets (buffer, sizeof buffer , stdin) != NULL )   /* Récupération de la saisie utilisateur donc stdin comme flux (dernier argument de fgets) */
    {
      c = (int) strtol (buffer, NULL, 10);               /* Conversion de la chaîne (buf) en entier */
    }
 
   return c;
 }

**Thierry Chappuis** · 12/04/2007, 19h47

Personnellement, je n'utilise jamais scanf(). C'est possible de l'utiliser proprement, mais la tâche est ardue. Pour les entrées utilisateur, j'utilise des fonctions personnalisées à base de fgets(). Voici un exemple de fonction pour saisir un entier de manière sécurisée (J'espère qu'il n'y a pas d'erreur... je n'ai pas de compilo aujourd'hui sous la main pour tester):

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
enum IOErrors {
    IO_OK,
    IO_ERR_NULL_ARG,
    IO_ERR_INPUT,
    IO_ERR_FORMAT,
    IO_ERR_BUFFER,
    IO_ERR_UNKNOWN
};
 
int get_long(long *number);
 
int main(void)
{
    long choix = 0;
    int fin = 0;
    int err = IO_OK;
 
    do
    {
        printf("choisissez (1), (2) ou (3): ");
        fflush(stdout);
        err = get_long(&choix);
        if (choix >= 1 && choix <= 3)
        {
            fin = 1;
        }
    } while (err != IO_OK || fin != 1);
 
    printf("choix vaut %ld\n", choix);
 
    return EXIT_SUCCESS;
}
 
int get_long(long *number)
{
    int err = IO_OK; 
 
    if (number != NULL)
    {
        char s_buffer[12] = {0};
 
        if (fgets(s_buffer, sizeof s_buffer, stdin) != NULL)
        {
            char *pc = strchr(s_buffer, '\n');
            if (pc != NULL) /* Saisie OK */
            {
                long nb;
                char *p_end = NULL;
 
                /* On élimine le caractère de fin de ligne du tampon */
                *pc = 0;
 
                nb = strtol(s_buffer, &p_end, 0);
                if (*p_end == 0)
                {
                    /* Format de l'entrée utilisateur OK */
                    *number = nb;
                }
                else
                {
                     /* Erreur dans le format de la saisie */
                     err = IO_ERR_FORMAT;
                }
            }
            else
            {
                /* L'entrée utilisateur a été tronquée: il faut vider le tampon */
                int c;
                while ((c = fgetc(stdin)) != '\n' && c != EOF)
                {
                    /* On ignore les caractères */
                    continue;
                }
                err = IO_ERR_BUFFER;
            }
        }
        else
        {
            /* Erreur durant la saisie */
            err = IO_ERR_INPUT;
        }
    }
    else
    {
        /* Erreur d'argument: number ne doit pas être NULL */
        err = IO_ERR_NULL_ARG;
    }
    return err;
}

Pour réaliser une fois pour toute tes propres fonctions d'entrée/sortie sécurisées, voici un lien qui peut t'intéresser: http://emmanuel-delahaye.developpez.com/inputs.htm

Thierry