fonctions fgets et gets

Version imprimable

Bonjour,

Je dois créer une fonction qui pourra venir s'inclure dans un programme précédemment codé .

Je voulais savoir si j'utilise bien la fonction fgets et si gets(t) n'aurait pas été préférable ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 /*description du travail*/ /*morceau de code à inclure au programme pour utiliser*/ /*la fonction lecture (include,variable...)*/ #include<ctype.h> /*déclaration de variable*/ char t[30]; /*prototype de fonction*/ void lecture(char t[]); /*fonction lecture*/ void lecture(char t[]) { int i; fgets(t,30,fichier); for(i=0;t[i]!='\0';i++) { if(isupper(t[i]) { t[i]=tolower(t[i]); } } }

03/07/2009, 16h19
Médinoc

gets() n'est jamais préférable.

De plus, la valeur 30, tu devrais la passer en paramètre à ta fonction lecture.
03/07/2009, 16h20
ram-0000

gets() ne devrait plus être utilisée.

Elle est buggé dans le sens où elle ne permet pas de spécifier la taille du buffer de lecture. D'ailleurs les quelques compilateurs que je connais émettent un warning lorsque on l'utilise.

Il faut lui préférer la fonction fgets()
03/07/2009, 16h25
gl
Citation:

Envoyé par chicabonux

Je dois créer une fonction qui pourra venir s'inclure dans un programme précédemment codé .

Je voulais savoir si j'utilise bien la fonction fgets

J'aurais juste deux remarques:
- Pourquoi avoir utilisé 30 en dur dans l'appel à fgets() ? Il est préférable de déterminer la taille réelle grâce à sizeof, qui s'adaptera en cas de changement de la taille de t sans risque d'oubli.
- La fonction fgets() lit également le caractère \n, il peut s'avérer utile de vérifier la présence de ce caractère dans la chaîne lue (pour supprimer ce caractère, pour détecter une lecture partielle, pour purger le buffer, etc.)
Citation:

Envoyé par chicabonux

et si gets(t) n'aurait pas été préférable ?

Certainement pas, gets() ne permet pas de vérifier la taille de la chaîne lue, tu as donc un risque important de débordement de buffer.

Je devrais faire comme ça avec size_of et pour enlever le caractère \n ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
/*description du travail*/
/*morceau de code à inclure au programme pour utiliser*/
/*la fonction lecture (include,variable...)*/
 
#include<ctype.h>
 
/*déclaration de variable*/
char t[30];
 
/*prototype de fonction*/
void lecture(char t[30]);
 
/*fonction lecture*/
void lecture(char t[30])
{
	int i;
 
	fgets(t,sizeof,fichier);
	for(i=0;t[i]!='\0';i++)
	{
	    if(t[i]!='\n')
	    {
		   if(isupper(t[i])
		    {
			  t[i]=tolower(t[i]);
		    }
	    }
	    else i++;
    }
}

Parce que sinon pour détecter une lecture partielle ou pour purger le buffer , je vois pas comment faire ?

La méthode classique, c'est ça:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 void fpurge(FILE *pIn) { int c; while((c=fgetc(pIn))!='\n' && c!=EOF) {} } void fgetspurge(char *buf, size_t taille, FILE *pIn) { char *pLF; fgets(buf, taille, pIn); pLF = strchr(buf, '\n'); if(pLF != NULL) *pLF = '\0'; else fpurge(pIn); }
Ce code lit jusqu'à capacité du buffer, supprime le \n en cas de remplissage partiel, et vide le buffer clavier en cas de remplissage total.

03/07/2009, 18h10
chicabonux

Comment faire pour insérer ces fonctions dans mon code ?

Et si je mettais jute :

while(getchar()!='\n');

ça n'efface pas la mémoire ?
03/07/2009, 18h12
Médinoc

Cette boucle while est sympa, mais trop simpliste, car elle ne couvre pas le cas d'une erreur de lecture: En cas d'erreur, elle tournerait indéfiniment.
C'est pour ça qu'on écrit la fonction fpurge à la place.

Citation:

Comment faire pour insérer ces fonctions dans mon code ?

Copier-coller?
03/07/2009, 18h14
gl

Citation:

Envoyé par chicabonux

Je devrais faire comme ça avec size_of

sizeof, pas size_of.

Citation:

Envoyé par chicabonux

Code:

fgets(t,sizeof,fichier);

A quoi correspond fichier ici ? Tu voulais probablement écrire fgets(t, sizeof t, fichier).

Citation:

Envoyé par chicabonux

Et si je mettais jute :

while(getchar()!='\n');

ça n'efface pas la mémoire ?

La boucle ne s'arrête pas sur une fin de fichier et va continuer de boucler indéfiniment.

[EDIT] Correction d'une petite faute inattention.
03/07/2009, 18h30
chicabonux

fichier correspond au fichier créé par le programme principal où vients'insérer le bout de code.

Par contre pour le copier coller ce que je ne comprends pas c'est que je dois l'insérer dans ce bout de code et pas le mettre en dehors ?

Je dois mettre les 2 fonctions avant ou après le code ? Désolé mais c'est nouveau pour moi de devoir faire ça:cry:
03/07/2009, 20h32
gl

Citation:

Envoyé par chicabonux

Par contre pour le copier coller ce que je ne comprends pas c'est que je dois l'insérer dans ce bout de code et pas le mettre en dehors ?

Je dois mettre les 2 fonctions avant ou après le code ? Désolé mais c'est nouveau pour moi de devoir faire ça:cry:

Ca n'a pas réellement d'importance tant que la fonction est visible de l'appelant.
Le plus simple est de les mettre avant.

Il suffit ensuite de remplacer ton appel à fgets() par l'appel à la fonction fgetspurge() fournie par Medinoc.
04/07/2009, 10h04
chicabonux
Encore une question,

pour lire une chaîne de caractères sur l'écran, je ne dois pas écrire :
Code:

1 2 printf("entrer une ligne de caracteres : \n"); gets(CH);
Je dois écrire quoi à la place ? fgets(CH,sizeof,stdin) ?
04/07/2009, 10h06
ram-0000

Code:

fgets(CH, sizeof(CH), stdin);

Tout simplement
04/07/2009, 10h10
chicabonux

merci :ccool:

Et je dois à chaque fois mettre les fonctions fpurge et fgetspurge ou ce n'est que quand je crée un grand programme ?

(excusez moi d'insister mais je cherche à bien comprendre :(:(:( )
04/07/2009, 11h59
Médinoc

Disons que fgetspurge() est préférable à fgets() dans la plupart des cas, donc autant l'utiliser systématiquement pour les entrées utilisateur, sauf quand tu as besoin de récupérer TOUTE la ligne tapée indépendamment d'une hypothétique "taille maximale".

Quand tu te trouves dans ce dernier cas, il faut une solution encore plus compliquée, toujours à base de fgets().