Discussion :

[chicabonux]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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]);
		}
	}
}

[Médinoc]

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

De plus, la valeur 30, tu devrais la passer en paramètre à ta fonction lecture.

[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()

[gl]

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

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.

[chicabonux]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[Médinoc]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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 ?

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

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

Copier-coller?

[gl]

Je devrais faire comme ça avec size_of

sizeof, pas size_of.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fgets(t,sizeof,fichier);

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

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.

[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

[gl]

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

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.

[chicabonux]

Encore une question,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
printf("entrer une ligne de caracteres : \n");
gets(CH);

Je dois écrire quoi à la place ? fgets(CH,sizeof,stdin) ?

[ram-0000]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Tout simplement

[chicabonux]

merci

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 )

[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().