Discussion :

[Link/DD]

Bonjour,

J'ai écris une fonction de saisie sécurisé que l'on utilise comme scanf, et qui utilise scanf, elle vide le buffer après la saisie et récupère le dernier \n, j'y ai inséré une boucle qui tourne tant que la saisie n'est pas bonne.

Ce que j'aimerai faire c'est inséré entre chaque format le format %n pour vérifier quelle saisie à planté.

Je vous poste le code commenté :

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
 
#include <stdio.h>
#include <stdarg.h>
#include <string.h>
 
void input (char *format, ...)
{
 
    int i;
    char seq[] = "%*[^\n]";
    char *ptn = format;
 
    /*
     * On cherche le nombre d'assignement
     * pour verifier la sortie du vscanf
     */ 
    for (i = 0; (ptn = strchr (ptn, '%')); ++i)
    {
        /* si le pointeur est null, alors il
         * n'y a plus d'assignement
         */
        if (ptn == NULL)
            break;
        else
        {
            ptn++;
            /* si le '%' est suivi d'un '*'
             * ou d'un 'n', il n'y a pas
             * d'assignement
             */
            if (*ptn == '*' || *ptn == 'n')
                i--;
        }
    }
 
    va_list v_arg;
    va_start (v_arg, format);
 
    /* tant que la saisie n'est pas bonne, on recommence */
    while (i != vscanf (format, v_arg))
    {
        printf ("Try again: ");
        scanf (seq); /* on vide le buffer */
        getchar(); /* on enleve le '\n' restant */
    }
    scanf (seq);
    getchar ();
 
    va_end (v_arg);
}

J'ai aussi cette erreur à la compilation :

[misc] david $ cc input.c -Wall
input.c: In function ‘input’:
input.c:26: attention : implicit declaration of function ‘vscanf’

D'après le manuel (man vscanf), elle se trouve dans stdarg.h qui est inclue pourtant.

Cordialement, Link/DD

[Médinoc]

Je crois que la fonction vscanf() n'est pas C90, mais C99. Essaie en rajoutant -std=c99 sur ta ligne de commande à gcc...

[Emmanuel Delahaye]

Je crois que la fonction vscanf() n'est pas C90, mais C99. Essaie en rajoutant -std=c99 sur ta ligne de commande à gcc...

Avec MinGW en mode C90 strict, aucun problème... (du moins, pas celui-la...)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c: In function `input':
C:\dev\hello\main.c:36: warning: ISO C90 forbids mixed declarations and code
C:\dev\hello\main.c:22: warning: will never be executed
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:58: warning: passing arg 1 of `input' discards qualifiers from pointer target type
Output size is 20.29 KB
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 3 warnings

Je recommande :

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
 
#include <stdio.h>
#include <stdarg.h>
#include <string.h>
 
void input (char const *format, ...)
{
 
   int i;
   char seq[] = "%*[^\n]";
   char const *ptn = format;
 
   /*
    * On cherche le nombre d'assignement
    * pour verifier la sortie du vscanf
    */
   for (i = 0; (ptn = strchr (ptn, '%')); ++i)
   {
      /* si le pointeur est null, alors il
       * n'y a plus d'assignement
       */
      if (ptn == NULL)
         break;
      else
      {
         ptn++;
         /* si le '%' est suivi d'un '*'
          * ou d'un 'n', il n'y a pas
          * d'assignement
          */
         if (*ptn == '*' || *ptn == 'n')
            i--;
      }
   }
 
   {
      va_list v_arg;
      va_start (v_arg, format);
 
      /* tant que la saisie n'est pas bonne, on recommence */
      while (i != vscanf (format, v_arg))
      {
         printf ("Try again: ");
         scanf (seq);           /* on vide le buffer */
         getchar ();            /* on enleve le '\n' restant */
      }
      scanf (seq);
      getchar ();
 
      va_end (v_arg);
   }
}

[muad'dib]

Excusez-moi mais je ne vois pas l'intérêt de réinventer la roue. À priori tu es entrain d'essayer de réecrire ce qui existe déja (fgets + vidage correct du buffer). Alors à moins que ce soit à but pédagogique ou intellectuel, pourquoi ne pas utiliser fgets ?

[Link/DD]

Emmanuel Delahaye: Il n'y a aucun problème si le développeur mets %4s, scanf récupèrera 4 caractères et le reste sera toujours dans le buffer, cependant ici le buffer est vidé.

L'utilisation de scanf ou de toute autre fonction de lecture sur l'entrée standard est dangereuse si on ne vérifie pas la taille de ce qui est récupéré. Pour l'instant ma fonction ne fais que vider le buffer et redemander la même chose à l'utilisateur tant que ce qu'il a entré est faux, je compte bien l'améliorer ^^

Je préfère bien utiliser scanf plutot qu'utiliser fgets et les fonctions de conversions adéquates ce qui fait souvent une masse non négligeable de code.

En compilant avec l'option -std=c99, je obtiens une erreur de segmentation.

Link/DD

[Emmanuel Delahaye]

J'ai écris une fonction de saisie sécurisé que l'on utilise comme scanf, et qui utilise scanf, elle vide le buffer après la saisie et récupère le dernier \n, j'y ai inséré une boucle qui tourne tant que la saisie n'est pas bonne.

Apparemment aussi mal sécurisée que scanf()...

Ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main (void)
{
   int n;
   char s[4];
 
   input ("%d%s", &n, s);
 
   printf ("%d '%s'\n", n, s);
 
   return 0;
}

provoque un crash...

Code :

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

1
2
3
4
5
6
 
123 unechaineunpeulongue...
123 'unechaineunpeulongue...'
 
Process returned -1073741819 (0xC0000005)   execution time : 33.872 s
Press any key to continue.

Je recommande de laisser tomber scanf() si on est pas un expert...

http://emmanuel-delahaye.developpez.com/inputs.htm