Les Allergies
Alimentaires
Posez votre question Signaler

Scanf marche et pas fgets [Résolu]

rocksider 106Messages postés 21 octobre 2008Date d'inscription 16 mars 2012Dernière intervention - Dernière réponse le 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?
Lire la suite 

Scanf marche et pas fgets »

12 réponses
Réponse
+0
moins plus
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
Ajouter un commentaire
Réponse
+0
moins plus
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
Ajouter un commentaire
Réponse
+0
moins plus
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
Ajouter un commentaire
Réponse
+0
moins plus
mercii pour ces précieux info, donc on scanf est plus pratique ?
fiddy- 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
Ajouter un commentaire
Réponse
+0
moins plus
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);
fiddy- 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 ;)
Ajouter un commentaire
Réponse
+0
moins plus
l'interruption Ctrl+D ? dsl ...
fiddy- 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
Ajouter un commentaire
Réponse
+0
moins plus
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 ...
Ajouter un commentaire
Réponse
+0
moins plus
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
Ajouter un commentaire
Réponse
+0
moins plus
merci beaucoup pour ces précieuse info, merci infiniment ^^
Ajouter un commentaire
Ce document intitulé « scanf marche et pas fgets » issu de CommentCaMarche (www.commentcamarche.net) est mis à disposition sous les termes de la licence Creative Commons. Vous pouvez copier, modifier des copies de cette page, dans les conditions fixées par la licence, tant que cette note apparaît clairement.
Dossier à la une
Passage au tout numérique : quel coût pour les particuliers ?