scanf vs fgets

**_SamSoft_** · 02/09/2007, 17h12

Bonjour, alors voilà comme je suis dans ma période de "remise à niveau en C" j'ai décidé de créer un code illustrant pourquoi il ne faut pas utiliser scanf mais fgets (tout le monde sait ca mais je voulais vous l'illustrer)

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
 
#include <stdio.h>
#include <stdlib.h>
#include <stddef.h>
#include "conio.h"
int main(void)
{
  //Exemple with good functions
  char str[256], str2[256];
  puts("Exemple with good functions");
  puts("str =");
  fgets(str, sizeof str, stdin);
  puts("str2=");
  fgets(str2, sizeof str2, stdin);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  clrscr();
  //Exemple with bad functions
  puts("Exemple with bad functions");
  printf("str=");
  scanf("%s", &str);
  printf("str2=");
  scanf("%s", &str2);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  return 0;
}

Si vous testez en entrant par exemple : "mon texte", aucune différence (visible) n'apparaîtra à l'écran mais testez avec cette chaine de caractères :

Test numéro 98. Test numéro x0çéà&à=ù$=lpé&às)

PS: Ceci s'adresse aux gens qui sont encore pour scanf

En souhaitant que je ne vais pas me faire descendre

**Skyounet** · 02/09/2007, 17h17

Ton scanf est faux, pas de & avec un %s...

De plus, la principale raison pour laquelle il ne faut pas utiliser scanf avec un %s est simplement parce qu'il n'y a pas de contrôle sur la taille entrée, pouvant provoquer un buffer overflow.

Ton exemple n'illustre pas ce danger. Et je n'ai d'ailleurs pas tout à fait compris ce qu'il est censé illustrer.

**_SamSoft_** · 02/09/2007, 17h22

Ayant 15ans et étant en période de remise à niveau, je me concentre sur l'apprentissage intensif

Je sais juste que mon code sert à montrer qu'on rencontre plus facilement des problèmes avec scanf qu'avec fgets, vous l'avez dit vous même, je me suis trompé avec scanf alors que fgets est plus simple d'utilisation

Regardez :

Si vous tapez lors du premier test (celui avec fgets) :

Test numéro 98. Test numéro x0çéà&à=ù$=lpé&às)

Deux fois, vous aurez : str = str2

Si vous effectuez la même manipulation lors du second exempel (celui avec scanf) vous aurez :

str= str!=str2 (ou un truc dans le genre) donc...

Code mis à jour :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <stddef.h>
#include "conio.h"
int main(void)
{
  //Exemple with good functions
  char str[256], str2[256];
  puts("Exemple with good functions");
  puts("str =");
  fgets(str, sizeof str, stdin);
  puts("str2=");
  fgets(str2, sizeof str2, stdin);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  clrscr();
  //Exemple with bad functions
  puts("Exemple with bad functions");
  printf("str=");
  scanf("%s", str);
  printf("str2=");
  scanf("%s", str2);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  return 0;
}

**Emmanuel Delahaye** · 02/09/2007, 19h09

Envoyé par _SamSoft_

Code :

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

1
2
3
4
5
6
7
 
  char str[256], str2[256];
 
  //Exemple with bad functions
  puts("Exemple with bad functions");
  printf("str=");
  scanf("%s", str);

Le problème de scanf() n'est pas que c'est une 'mauvaise fonction' (je rappelle au passage que 'exemple' se dit 'example' en anglais...), mais que la plupart du temps, elle est mal utilisée.

Ici, par exemple, il n'est fait aucun contrôle de débordement.

Par contre, il existe un vrai problème avec scanf() et "%s", c'est pour la saisie de textes séparés par des 'blancs' (blank : espace, tabulation, fin de ligne). A ma connaissance, il n'y a pas de solution pour que scanf() ignore qu'un 'blanc' est un séparateur.

**Vincent Rogier** · 02/09/2007, 19h32

De plus, la principale raison pour laquelle il ne faut pas utiliser scanf avec un %s est simplement parce qu'il n'y a pas de contrôle sur la taille entrée, pouvant provoquer un buffer overflow.

Skyrunner, c'est faux car scanf te permet de spéficier la taille du buffer, ce qu'il faut toujours faire avec %s !

Le soucis, comme le dit Emmanuel Delahaye, c'est la gestion des blancs.

Sinon, je ne vois pas l'utilité de ce code et de cette démonstration ....

gl · 02/09/2007, 19h39

Envoyé par Emmanuel Delahaye

Par contre, il existe un vrai problème avec scanf() et "%s", c'est pour la saisie de textes séparés par des 'blancs' (blank : espace, tabulation, fin de ligne). A ma connaissance, il n'y a pas de solution pour que scanf() ignore qu'un 'blanc' est un séparateur.

C'est possible en jouant avec %[^...]. Par exemple pour ne conserver que \n comme separateur, il faut remplacer %s par %[^\n].
De meme il est possible de limiter la taille maximale de la saisie.

Pour plus d'information sur l'utilisation de scanf : Scanf démythifiée.

Il est tout de meme largement preferable d'utiliser fgets() qui est bien plus simple d'utilisation (et generalement bien mieux utiliser que scanf()).

**Skyounet** · 02/09/2007, 19h41

Envoyé par vicenzo

Skyrunner, c'est faux car scanf te permet de spéficier la taille du buffer, ce qu'il faut toujours faire avec %s !

Le soucis, comme le dit Emmanuel Delahaye, c'est la gestion des blancs.

Sinon, je ne vois pas l'utilité de ce code et de cette démonstration ....

Je parlais simplement par défaut, il est vrai qu'en jouant avec les options de scanf on peut faire des choses assez intéressantes, mais le débutant de base, ne se jete pas à bras ouverts dans les pages de man.

gl · 02/09/2007, 19h45

Envoyé par vicenzo

Skyrunner, c'est faux car scanf te permet de spéficier la taille du buffer, ce qu'il faut toujours faire avec %s !

Oui, il est possible de gerer la taille des buffer avec le scanf (tout comme il est possible de gerer les blancs), mais c'est relativement complexe a mettre en oeuvre (par rapport a une fonction comme fgets()) et assez facile de l'oublier.
En conclusion, scanf() n'est certainement pas la fonction la plus adaptee pour saisir une chaine de caractere. Il est largement preferable d'utiliser fgets()

**Vincent Rogier** · 02/09/2007, 19h53

...il est vrai qu'en jouant avec les options de scanf on peut faire des choses assez intéressantes, mais le débutant de base....

Préciser la taille du buffer avec scanf() et %s ne relève pas de jouer avec des options, c'est vital pour produire du code sur !

Donc, un même un débutant devrait toujours spécifier la taille du buffer avec un %s.....

**Skyounet** · 02/09/2007, 19h57

Envoyé par vicenzo

Préciser la taille du buffer avec scanf() et %s ne relève pas de jouer avec des options, c'est vital pour produire du code sur !

Donc, un même un débutant devrait toujours spécifier la taille du buffer avec un %s.....

Non, la seule chose vitale est de laisser tomber scanf lorsqu'on est débutant et se tourner vers fgets qui lui est simple à utiliser.

gl · 02/09/2007, 20h00

Envoyé par vicenzo

Préciser la taille du buffer avec scanf() et %s ne relève pas de jouer avec des options, c'est vital pour produire du code sur !

Donc, un même un débutant devrait toujours spécifier la taille du buffer avec un %s.....

Ou ne pas utiliser scanf() dans ce cas.

Autant je suis d'accord pour dire que scanf() est une fonction tres pratique et tres puissante dans bien des cas.
Autant je ne vois pas pourquoi vouloir absolument utiliser scanf() pour effectuer une tache aussi simple que la saisie d'une chaine de caractere ?

**_SamSoft_** · 02/09/2007, 20h02

Je suis un peu un débutant (qui a sauté trop d'étapes) donc là je revois tout et je suis pour l'utilisation de fgets au quotidien (on dirait une pub

)

**Thierry Chappuis** · 03/09/2007, 00h10

Par rapport au code proposé par __SamSoft__, je voudrais ajouter qu'une bonne utilisation de fgets() doit en principe toujours s'accompagner d'une purge du tampon du flux d'entrée comme dans l'exemple publié sur ce poste.

Si, conjointement à la purge du flux d'entrée, on désire enlever le caractère de fin de ligne saisie avec fgets(), je conseille l'utilisation systématique de la fonction suivante avec fgets():

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
#include <stdio.h>
#include <string.h>
 
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL && stream != NULL)
    {
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL) /* La saisie n'a pas ete tronquee */
        {
             /* On remplace '\n' par le caractere nul '\0' */
             *pc = 0;
        }
        else
        {
            /* La saisie a ete tronquee, on purge le flux d'entree */
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
            }
        }
    }
}

Thierry

**_SamSoft_** · 03/09/2007, 09h23

Merci pour la précision

On se retrouve donc avec ceci :

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
 
/*
  Name: fgets vs scanf
  Copyright: GPL
  Authors: Samy Hocine and some membres of the C forum from developpez.com
  Date: 03/09/07 09:24
  Description: Little code
*/
#include <stdio.h>
#include <stdlib.h>
#include <stddef.h>
#include "conio.h"
void fclean(char *s_buffer, FILE *stream);
int main(void)
{
  //Example with good functions
  char str[256], str2[256];
  puts("Exemple with good functions");
  puts("str =");
  fgets(str, sizeof str, stdin);
  fclean(str, stdout);
  puts("str2=");
  fgets(str2, sizeof str2, stdin);
  fclean(str2, stdout);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  clrscr();
  //Example with bad functions
  puts("Exemple with bad functions");
  printf("str=");
  scanf("%s", str);
  printf("str2=");
  scanf("%s", str2);
  if(strcmp (str, str2) == 0)
           puts("str = str2");
  else
           puts("str != str2");
  getch();
  return 0;
}
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL && stream != NULL)
    {
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL) /* La saisie n'a pas été tronquée */
        {
             /* On remplace '\n' par le caractère nul '\0' */
             *pc = 0;
        }
        else
        {
            /* La saisie a été tronquée, on purge le flux d'entrée */
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                continue;
            }
        }
    }
}

**Thierry Chappuis** · 03/09/2007, 09h36

Envoyé par _SamSoft_

Merci pour la précision

On se retrouve donc avec ceci :

Code :

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

1
2
3
4
5
6
7
8
 
/*
  Name: fgets vs scanf
  Copyright: GPL
  Authors: Samy Hocine, Thierry Chappuis
  Date: 03/09/07 09:24
  Description: Little code
*/

Salut,

Je ne suis pas l'auteur de la fonction fclean(). Met plutôt "Membres du forum C de www.developpez.net", ça sera plus correct.

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
/*
  Name: fgets vs scanf
  Copyright: GPL
  Authors: Samy Hocine, Thierry Chappuis
  Date: 03/09/07 09:24
  Description: Little code
*/
#include <stdio.h>
#include <stdlib.h>
#include <stddef.h>
/* Si tu utilises strcmp(), il faut include ce fichier d'en-tête */
#include <string.h>
 
void fclean(char *s_buffer, FILE *stream);
 
int main(void)
{
    /* -TC- C'est une bonne habitude de déclarer une variable par ligne. Dans
       le contexte d'une chaine de caractères, il est également conseillé
       d'initialiser avec une chaine vide.
    */
    char str[256] = "";
    char str2[256] = "";
 
    puts("Exemple with good functions");
 
    puts("str =");
    /* -TC- Je teste en général la valeur retournée par fgets(). C'est en
       général une bonne habitude. Ici, ce n'est pas primordial.
    */
    fgets(str, sizeof str, stdin);
    /* -TC- Cela n'a pas de sens d'utiliser fclean() avec le flux stdout. Pour
       purger le flux de sortie standard, il y a la fonction fflush() déclarée
       dans le fichier d'en-tête standard stdio.h.
     */
    fclean(str, stdin);
 
    puts("str2=");
    fgets(str2, sizeof str2, stdin);
    /* -TC- Idem: stdout remplacé par stdin */
    fclean(str2, stdin);
 
    if (strcmp (str, str2) == 0)
    {
        puts("str = str2");
    }
    else
    {
        puts("str != str2");
    }
 
    /* A quoi te servent ces fonctions dans ta démonstration. Essaie de ne pas
        utiliser des fonctions non-standard lorsque ce n'est pas nécessaire */
    /* getch(); */
    /* clrscr(); */
    //Example with bad functions
    puts("Exemple with bad functions");
 
    /* ... */
    return 0;
}
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL && stream != NULL)
    {
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL) /* La saisie n'a pas été tronquée */
        {
            /* On remplace '\n' par le caractère nul '\0' */
            *pc = 0;
        }
        else
        {
            /* La saisie a été tronquée, on purge le flux d'entrée */
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                continue;
            }
        }
    }
}

scanf() n'est pas une mauvaise fonction, comme tu la présentes dans ton exemple. C'est une fonction complexe, réservée aux spécialistes (les débutants ne devraient jamais l'utiliser. Perso, je ne l'utilise jamais). Elle est utile dans certains cas, mais on peut presque toujours faire sans, ou alors passer par le couple fgets()/sscanf(), plus facile à manipuler au niveau de la gestion du tampon. Le grand défaut de ta démonstration, c'est qu'elle ne met pas vraiment en avant les problèmes posée par une utilisation malheureuse de scanf(). Pour te rendre compte des ces problèmes, je conseille une lecture attentive de l'article suivant qui est extrêment instructif: http://xrenault.developpez.com/tutoriels/c/scanf/

Cet article m'a définitivement convaincu que scanf(), c'était trop fort pour moi.

Thierry

**Emmanuel Delahaye** · 03/09/2007, 09h49

Envoyé par _SamSoft_

On se retrouve donc avec ceci :

C'est mieux.

Il manque quand même le <string.h> nécessaire pour déclarer correctement strchr().

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:24: warning: implicit declaration of function `strcmp'
main.c:29: warning: implicit declaration of function `clrscr'
main.c: In function `fclean':
main.c:47: warning: implicit declaration of function `strchr'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 5 seconds)
0 errors, 3 warnings

Si ton compilateur ne dit rien, c'est qu'il est mal configuré :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Reste ensuite à éliminer conio et les fonctions qui vont avec, inutiles ici. Et non, le remplacement de getch() n'est pas getchar(), mais une fonction 'maison' genre 'pause()' construite à base de fgets() et de fclean() comme montré précédemment.

Il y a des fautes d'anglais (membres -> members, exemple -> example)...

Enfin, ton code ne prouve pas grand chose et c'est normal, la saisie est libre par définition...

Je te conseille de placer ces fonctions (fclean(), pause() etc.) dans un fichier à part qui constituera la première pierre de ta bibliothèque personnelle.

**Emmanuel Delahaye** · 03/09/2007, 09h54

Envoyé par Thierry Chappuis

Code :

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

1
2
3
4
5
6
7
8
 
            /* La saisie a été tronquée, on purge le flux d'entrée */
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                continue;
            }
}

Le 'continue' est inutile...
J'évite les accents dans les codes sources. Pas portable...

**_SamSoft_** · 03/09/2007, 09h58

Envoyé par Emmanuel Delahaye

C'est mieux.

Il manque quand même le <string.h> nécessaire pour déclarer correctement strchr().

Reste ensuite à éliminer conio et les fonctions qui vont avec, inutiles ici. Et non, le remplacement de getch() n'est pas getchar(), mais une fonction 'maison' genre 'pause()' construite à base de fgets() et de fclean() comme montré précédemment.

Je te conseille de placer ces fonctions (fclean(), pause() etc.) dans un fichier à part qui constituera la première pierre de ta bibliothèque personnelle.

Je veux bien éliminer conio mais alors quel est l'équivalent de clrscr(); (ne me dites surtout pas system("CLS"); Oh que c'est pas portable !)
Sinon vous voyez des accents où ? (dans les commentaires ?)

**Skyounet** · 03/09/2007, 10h03

Envoyé par _SamSoft_

Je veux bien éliminer conio mais alors quel est l'équivalent de clrscr(); (ne me dites surtout pas system("CLS"); Oh que c'est pas portable !)
Sinon vous voyez des accents où ? (dans les commentaires ?)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#ifdef WIN32
   #define CLEAR "CLS"
#elif defined (linux)
   #define CLEAR "clear"
#else
   #error Not defined for this plateforme
#endif
 
[...]
system(CLEAR);

Oui les accents dans les commentaires.