Scanf marche et pas fgets

Résolu/Fermé
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 - 29 nov. 2008 à 21:26
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 - 1 déc. 2008 à 02:16
Bonjour,
dernierement on m'a conseillé de travailler avec fgets à la place de scanf pour + de securité, mais malheuresement il marche pas dans des fcts que j'ai créé comme par exemple là :
void chgpass()
{
char pass[15],pass1[15],pass2[15];
printf("Donner votre ancien mot de passe:");
//fgets(pass,sizeof(pass),stdin);
scanf("%s",&pass);
fflush(stdin);
if(!strcmp(user[actif].pass,pass))
{ ...}
else
{...}

donc quand je travaille avec scanf, ça entre dans la condition, et avec fgets il me renvoi tjr dans le else ...
est ce que je devrai abondonné fgets même si il est meilleur ? l'erreur peu provenir de quoi?
A voir également:

9 réponses

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
29 nov. 2008 à 21:32
Salut,
Plusieurs erreurs dans ton code.
scanf("%s",&pass); Il faut enlever l'esperluette (&).
Ensuite, le fgets marche très bien, mais il ne faut pas oublier que le '\n' va se loger dans la chaîne aussi s'il y a de la place.
Et enfin, il ne faut pas utiliser fflush(stdin). Cela ne marche pas forcément.. Il vaut mieux vider le buffer à coup de fgetc.
Cdlt
0
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
29 nov. 2008 à 21:37
pour l'esperluette c fait je l'ai enlevé, pour l'espace i reste toujours, ça voudrai toujours dire qui y'aurai le '\n' apres le pseudo?
sinon pour le fflush(stdin), j'ai beau chercher à un truc meilleur,d'ailleurs j'avait trouvé le fameux fgetc, mais j'ai pas bien saisis
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
29 nov. 2008 à 21:41
Si tu utilises fgets, et que tu veux enlever le '\n' tu utilises, strchr pour avoir un pointeur sur le \n et le transformer en \0.
Et pour le fflush(stdin), utilise plutôt
int c; while((c=fgetc())!='\n' &&  c!=EOF);

Cdlt
0
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
29 nov. 2008 à 21:51
mercii pour ces précieux info, donc on scanf est plus pratique ?
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
29 nov. 2008 à 21:53
scanf est assez dangereux en fait, surtout de la façon dont tu l'utilises.
Reprends ton programme avec ton scanf. Et rentre une longue chaîne de caractère. Et admire le résultat ;)
Cdlt
0

Vous n’avez pas trouvé la réponse que vous recherchez ?

Posez votre question
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
29 nov. 2008 à 21:59
merci je vais continué avec les fgets alors, et svp (dsl pour le derangement dejà) vous pouvez m'expliquer ce que fait cette boucle ? EOF c'est quoi?
while((c=fgetc())!='\n' && c!=EOF);
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
29 nov. 2008 à 22:00
La boucle permet de lire tous les caractères dans le buffer clavier jusqu'au \n final ou l'interruption Ctrl+D ;)
0
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
29 nov. 2008 à 22:03
l'interruption Ctrl+D ? dsl ...
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
29 nov. 2008 à 22:12
C'est pour mettre fin à la ligne directement avec une séquence d'échappement.
Mais, t'inquiète ce n'est pas important. ;)
EOF c'est End Of File. Donc tu le vois notamment en fin de fichiers (caractère final). Eh bien t'imagines que ce caractère vient dans le buffer (grosso modo).
Cdlt
0
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
30 nov. 2008 à 00:31
mercii pour m'avoir eclairci le truc ^^, sinon apres changer les fflush par la ligne que vous m'avez passé il me donne 1 erreur. j'utilise dev-c++ , j code avec du c.
Two few arguments to function fgetc.

et puis j'ai pas su utilisé strchr ...
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
30 nov. 2008 à 11:32
Ah oui, désolé, j'ai fait ça de tête. fgetc(stdin) ou alors getchar() ;)
char *p=strchr(chaine,'\n');
if(p!=NULL) *p='\0';

Cdlt
0
rocksider Messages postés 106 Date d'inscription mardi 21 octobre 2008 Statut Membre Dernière intervention 16 mars 2012 9
1 déc. 2008 à 02:16
merci beaucoup pour ces précieuse info, merci infiniment ^^
0