scanf vs fgets

Citation:
Envoyé par _SamSoft_
Code:

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:

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:

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

Citation:

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) ?

Code:

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:

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.

Citation:
Envoyé par _SamSoft_
Code:

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:

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.

Citation:

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 ?

Citation:

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

Pourtant signalée...

Citation:

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 :oops: 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 !

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.

Comme ceci ? :

Code:

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

Citation:

Envoyé par _SamSoft_

Pour le problème de NULL, pouvez vous me donner un exemple :oops: 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...

Citation:

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é).

Citation:

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.

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

Citation:

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...

Citation:

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

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 :D] ) alors dans le post 41 je vois mon code ainsi que cela :

Code:

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:

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

Citation:

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:

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:

fflush(stdout);

est indispensable ici.

Je recommande ceci
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 #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...

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 ! 8O

Citation:

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 ! 8O

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

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 ?

Merci pour votre code (plus lisible que le miens :D)

Mais je crois que vous avez oublié un else :
Code:

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

Citation:

Envoyé par _SamSoft_

Merci pour votre code (plus lisible que le miens :D)

Mais je crois que vous avez oublié un else :

Non.

Citation:

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.)

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:

Citation:

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):

Citation:

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

Citation:

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 :roll:), fflush(stdout) permet de vider le buffer de sortie, donc ce n'est pas un fflush(stdout) qui empeche s_message de s'afficher.