[debutant] - Probleme de saisie avec fgets.

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [debutant] - Probleme de saisie avec fgets.

le fou

Forza Massallia

Bonsoir

J'ai un petit programme, pour repondre a un exos, qui permet de saisir une structure (nom, prenom, age, n° secu).
Bon,donc mon probleme vient de la saisie des ces valeurs.

La structure est declare en parametres general comme suit :

struct identity {
char nom[15];
char prenom[10];
int age;
int secu;
struct identity *next;
}st,tmp;

Bon, ensuite dans mon main, j'appele ma fonction saisie qui est :

void saisie (char *anu_secu)

{

FILE *fp;
char temp[10];

printf("Informations :\n" );

printf("==============\n" );

printf ("Nom\t\t : " );
fgets(st.nom, sizeof(st.nom), stdin);
fflush (stdout);

printf ("Prenom\t\t : " );

fgets(st.prenom, sizeof(st.prenom), stdin);

fflush(stdout);

printf ("Age\t\t : " );

fgets(temp, sizeof(temp), stdin);
st.age = strtol(temp, NULL, 10);

fflush(stdout);

printf ("Numero de secu\t : " );

fgets(temp, sizeof(temp), stdin);
st.secu = strtol(temp, NULL, 10);

fflush(stdout);
}

Alors le probleme vient du fait que, lorsque j'arrive sur cette fonction au lieu de demander le nom, attendre la saisie, et demander le prenom. J'ai un affichage comme suit :

Informations :
==============
Nom : Prenom :

En fait il affiche la ligne "prenom", alors que j'ai pas encore rentre le nom.

Ps : les librairies utilisés sont : stdio, stdlib, et ncurses.
Compilateur gcc-4.0.1 (linux)

Merci de votre aide

Message édité par le fou le 03-11-2005 à 23:53:55

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Publicité

Taz

bisounours-codeur

printf("..." );
fflush(stdout);
fgets( .... );

il ne serait peut être pas mauvais de vérifier que fgets ne renvoit pas NULL (auquel cas fin de fichier / erreur)

Message cité 1 fois

le fou

Forza Massallia

Taz a écrit :

printf("..." );
fflush(stdout);
fgets( .... );

il ne serait peut être pas mauvais de vérifier que fgets ne renvoit pas NULL (auquel cas fin de fichier / erreur)

Non, meme en mettant le fflush entre le printf et le fgets ca ne marche pas.

Ce que je ne comprends pas , c'est que si j'utilise un scanf a la place de fgets, je n'ai pas ce probleme la.

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Taz

bisounours-codeur

montre ton code corrigé.

le fou

Forza Massallia

Voila l'ensemble de mon code, si j'avais oublie une ligne, et que ce soit elle qui fasse planter :

Code :

#include <stdlib.h>
#include <stdio.h>
#include <ncurses.h>
struct identity {
char nom[15];
char prenom[10];
int age;
int secu;
}st,tmp;
void saisie (char *anu_secu)
{
FILE *fp;
char temp[10];
puts("Informations :" );
puts("==============" );
printf ("Nom\t\t : " );
fflush (stdout);
fgets(st.nom, sizeof st.nom, stdin);
printf ("Prenom\t\t : " );
fflush (stdout);
fgets(st.prenom, sizeof st.prenom, stdin);
printf ("Age\t\t : " );
fflush(stdout);
fgets(temp, sizeof(temp), stdin);
st.age = strtol(temp, NULL, 10);
printf ("Numero de secu\t : " );
fflush(stdout);
fgets(temp, sizeof(temp), stdin);
st.secu = strtol(temp, NULL, 10);
//ECRITURE DU NOM
puts ("ouverture du fichier en cours.." );
fp=fopen (anu_secu,"a+" );
puts("\nEcriture du nom dans le fichier" );
fprintf (fp, "%s",st.nom);
fprintf(fp,"\n" );
//ECRITURE DU PRENOM
puts("\nEcriture du nom dans le fichier" );
fprintf (fp, "%s",st.prenom);
fprintf(fp,"\n" );
// ECRITURE DE L AGE
puts("\nEcriture de l'age dans le fichier" );
fprintf (fp, "%d",st.age);
fprintf(fp,"\n" );
// ECRITURE DU NUMERO DE SECU
puts("\nEcriture du numero de secu dans le fichier" );
fprintf (fp, "%d",st.secu);
fprintf(fp,"\n" );
puts("\nFermeture du fichier" );
fclose(fp);
}
void visu (char *anu_secu)
{
FILE *fp;
printf("Nom desire\t: " );
scanf("%s",tmp.nom);
getchar();
puts ("ouverture du fichier en cours.." );
fp=fopen (anu_secu,"r+" );
fseek (fp,0,SEEK_SET);
//RECHERCHE
while (strcmp(tmp.nom,st.nom)!=0) {
fscanf(fp,"%[^\n]\n",st.nom);
fscanf (fp,"%[^\n]\n",tmp.prenom);
fscanf (fp,"%d[^\n]\n",&tmp.age);
fscanf (fp,"%d[^\n]\n",&tmp.secu);
}
//AFFICHAGE
system("clear" );
puts("Informations : " );
puts("===============" );
puts("\n" );
printf("Nom\t\t : %s\n",tmp.nom);
printf("Prenom\t\t : %s\n",tmp.prenom);
printf("Age\t\t : %d\n",tmp.age);
printf("Numero de secu\t : %d\n",tmp.secu);
getchar();
}
int main (void)
{
char *anu_secu="/home/beware/TPC/anu_secu.txt";
char *tmp="/home/beware/TPC/tmp.txt";
int c=0;
char temp[2];
while (c != 3)
{
system("clear" );
printf("\t\t{-------------------MENU-------------------}\n" );
printf("\t\t{ }\n" );
printf("\t\t{ }\n" );
printf("\t\t{ 1 - Saisir une nouvelle fiche }\n" );
printf("\t\t{ 2 - Visualiser les fiches }\n" );
printf("\t\t{ 3 - Quitter }\n" );
printf("\t\t{ }\n" );
printf("\t\t{------------------------------------------}\n" );
puts("\n\n\n" );
printf("Choix : " );
fflush(stdout);
fgets(temp, sizeof(temp), stdin);
c = strtol(temp, NULL, 10);
system("clear" );
if (c==1)
{
saisie(anu_secu);
}
else if (c==2)
{
visu(anu_secu);
}
}
}

Message édité par le fou le 04-11-2005 à 01:58:01

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Taz

bisounours-codeur

1) <ncruses.h> ne te sert à rien. manque <string.h>.
2) // commentaire C++, pas C
3) char *anu_secu = "/home/beware/TPC/anu_secu.txt"; mauvais
const char *anu_secu = "/home/beware/TPC/anu_secu.txt"; OK
4) t'as le droit de grouper tes prinfs

printf("ligne1\n"
"ligne2\n"
... );

5) et enfin :
fgets(temp, sizeof(temp), stdin);

bravo tu as lu 2 caractère. Un chiffre (espérons le) et un '\0' sont désormais dans temp. Et l'\n' il est passé où ? il est toujours dans le buffer ! Résultat, le prochain fgets il lit une ligne vide (ce qui lui convient très bien). Bon vu ton niveau, on a qu'à dire "agrandit temp" à 80caratères.

Message cité 2 fois

Taz

bisounours-codeur

ah oui : pourquoi des variables globales ...

Elmoricq

Modérateur

Taz a écrit :

2) // commentaire C++, pas C

// est un commentaire C valable en C99 (m'étais fait avoir aussi).

---------------
Blacker than a moonless night, hotter and more bitter than Hell itself... THAT is coffee.

le fou

Forza Massallia

Taz a écrit :

Mais quand j'appele ma fonction :
saisie(anu_secu);
Il me dit que le parametre anu_secu n'est pas bon.

Taz a écrit :

4) t'as le droit de grouper tes prinfs

printf("ligne1\n"
"ligne2\n"
... );

D'accord je savais pas qu'il etait possible de faire ca.
merci

Taz a écrit :

5) et enfin :
fgets(temp, sizeof(temp), stdin);

bravo tu as lu 2 caractère. Un chiffre (espérons le) et un '\0' sont désormais dans temp. Et l'\n' il est passé où ? il est toujours dans le buffer ! Résultat, le prochain fgets il lit une ligne vide (ce qui lui convient très bien). Bon vu ton niveau, on a qu'à dire "agrandit temp" à 80caratères.

Ah oui je comprends mieux pourquoi alors je sautais une saisie. En fait ormis, le cas du temp pas assez grand, je pensais naivement que le fflush vidait le buffer d'entree et donc empechait qu' un \n ne s'y trouve. Mais apparement c'est pas ca.

PS :
merci beaucoup pour ton aide.

La variable globale n'a rien a faire, il est vrai il faut que je l'integre dans mon main, et que je balance les pointeurs quand j'en ai besoin.

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

le fou

Forza Massallia

Lorsque je fais un fgets pour ma saisie au clavier, je capture donc le \n.

Y a til un moyen, ormis faire une boucle pour le detecter, de ne pas ecrire le \n dans le fichier?

Message cité 1 fois

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Publicité

Elmoricq

Modérateur

le fou a écrit :

Lorsque je fais un fgets pour ma saisie au clavier, je capture donc le \n.

Y a til un moyen, ormis faire une boucle pour le detecter, de ne pas ecrire le \n dans le fichier?

Oui, en utilisant par exemple strrchr() qui retourne un pointeur sur un caractère recherché dans une chaîne.

Message cité 1 fois

---------------
Blacker than a moonless night, hotter and more bitter than Hell itself... THAT is coffee.

le fou

Forza Massallia

Elmoricq a écrit :

Oui, en utilisant par exemple strrchr() qui retourne un pointeur sur un caractère recherché dans une chaîne.

Merci beaucoup
ca marche.

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

nlc

Le mieux est l'ennemi du bien

y'a plus simple :

char* chaine = "Coucou\n";

chaine[strlen(chaine)-1] = '\0'; // On raccourcit la chaine de 1 caractere pour virer le \n

Message cité 1 fois

---------------
char table[] = {112,114,105,110,116,102,40,34,37,99,37,99,37,99,34,44,49,49,48,44,49,48,56,44,57,57,41,59,0}; char* tablePtr = table; while(*tablePtr) printf( "%c",*tablePtr++ );

Taz

bisounours-codeur

...

nlc

Le mieux est l'ennemi du bien

Ca pourrait bugguer si la chaine etait vide, mais comme elle vient d'un fgets, on recupe forcement au moins un caractere, le \n

Donc tu peux faire :

fgets(st.prenom, sizeof st.prenom, stdin);
st.prenom[strlen(st.prenom)-1] = '\0'; // Vire le \n

Message cité 2 fois

---------------
char table[] = {112,114,105,110,116,102,40,34,37,99,37,99,37,99,34,44,49,49,48,44,49,48,56,44,57,57,41,59,0}; char* tablePtr = table; while(*tablePtr) printf( "%c",*tablePtr++ );

Elmoricq

Modérateur

nlc a écrit :

[:mlc]

man fgets :

Citation :

The fgets() function shall read bytes from stream into the array pointed to by s, until n-1 bytes are read, or a <newline> is read and transferred to s, or an end-of-file condition is encountered. The string is then terminated with a null byte.

Ta chaîne n'est pas obligatoirement terminée par un '\n'.

Tiens et puis pour rire, tape directement CTRL+C (ou CTRL+D sous Unix) à la saisie de ta chaîne, comme ça, pour rire. [:itm]

Message édité par Elmoricq le 04-11-2005 à 22:36:55

---------------
Blacker than a moonless night, hotter and more bitter than Hell itself... THAT is coffee.

nlc

Le mieux est l'ennemi du bien

Si on fait un control C sur un fgets( temp, sizeof(temp), stdin ) ca coupe completement le programme.

Ensuite, si le fgets a lu n-1 octets, c'est qu'il arrive en bout de buffer, donc il s'arrete pour se reserver un octet pour mettre le 0 de fin de chaine.

Ce que je voulais dire plus haut, c'est qu'il ne faut pas faire un st.prenom[strlen(st.prenom)-1] = '\0' sur une chaine vide, sinon ca ecrit hors de la zone st.prenom.

Mais avec un fgets( temp, sizeof(temp), stdin ), y'a peu de chance pour que la chaine temp fasse 0 !

Edit : Ah ! Autant pour moi, le Ctrl D renvoit bien 0 (si on a rien tapé avant), donc pour etre blindé faut faire un test.
Ou mieux, utiliser strchr() :whistle:

[:anathema]

Message édité par nlc le 05-11-2005 à 00:05:06

---------------
char table[] = {112,114,105,110,116,102,40,34,37,99,37,99,37,99,34,44,49,49,48,44,49,48,56,44,57,57,41,59,0}; char* tablePtr = table; while(*tablePtr) printf( "%c",*tablePtr++ );

Emmanuel Delahaye

C is a sharp tool

nlc a écrit :

y'a plus simple :

char* chaine = "Coucou\n";

chaine[strlen(chaine)-1] = '\0'; // On raccourcit la chaine de 1 caractere pour virer le \n

Non, une chaine n'est pas modifiable.

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

Emmanuel Delahaye

C is a sharp tool

nlc a écrit :

Non.

La seule méthode reconnue pour éliminer le '\n' est le search'n kill.

char *p = strchr(s, '\n');

if (p != NULL)
{
*p = 0;
}

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

le fou

Forza Massallia

bonsoir
est il possible de tester si le nombre de caractere rentre lors d'un fgets est compatible avec la capacité du tableau?

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Elmoricq

Modérateur

fgets() ne va pas plus loin que la taille maximum - 1 (pour le caractère '\0' terminal) que tu lui passes en paramètre (et il s'arrête avant s'il rencontre un saut de ligne, ou une fin de fichier).

C'est à toi de passer le bonne taille max.

Message cité 1 fois
Message édité par Elmoricq le 07-11-2005 à 23:22:04

---------------
Blacker than a moonless night, hotter and more bitter than Hell itself... THAT is coffee.

le fou

Forza Massallia

Elmoricq a écrit :

Oui mais dans le cas, d'une saisie au clavier, par exemple
L utilisateur ne sait pas forcement le nombre max de caractere qui lui est permis de rentrer.
Doit on alors, prevoir plus de place pour le stockage, ou peut ton tester la longueur de la chaine rentre, (par exemple en verifiant, si aucun /n n'apparait dans la chaine stocké?)

Message cité 1 fois

---------------
Celui qui sauve une vie, sauve l'humanité (Le Talmud) - Personne n'a plus grand amour que celui de donner sa vie pour ses amis (Jean XV, 13) - Ma taniere ubuntiste

Emmanuel Delahaye

C is a sharp tool

le fou a écrit :

Si il est bien utilisé, fgets() ne permettra pas le débordement. Si une tentative de débordement a eu lieu, le '\n' n'est pas présent dans la chaine. Les caractères nons lus seront encore à disposition dans stdin. Il suffit de rappeler fgets() pour lire la suite. On peut aussi les ignorer en bouclant sur stdin avec fgetc(), par exemple, jusquà detection de '\n' ou de EOF.

http://mapage.noos.fr/emdel/notes.htm#saisie
http://mapage.noos.fr/emdel/notes.htm#fichiers

A l'aide d'une combinaison astucieuse de realloc(), de fgets() (ou de fgetc()) et d'une boucle, on peut saisir une ligne de longueur arbitraire.

Bref, il y a plusieurs possibilités. Il est conseillé de les recenser et d'écrire les fonctions correspondantes une bonne fois pour toutes.

http://mapage.noos.fr/emdel/clib.htm
Module IO

Message édité par Emmanuel Delahaye le 08-11-2005 à 08:12:10

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

Publicité

FORUM HardWare.fr

Programmation

[debutant] - Probleme de saisie avec fgets.

Sujets relatifs
[php] probleme avec ecriture de fichier	probleme avec un code
problème de session en php	Problème traduction avec ressource hacker
[debutant] [swing] bulle d'aide à la eclipse	[ACCESS] probleme cumul sur requete
Problème de vitesse d'envoi de données par socket	probleme pour débuter java
Problème de forumaire d'inscription	Petit problème de FTP
Plus de sujets relatifs à : [debutant] - Probleme de saisie avec fgets.

Page générée en 0.483 secondes