scanf vs fgets

**Thierry Chappuis** · 03/09/2007, 13h49

Envoyé par _SamSoft_

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

Peut-être pourrais-tu commencer la fonction avec quelque chose comme:

Code :

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

1
2
3
4
if (s_message != NULL)
{
   /* SUite du code */
}

De plus, on n'utilise jamais fflush() sur stdin (comportement indéfini), mais ici, je pense que c'est une erreur. Finalement, pourquoi ton tampon contient-il 256 caractères (c'est beaucoup pour n'en saisir qu'un seul).

Thierry

**Emmanuel Delahaye** · 03/09/2007, 14h02

Envoyé par _SamSoft_

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 ?

Tu ne peux pas empêcher l'utilisateur de taper ce qu'il veut... Ca n'a aucune importance.

J'ai toujours des questions non résolues :

Que se passe-t-il si s_message vaut NULL ?
Pourquoi 256 caractères pour saisir un simple <enter> ?
Pourquoi tu passes stdout à fclean() ?

et des nouvelles questions

D'où sors-tu ce fflush(stdin) ?

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

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[2];
    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, stdin);
}

Pour s_message et NULL :

Code :

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

1
2
 
else if(s_message==NULL)

Pour fflush, si je ne le met pas, le texte contenu dans s_message ne s'affiche pas !

Pour stdout à fclean, c'est une erreur de ma part, je n'avais pas vu.

**Emmanuel Delahaye** · 03/09/2007, 14h13

Envoyé par _SamSoft_

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[2];
    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, stdin);
}

Pour s_message et NULL :

Code :

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

1
2
 
else if(s_message==NULL)

Pour la 3ème fois, je repose la question :

Que se passe-t-il si on passe NULL à la fonction pause() ?

J'aimerais bien une réponse détaillée.

Pour fflush, si je ne le met pas, le texte contenu dans s_message ne s'affiche pas !

Si c'est ça le problème, c'est fflush(stdout) qu'il faut mettre... Tu confonds les deux ou quoi ?

Pour stdout à fclean, c'est une erreur de ma part, je n'avais pas vu.

Pourtant signalée...

**_SamSoft_** · 03/09/2007, 14h20

Envoyé par Emmanuel Delahaye

Pour la 3ème fois, je repose la question :

Que se passe-t-il si on passe NULL à la fonction pause() ?

J'aimerais bien une réponse détaillée.

Si c'est ça le problème, c'est fflush(stdout) qu'il faut mettre... Tu confonds les deux ou quoi ?
Pourtant signalée...

Pour le problème de NULL, pouvez vous me donner un exemple

J'ai pas compris.

Pour flush, j'ai bien mis stdout mais ca ne marchait toujours pas alors j'ai mis stdin et là ca fonctionne !

**Skyounet** · 03/09/2007, 14h23

Ben NULL à strlen ça plante je pense.

Il faut que tu inverses ton if, tu fais d'abord le strlen et ensuite tu testes si c'est = NULL.

**_SamSoft_** · 03/09/2007, 14h30

Comme ceci ? :

Code :

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

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

**Emmanuel Delahaye** · 03/09/2007, 14h35

Envoyé par _SamSoft_

Pour le problème de NULL, pouvez vous me donner un exemple

J'ai pas compris.

C'est simple. Si c_message vaut NULL, c'est cette valeur que tu passes à strlen(). Or passer NULL à strlen(), comme l'indique sa documentation, provoque un comportement indéfini. Tout peut arriver.

D'autre part, je n'ai pas demandé qu'un message soit affiché en cas de NULL, mais simplement que rien ne soit affiché du tout. Il faut donc revoir ton algorithme en conséquence.

C'est assez basique...

Pour flush, j'ai bien mis stdout mais ca ne marchait toujours pas alors j'ai mis stdin et là ca fonctionne !

Tss... La programmation, c'est pas du hasard. Il est certain que ça doit fonctionner avec fflush(stdout). Si ce n'est pas le cas, c'est qu'il y a un autre problème.

D'autre part, fflush(stdin) n'existe pas (comportement indéfini comme déjà signalé).

**Emmanuel Delahaye** · 03/09/2007, 14h37

Envoyé par _SamSoft_

Comme ceci ? :

Non. Réfléchis avant de coder ou de poser une question. On est pas sur un chat ici. On se pose, et on prend du temps...

Personne ne code au hasard. Tout a une logique parfaitement claire.

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

Je réflechis ! Mais je ne vois vraiment pas !

**Emmanuel Delahaye** · 03/09/2007, 18h58

Envoyé par _SamSoft_

Je réflechis ! Mais je ne vois vraiment pas !

C'est pourtant simple. Pour que le message soit affiché, il faut respecter 2 conditions :

- Le pointeur n'est pas NULL
- La chaine a une longueur non nulle.

On t'a donné l'idée de tester la longueur de la chaine avec strlen(), OK. Problème, si on passe un NULL à strlen(), pour faire simple, ça plante.

Donc il faut traiter les conditions dans un ordre bien précis qui fait que quand on teste la longueur, on est sûr que le pointeur n'est pas NULL.

A toi de jouer.

Ce petit problème est très simple... Je ne sais pas ce que tu codes d'habitude, mais là, franchement c'est très facile...

**Thierry Chappuis** · 03/09/2007, 19h23

Envoyé par _SamSoft_

Je réflechis ! Mais je ne vois vraiment pas !

Tu as un début de réponse dans le post 41...

Thierry

**_SamSoft_** · 05/09/2007, 12h35

Merci (désolé pour l'attente, je viens de reprendre le lycée [et la prog n'est plus ma priorité

mais c'est mes études

] ) alors dans le post 41 je vois mon code ainsi que cela :

Code :

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

1
2
3
4
5
if (s_message != NULL)
{
   /* SUite du code */
}

Or dans la nouvelle version de mon code j'ai cette ligne, donc je me demande bien ce que vous voulez ?!

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
 
void pause(const char *s_message)
{
    char c[2];
    if(s_message!=NULL)
    {
                        if (strlen(s_message)!=0)
                        {
                                                 fprintf(stdout, "%s", s_message);
                        }
    fflush(stdin); //If I write “stdout”, s_message is not visible !
    fgets(c, sizeof c, stdin);
    fclean(c, stdin);
    }
}

**Emmanuel Delahaye** · 05/09/2007, 12h48

Envoyé par _SamSoft_

Or dans la nouvelle version de mon code j'ai cette ligne, donc je me demande bien ce que vous voulez ?!

A ma connaissance, c'est la première fois que tu publies cette version. Alors les remarques dans le genre "donc je me demande bien ce que vous voulez ?!", tu te les garde...

Quand à ça,

Code :

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

    fflush(stdin); //If I write “stdout”, s_message is not visible !

C'est n'importe quoi. et il faut le retirer.

Par contre, j'insiste,

Code :

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

    fflush(stdout);

est indispensable ici.

Je recommande 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
 
#include <stdio.h>
#include <stdlib.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 é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;
         }
      }
   }
}
 
void pause (const char *s_message)
{
   if (s_message != NULL && strlen (s_message) != 0)
   {
      fprintf (stdout, "%s", s_message);
      fflush (stdout);          //If I write “stdout”, s_message is not visible !
   }
   {
      char c[2];
      fgets (c, sizeof c, stdin);
      fclean (c, stdin);
   }
}
 
int main (void)
{
   pause (NULL);
   pause ("");
   pause ("Appuyer sur une touche pour continuer");
   return 0;
}

selon les consignes publiées ici :

http://www.developpez.net/forums/sho...6&postcount=36

et que tu as du mal à suivre visiblement...

**_SamSoft_** · 05/09/2007, 13h07

Je m'excuse

Je vais voir votre code.

PS: Pour fflush, je continu de le dire, chez moi si je met fflush(stdout); je vois pas le contenu de s_message à l'écran !

**Emmanuel Delahaye** · 05/09/2007, 13h12

Envoyé par _SamSoft_

Je m'excuse

Je vais voir votre code.

PS: Pour fflush, je continu de le dire, chez moi si je met fflush(stdout); je vois pas le contenu de s_message à l'écran !

C'est pas possible. Tu as essayé avec mon code ? Tu dois obtenir ceci :

Code :

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

1
2
3
4
 
 
 
Appuyer sur une touche pour continuer

(2 pauses 'muettes' et une pause 'bavarde').

Quel est ton système ?
Quel est ton environnement de développement ?

**_SamSoft_** · 05/09/2007, 13h16

Merci pour votre code (plus lisible que le miens

)

Mais je crois que vous avez oublié un else :

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
 
void pause (const char *s_message)
{
   if (s_message != NULL && strlen (s_message) != 0)
   {
      fprintf (stdout, "%s", s_message);
      fflush (stdout);          //If I write “stdout”, s_message is not visible !
   }
/* Here */
   {
      char c[2];
      fgets (c, sizeof c, stdin);
      fclean (c, stdin);
   }
}

Sinon toujours rien (même après une reconstruction) sauf si je met fflush(stdin);

Windows Vista Home Premium (achetté il y a 2mois)
J'utilise DevCpp 4.9.9.2

**Emmanuel Delahaye** · 05/09/2007, 13h36

Envoyé par _SamSoft_

Merci pour votre code (plus lisible que le miens

)

Mais je crois que vous avez oublié un else :

Non.

Sinon toujours rien (même après une reconstruction) sauf si je met fflush(stdin);

Windows Vista Home Premium (achetté il y a 2mois)
J'utilise DevCpp 4.9.9.2

OK.

Tu peux faire un copié-collé de l'écran de sortie ? (click droit etc.)

**_SamSoft_** · 05/09/2007, 13h46

Entendu.

Je ne comprend pas, vous voulez que je fasse un copier-coller ? A quoi cela sert-il ? De plus si je met stdout ca se passe comme ceci:

Avec fgets
str=
montexte
/me presse entrer
str2=
montexte
/me presse entrer
str=str2
Appuyez sur une touche pour continuer...
/me presse entrer
Efface Ecran
avec scanf
str=
montexte
/me presse entrer
str2=
montexte
/me presse entrer
/prog Fermeture

Avec stdin dans fflush j'ai (lors du deuxième exemple):

avec scanf
str=
montexte
/me presse entrer
str2=
montexte
/me presse entrer
Appuyez sur une touche pour continuer...
/me presse entrer
/prog Fermeture

**Skyounet** · 05/09/2007, 13h56

Envoyé par _SamSoft_

Mais je crois que vous avez oublié un else :

En fait il a juste déclaré un nouveau bloc pour faire un petit traitement du coup dans ce bloc on peut déclarer de nouvelles variables valable juste dans ce bloc, c'est assez pratique.

Je plussoie, fflush(stdin) ne fait rien (sauf sous Windows

), fflush(stdout) permet de vider le buffer de sortie, donc ce n'est pas un fflush(stdout) qui empeche s_message de s'afficher.

scanf vs fgets

C

Discussions similaires

Partager

Partager