Probleme fgets

Probleme fgets - C - Programmation

Marsh Posté le 10-12-2004 à 00:28:47    

Bonjour, j'ai un probleme d'affichage avec fgets. J'essaie de recuperer une chaine de caracteres sur la sortie standard:
 

Code :
  1. int
  2. main()
  3. {
  4. int choix;
  5. printf("numero?" );
  6. fflush(stdout);
  7. scanf("%d",&choix);
  8. mafonction();
  9. }
  11. int
  12. get_line(char *buf, size_t size)
  13. {
  14.    int ret = 0;
  15.    if (fgets(buf, size, stdin) != NULL){
  16.       char *p = strchr(buf, '\n');
  17.       if (p != NULL){
  18.          *p = 0;
  19.          ret = 1;
  20.       }
  21.    }
  22.    return ret;
  23. }
  25. void
  26. mafonction()
  27. {
  28.    char *c;
  29.    c = (char *)malloc( TAILLE*sizeof(char) );
  31.    printf("Entrez une chaine de caracteres :" );
  32.    fflush(stdout);
  33.    if ( get_line(c, TAILLE) ){
  34.       printf("result=%s\n",c);
  35.       fflush(stdout);
  36.    }
  37.    else printf("Erreur!" );
  38.    
  39.    if ( c != NULL ){
  40.       printf("Liberation c\n" );
  41.       fflush(stdout);
  42.       free(c);
  43.    }
  44. }


 
Si dans ma fonction main(), je ne mets pas les 3 lignes printf,fflush et scanf avant l'appel de mafonction(), TOUT SE PASSE BIEN!
 
Dans le cas contraire, il passe directement toute la fonction sans me laisser ecrire, ce qui donne sur la sortie standard :  
"Entrez une chaine de caracteres :result="
 
De quoi cela vient il ?
 
Merci d'avance pour vos reponses..  

Reply

Marsh Posté le 10-12-2004 à 00:28:47   

Reply

Marsh Posté le 10-12-2004 à 00:58:31    

traduit en C sans se prendre le choux ton code se traduit par :

Code :
  1. #define _GNU_SOURCE
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <string.h>
  7. int main(void){
  8.   char * chaine;
  9.   size_t taille;
  10.   ssize_t retour;
  11.   int choix;
  13.   taille = 0;
  14.   chaine = NULL;
  16.   fprintf(stdout, "numero? : " );
  18.   if((retour = getline(&chaine, &taille, stdin)) == -1)
  19.     exit(1);
  21.   if((sscanf(chaine,"%d", &choix) != 1))
  22.     fprintf(stdout, "Erreur\n" );
  24.   free(chaine);
  26.   taille = 0;
  27.   chaine = NULL;
  29.   fprintf(stdout, "Entrez une chaine de caracteres : " );
  31.   if((retour = getline(&chaine, &taille, stdin)) == -1)
  32.     exit(1);
  34.   fprintf(stdout,"result=%s\n",chaine);
  36.   free(chaine);
  38.   return 0;
  39. }


Bon ok je triche en utilisant getline qui est GNU mais quand meme tu n'as pas un peu l'impression de te compliquer la vie avec scanf (qui est une verrue), fflush (??) etc...


Message édité par manatane le 10-12-2004 à 00:58:50
Reply

Marsh Posté le 10-12-2004 à 01:03:57    

Oui mais en fait j'ai absolument besoin de garder des fonctions en dehors de main().
 
Bon je vais essayer de changer ma ligne scanf.
 
Pour le fflush j'etais vraiment désespéée alors j'essyais un peu tout ;)

Reply

Marsh Posté le 10-12-2004 à 01:09:06    

en fait, c'est simple : quand tu lis en entier avec scanf. Toi tu tappes "69\n"  et scanf ne lis que le '6' et le '9'. Donc il reste des choses dans ton buffer de lecture.
 
 
Donc deux solutions simples:
- soit tu vides ce buffer jusqu'à l' '\n' -> http://www.isty-info.uvsq.fr/~rume [...] 4.html#q_5
- soit tu fais comme manatane, mais pas la peine d'utiliser getline. Tu lis une ligne avec fgets, et après tu l'analyse avec sscanf (même fonctionnement que scanf, sauf que le premier argument est la chaine contenant les données.

Reply

Marsh Posté le 10-12-2004 à 01:12:41    

ALors la merci beaucoup, je vais essayer tout de suite..
 
C'est quand même pas evident le C ;)

Reply

Marsh Posté le 10-12-2004 à 08:52:17    

myeve a écrit :

Bonjour, j'ai un probleme d'affichage avec fgets. J'essaie de recuperer une chaine de caracteres sur la sortie standard:
<...>
 
Si dans ma fonction main(), je ne mets pas les 3 lignes printf,fflush et scanf avant l'appel de mafonction(), TOUT SE PASSE BIEN!
 
Dans le cas contraire, il passe directement toute la fonction sans me laisser ecrire, ce qui donne sur la sortie standard :  
"Entrez une chaine de caracteres :result="
 
De quoi cela vient il ?
 
Merci d'avance pour vos reponses..


Déjà, ça, ça n'aide pas (je suppose qu'il s'agit d'un oubli).


Compiling MAIN.C:
Warning MAIN.C 5: Call to function 'printf' with no prototype
Error MAIN.C 6: Undefined symbol 'stdout'
Warning MAIN.C 6: Call to function 'fflush' with no prototype
Warning MAIN.C 7: Call to function 'scanf' with no prototype
Warning MAIN.C 8: Call to function 'mafonction' with no prototype
Warning MAIN.C 9: Function should return a value
Error MAIN.C 12: ) expected


Pour le reste, tu as écris une fonction get_line() qui est intéressante (je crois en reconnaître l'origine...héhé...), mais incomplète. En effet, elle ne traite pas le cas où la saisie est plus longue que la chaine. Dans ce cas, un '\n' traine dans stdin.
 
D'autre part, tu utilises scanf()  qui systématiquement laisse trainer un '\n' dans stdin. (fonction d'usage difficile peu recommandée)
 
http://mapage.noos.fr/emdel/notes.htm#saisie
 
ton utilisation de malloc() est compliquée :
 
http://mapage.noos.fr/emdel/notes.htm#malloc
 
et pas forcément justifiée ici (mais je suppose que c'est un exercice, au fait TAILLE n'est pas définie). De plus, il manque l'essentiel, à savoir le test de la valeur retournée par malloc() avant son utilisation.
 
sizeof(char) vaut 1 par définition.
 
Il est inutile de mettre fflush(stdout) si le texte envoyé à stdout est une ligne complète.
 
http://mapage.noos.fr/emdel/notes.htm#fflush_stdout
 
Voici ton code mis au point. Pose des questions si tu ne comprends pas.

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  5. int get_line (char *buf, size_t size)
  6. {
  7.    int ret = 0;
  8.    if (fgets (buf, size, stdin) != NULL)
  9.    {
  10.       char *p = strchr (buf, '\n');
  11.       if (p != NULL)
  12.       {
  13.          *p = 0;
  14.          ret = 1;
  15.       }
  16.       else
  17.       {
  18.          int c;
  20.          /* vider stdin proprement (ignorer les caracteres non lus) */
  21.          while ((c = getchar ()) != '\n' && c != EOF)
  22.          {
  23.          }
  24.       }
  25.    }
  26.    return ret;
  27. }
  29. void mafonction (void)
  30. {
  31.    size_t const TAILLE = 128;
  32.    char *c = malloc (TAILLE);
  34.    if (c != NULL)
  35.    {
  36.       printf ("Entrez une chaine de caracteres : " );
  37.       fflush (stdout);
  39.       if (get_line (c, TAILLE))
  40.       {
  41.          printf ("result=%s\n", c);
  42.       }
  43.       else
  44.       {
  45.          printf ("Erreur!\n" );
  46.       }
  48.       printf ("Liberation c\n" );
  49.       free (c);
  50.    }
  51. }
  53. int main ()
  54. {
  55.    int choix;
  57.    printf ("numero : " );
  58.    fflush (stdout);
  59.    {
  60.       char s[32];
  62.       get_line (s, sizeof s);
  63.       choix = (int) strtol (s, NULL, 10);
  64.    }
  66.    mafonction ();
  68.    return 0;
  69. }


Message édité par Emmanuel Delahaye le 10-12-2004 à 08:58:14

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 10-12-2004 à 09:53:42    

Emmanuel Delahaye a écrit :

Déjà, ça, ça n'aide pas (je suppose qu'il s'agit d'un oubli).

tu te souviens de ce que je t'ai dit à propos des devoirs

Reply

Marsh Posté le 10-12-2004 à 12:12:23    

Taz a écrit :

tu te souviens de ce que je t'ai dit à propos des devoirs


Oui, mais la bonne volonté du questionneur est évidente. C'est pas comme quand il poste 0 ligne de code en demandant de tout faire... Mais bon, si ce forum est strict à ce point, je vais me conformer... C'est frustrant...


Message édité par Emmanuel Delahaye le 10-12-2004 à 12:13:17

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 10-12-2004 à 14:34:35    

ce qu'il faut que tu vois bien, c'est que y a une fonction recherche sur le forum : y a des tas de topics remplis de bonnes choses. Et pour getline, la question est là toutes les semaines. Donc on essaie aussi de diminuer le rapport signal/bruit pour pas avoir 36 sujets sur la même chose. Les devoirs, attend un peu la rentrée de janvier : faut être ferme, sinon y a de l'abus :)

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed