Aide avec scanf

Version imprimable

Aide avec scanf

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:

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 :

Citation:

[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

03/11/2008, 14h05
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...
03/11/2008, 15h47
Emmanuel Delahaye
Citation:

Envoyé par Link/DD

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:

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:

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

Citation:

Envoyé par 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...

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

Code:

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:

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);
   }
}

03/11/2008, 16h01
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 ?
03/11/2008, 17h55
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