scanf marche et pas fgets Résolu/Fermé

Question

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?

fiddy · Answer

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

rocksider · Answer

pour l'esperluette  c fait je l'ai enlevé, pour l'espace i reste toujours, ça voudrai toujours dire qui y'aurai le '
' 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

fiddy · Answer

Si tu utilises fgets,  et que tu veux enlever le '
' tu utilises, strchr pour avoir un pointeur sur le 
 et le transformer en \0.
Et pour le fflush(stdin), utilise plutôtint c; while((c=fgetc())!='
' &&  c!=EOF);
Cdlt

rocksider · Answer

mercii pour ces précieux info, donc on scanf est plus pratique ?

rocksider · Answer

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())!='
' &&  c!=EOF);

rocksider · Answer

l'interruption Ctrl+D ? dsl ...

rocksider · Answer

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

fiddy · Answer

Ah oui, désolé, j'ai fait ça de tête. fgetc(stdin) ou alors getchar() ;)
char *p=strchr(chaine,'
');
if(p!=NULL) *p='\0';
Cdlt

rocksider · Answer

merci beaucoup pour ces précieuse info, merci infiniment ^^

Scanf marche et pas fgets

9 réponses

Discussions similaires