scanf vs fgets

**Thierry Chappuis** · 03/09/2007, 10h15

Envoyé par Emmanuel Delahaye

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

J'utilise souvent 'continue' pour documenter une boucle sans instructions. Cette pratique me vient de Richard Heathfield dans "C Unleashed". Je trouve de toute manière cela bien plus clair et lisible que:

Code :

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

while ((c = fgetc(stdin)) != '\n' && c != EOF);

ou

Code :

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

1
2
while ((c = fgetc(stdin)) != '\n' && c != EOF)
    ;

ou

Code :

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

while ((c = fgetc(stdin)) != '\n' && c != EOF) {}

Comme on le voit souvent. Maintenant, les questions de style... Je pourrais utiliser un commentaire à la place:

Code :

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

1
2
3
4
5
 
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
    /* This loop does nothing, absolutely nothing */
}

En ce qui concerne les accents, je ne les utilise que pour commenter mes code sur les forums de developpez...

Thierry

**_SamSoft_** · 03/09/2007, 10h31

Mise à jour :

main.c

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
 
/*
  Name: fgets vs scanf
  Copyright: GPL
  Authors: Samy Hocine and some members 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 <string.h>
#include "myHeader.h"
int main(void)
{
  //Example with good functions
  char str[256], str2[256];
  puts("Example 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();
  system(CLEAR);
  //Example with a difficult functions reserved for good programmers
  puts("Example 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;
}
//So if you're a beginner, you should use 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
25
26
27
28
29
30
31
32
33
 
#ifndef MYHEADER_H
#define MYHEADER_H
#ifdef WIN32
   #define CLEAR "CLS"
#elif defined (linux)
   #define CLEAR "clear"
#else
   #error Not defined for this plateforme
#endif
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)
            {
                  //Rien
            }
        }
    }
}
#endif //MYHEADER.H

Donc j'ai bien configuré le compilateur et il me met un warning pour getch(); ce qui est tout à fait normal vu que je n'ai pas inclu conio ! Mais je ne trouve pas d'équivalent, bien sûr il y a system("PAUSE"); (du moins sous Windows) mais je n'aime pas le texte. Je préferai getch();
Je l'ai quand même laissé.

**Emmanuel Delahaye** · 03/09/2007, 10h36

Envoyé par Thierry Chappuis

J'utilise souvent 'continue' pour documenter une boucle sans instructions. Cette pratique me vient de Richard Heathfield dans "C Unleashed". Je trouve de toute manière cela bien plus clair et lisible que:
<...>

La boucle vide explicite (et développée) me va très bien

Code :

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

1
2
3
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
}

Elle permet la maintenance (trace...)

Code :

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

1
2
3
4
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
   PRINTF(("purge\n"));
}

avec

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#ifndef DBG
#error DBG must be defined to 0 or 1
#endif
 
#if DBG
#define PRINTF(a) printf a
#else
#define PRINTF(a)
#endif

dans le header qui va bien (dbg.h, par exemple...)

En ce qui concerne les accents, je ne les utilise que pour commenter mes code sur les forums de developpez...

What ? Tu publies du code non portable ? En cas d'ambigüité j'utilise ' ou ` après le caractère à accentuer.
a`: à
e' : é
etc.

**stephl** · 03/09/2007, 10h43

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
#ifndef DBG
#error DBG must be defined to 0 or 1
#endif
 
#if DBG
#define PRINTF(a) a
#else
#define PRINTF(a)
#endif
 
 
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
   PRINTF(("purge\n"));
}

N'y aurait-il pas un problème au niveau de la définition de PRINTF() pour DBG valant 1 étant donné l'utilisation qui en est faite?

**Emmanuel Delahaye** · 03/09/2007, 10h46

Envoyé par _SamSoft_

Mise à jour :

Je vois que la leçon du conio ne t'as pas suffit...

On ne met pas de code dans un header
On inclue pas de .c

La compilation séparée se fait ainsi :

choisir un nom qui va bien (ici, je dirais 'saisie')
créer un fichier d'interface (header) appelé "saisie.h"
dans ce fichier, mettre
- une protection contre les inclusion multiples
- le nom du fichier /* saisie.h */
- les prototypes séparés des fonctions
- tout ce qui est nécessaire pour définir ces prototypes.
créer un fichier d'implémentation "saisie.c"
dans ce fichier mettre
- le nom du fichier /* saisie.c */
- un include de "saisie.h" en premier
- les définitions des fonctions
- tout ce qui est nécessaire à la définition des fonctions
inclure "saisie.h" dans tous les fichiers qui utilisent les fonctions concernées.
ajouter "saisie.c" au projet.

Détails ici :

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

(à lire de A à Z plusieurs fois si nécessaire. C'est dense...)

**Emmanuel Delahaye** · 03/09/2007, 10h48

Envoyé par stephl

N'y aurait-il pas un problème au niveau de la définition de PRINTF() pour DBG valant 1 étant donné l'utilisation qui en est faite?

Quel genre de problème ?

**_SamSoft_** · 03/09/2007, 10h50

Pour conio: j'utilise la version que vous m'avez fournit donc je ne vois pas où est le problème

Je vais lire ce que vous m'avez donné mais le fichier myHeader (je ne suis pas assez créatif pour trouver un bon nom) est tout à fait correct !

**stephl** · 03/09/2007, 10h51

Envoyé par Emmanuel Delahaye

Quel genre de problème ?

Si je ne m'abuse, avec:

Code :

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

#define PRINTF(a) a

La ligne:

Code :

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

PRINTF(("purge\n"));

devient:

Code :

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

("purge\n");

et je ne vois pas trop l'intérêt. J'aurais plutôt fait:

Code :

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

1
2
3
4
5
#if DBG
#define PRINTF(a) printf(a)
#else
#define PRINTF(a)
#endif

voire

Code :

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

1
2
3
4
5
#if DBG
#define PRINTF(a) printf("%s",(a))
#else
#define PRINTF(a)
#endif

**Emmanuel Delahaye** · 03/09/2007, 10h56

Envoyé par _SamSoft_

Pour conio: j'utilise la version que vous m'avez fournit donc je ne vois pas où est le problème

Donc, tu as remarqué qu'il y avait 2 fichiers : conio.h et conio.c. Là, il faut faire pareil. J'ai tout expliqué. Tu suis les explications à la lettre et ça marche. Si tu ne les comprends pas, pose des questions.

Je vais lire ce que vous m'avez donné mais le fichier myHeader (je ne suis pas assez créatif pour trouver un bon nom) est tout à fait correct !

Non. On ne peux pas faire sérieusement de la compilation séparée si il n'y a pas au moins 2 fichiers séparés : un .h pour l'interface, un .c pour l'implémentation.

Tu fais du C++, donc tu sais exactement de quoi je parle. (la définition des classes dans le .h et la définition des méthodes dans le .cpp correspondant).

**Emmanuel Delahaye** · 03/09/2007, 11h01

Envoyé par stephl

<...>J'aurais plutôt fait:

Code :

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

1
2
3
4
5
#if DBG
#define PRINTF(a) printf(a)
#else
#define PRINTF(a)
#endif

Euh, presque. En fait, c'est

Code :

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

1
2
3
4
5
#if DBG
#define PRINTF(a) printf a
#else
#define PRINTF(a)
#endif

Corrigé, merci.

voire

Code :

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

1
2
3
4
5
#if DBG
#define PRINTF(a) printf("%s",(a))
#else
#define PRINTF(a)
#endif

Euh, non...

Code :

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

PRINTF (("Count = %d\n", count));

ne fonctionnerait pas...

**stephl** · 03/09/2007, 11h06

Envoyé par Emmanuel Delahaye

Code :

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

PRINTF (("Count = %d\n", count));

ne fonctionnerait pas...

Oui, c'est juste. Désolé pour mon erreur, je n'avais songé qu'à une chaîne pour l'argument de PRINTF() mais rien n'interdit de passer plusieurs arguments de tout type avec votre définition (printf a).

**_SamSoft_** · 03/09/2007, 11h07

Envoyé par Emmanuel Delahaye

Donc, tu as remarqué qu'il y avait 2 fichiers : conio.h et conio.c. Là, il faut faire pareil. J'ai tout expliqué. Tu suis les explications à la lettre et ça marche. Si tu ne les comprends pas, pose des questions.
Non. On ne peux pas faire sérieusement de la compilation séparée si il n'y a pas au moins 2 fichiers séparés : un .h pour l'interface, un .c pour l'implémentation.

Tu fais du C++, donc tu sais exactement de quoi je parle. (la définition des classes dans le .h et la définition des méthodes dans le .cpp correspondant).

Ah ! En effet, je vois maintenant de quoi vous voulez parler

Je dois être hyper fatigué

(quand même, mettre des fonctions toutes faites dans un h

) Donc là je vais mettre les prototypes dans le fichier h et les fonctions dans le C. Mais je n'ai pas compris pourquoi je ne dois pas utiliser votre conio ?

**Thierry Chappuis** · 03/09/2007, 11h15

Envoyé par Emmanuel Delahaye

La boucle vide explicite (et développée) me va très bien

Code :

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

1
2
3
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
}

Elle permet la maintenance (trace...)

Code :

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

1
2
3
4
while ((c = fgetc(stdin)) != '\n' && c != EOF) 
{
   PRINTF(("purge\n"));
}

avec

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#ifndef DBG
#error DBG must be defined to 0 or 1
#endif
 
#if DBG
#define PRINTF(a) printf a
#else
#define PRINTF(a)
#endif

dans le header qui va bien (dbg.h, par exemple...)
What ? Tu publies du code non portable ? En cas d'ambigüité j'utilise ' ou ` après le caractère à accentuer.
a`: à
e' : é
etc.

Oui, je publie du code non portable (volontairement) sur ce forum. Je pense que la lisibilité et l'apprentissage y gagne. Mais j'essaierai de faire autrement à l'avenir (commentaires dans le texte et non dans le code).

Thierry

**Emmanuel Delahaye** · 03/09/2007, 11h22

Envoyé par _SamSoft_

Mais je n'ai pas compris pourquoi je ne dois pas utiliser votre conio ?

Bah, techniquement, tu peux, mais ce n'est pas portable...

Si tu y tiens, utilise PDCurses (qui est multi-plateformes), mais est-ce vraiment utile pour le moment (révisions du C de base...)

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

Je reprend : Emmanuel->Pourquoi ne dois-je pas utiliser votre conio

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
 
/*
  Name: fgets vs scanf
  Copyright: GPL
  Authors: Samy Hocine and some members 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 <string.h>
#include "myHeader.h"
int main(void)
{
  //Example with good functions
  char str[256], str2[256];
  puts("Example 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();
  system(CLEAR);
  //Example with a difficult functions reserved for good programmers
  puts("Example 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;
}
//So if you're a beginner, you should use fgets !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#ifndef MYHEADER_H
#define MYHEADER_H
#ifdef WIN32
   #define CLEAR "CLS"
#elif defined (linux)
   #define CLEAR "clear"
#else
   #error Not defined for this plateforme
#endif
void fclean(char *s_buffer, FILE *stream);
#endif //MYHEADER_H

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
 
#include <stdio.h>
#include <string.h>
#include "myHeader.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 é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)
            {
                  //Rien
            }
        }
    }
}

**Emmanuel Delahaye** · 03/09/2007, 11h29

Envoyé par _SamSoft_

Je reprend : Emmanuel->Pourquoi ne dois-je pas utiliser votre conio

C'est pas 'mon' conio. C'est juste l'ancien d'un contributeur de Dev-C++ que j'ai corrigé, c'est tout...

OK. Il ne reste plus qu'à créer la fonction pause() selon les indication déjà données...

Je te donne le prototype :

Code :

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

void pause (char const *s_message);

Si on passe NULL ou "" on n'affiche pas de message (évidemment)...

**_SamSoft_** · 03/09/2007, 11h45

Entendu pour conio.

Pour la fonction pause, je nage dans le néant :

Code :

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

1
2
3
4
5
6
7
8
 
void pause(char const *s_message)
{
     fprintf(stdout, "%c", *s_message);
     char c[256]="";
     fgets(c, sizeof c, stdin);
     fclean(c, stdout);
}

Merci d'avance

**Emmanuel Delahaye** · 03/09/2007, 12h00

Envoyé par _SamSoft_

Pour la fonction pause, je nage dans le néant :

Code :

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

1
2
3
4
5
6
7
8
9
 
void pause(char const *s_message)
{
     fprintf(stdout, "%c", *s_message);
 
     char c[256]="";
     fgets(c, sizeof c, stdin);
     fclean(c, stdout);
}

Pourquoi n'afficher que le premier caractère ?
Que se passe-t-il si s_message vaut NULL ?
Pourquoi 256 caractères pour saisir un simple <enter> ?
Pourquoi tu passes stdout à fclean() ?
Tu programmes toujours au hasard comme ça, ou ça t'arrive de réflechir avant de coder ?

**_SamSoft_** · 03/09/2007, 12h06

Désolé mais je débute (je viens de m'en rendre compte), je vais arranger tout ca

**_SamSoft_** · 03/09/2007, 13h41

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void pause(const char *s_message)
{
    char c[256];
    if (strlen(s_message) != 0)
        fprintf(stdout, "%s", s_message);
    else if(s_message==NULL)
         fprintf(stderr, "Erreur avec le pointeur");
    fflush(stdin);
    fgets(c, sizeof c, stdin);
    fclean(c, stdout);
}

Un ami m'a aidé pour la ligne

Code :

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

1
2
 
if (strlen(s_message) != 0)

Si je presse entrer c'est bon mais si j'appui sur une autre touche ca entre le caractère ce qui est tout à fait normal sachant que j'utilie fgets !
Comment puis-je régler cela ?

Merci d'avance

scanf vs fgets

C

Discussions similaires

Partager

Partager