Malloc / Lecture fichier

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Malloc / Lecture fichier

fatloui

I'll be back

Bonjour,
j'ai un nouveau probleme de lecture de fichier [:alph-one]

J'ai ecrit une fonction qui me permet de lire plusieurs caracteres dans un fichier jusqu'au caractere choisi (si je veux lire une ligne, je met '\n' comme parametre)

Code :

//Fonction qui permet la lecture d'une ligne jusqu'au caractere choisi
char* lecture(FILE*fp,char caractere)
{
char temp[100];
char *ligne;
int i=0;
int j=0;
char c='a';
//initialisation du tableau
for(j=0;j<100;j++) temp[j]='\0';
while(c!=caractere && c!=EOF)
{
c=fgetc(fp);
temp[i]=c;
i++;
}
ligne = (char*) malloc (sizeof(char)*(i));
strncpy (ligne,temp,i);
return ligne;
}

Simplement, je dois faire quelquechose de mal etant donné que si je lis "texte.txt", la fonction deconne et renvoi "texte.txtsWindows".
je comprend pas d'ou vient l'erreur. Quand j'affiche la valeur de i est 10, la taille de temp est 10 et la taille de ligne est 17. :s pourtant je fais bien mon malloc (il me semble)
Merci de votre aide

Message édité par fatloui le 23-07-2008 à 13:54:46

Publicité

fatloui

I'll be back

petit up

olivthill

Il faut toujours mettre un caractère nul en fin d'une chaine en C. Donc, ajouter ligne 19 :

temp[i] = '\0';

fatloui

I'll be back

merci pour ta reponse

ben en fait temp est initialisé à '\0' et le fait que j'increment encore une fois i sous entend que je copie tout les caracteres de temp+un '\0'.

j'ai donc toujours la meme erreur...

Elmoricq

Modérateur

Code :

char* lecture(FILE*fp,char caractere)

Le type naturel pour les caractères, en C, c'est int :
char * lecture (FILE *fp, int caractere)

(regarde par exemple le type de retour des fonctions fget...() )

Code :

char c='a';

Cf. juste au dessus :
int c = 'a';

Code :

//initialisation du tableau
for(j=0;j<100;j++) temp[j]='\0';

Tu peux écrire, à la place :
memset(temp, '\0', 100);

Code :

ligne = (char*) malloc (sizeof(char)*(i));

De plus, il est tout à fait inutile de caster le type de retour de malloc(), et sizeof(char) vaut toujours 1 par définition :
ligne = malloc(i+1);

Ou, si tu veux être absolument propre : ligne = malloc(sizeof(char) * (i+1));

Code :

strncpy (ligne,temp,i);

Comme tu connais exactement la taille de temp et de ligne, utilise tout simplement :
strcpy(ligne, temp);

Message cité 1 fois
Message édité par Elmoricq le 23-07-2008 à 14:23:43

fatloui

I'll be back

Elmoricq a écrit :

Code :

char* lecture(FILE*fp,char caractere)

Le type naturel pour les caractères, en C, c'est int :
char * lecture (FILE *fp, int caractere)

(regarde par exemple le type de retour des fonctions fget...() )

Code :

char c='a';

Cf. juste au dessus :
int c = 'a';

c'est seulement pour les puristes non ?
j'ai quand meme testé changé le type [:arn0]

Elmoricq a écrit :

Code :

//initialisation du tableau
for(j=0;j<100;j++) temp[j]='\0';

Tu peux écrire, à la place :
memset(temp, '\0', 100);

Merci je connaissais pas cette fonction

Elmoricq a écrit :

Code :

ligne = (char*) malloc (sizeof(char)*(i));

Sauf que "i" contient le nombre de caractères saisis : 1 pour 1 caractère, 2 pour 2 caractères, ..., n pour n caractères.
Or, comme le fait remarquer olivthill, il faut toujours qu'une chaîne se termine par '\0'.
Donc, il faut allouer (i+1) caractères pour ligne, et terminer cette variable par '\0'

De plus, il est tout à fait inutile de caster le type de retour de malloc(), et sizeof(char) vaut toujours 1 par définition :
ligne = malloc(i+1);

Ou, si tu veux être absolument propre : ligne = malloc(sizeof(char) * (i+1));

je savais qu'il fallait ajouter un caractere mais vu que la boucle incrementait une fois le i à la fin, je pensais que j'avais deja mon n+1 nombre de caractere. non?

Elmoricq a écrit :

Code :

strncpy (ligne,temp,i);

strncpy() est une fonction dangereuse si tu ne sais pas exactement ce qu'elle fait.
En l'occurence, si la taille de temp est supérieure ou égale à i, ce qui est le cas, alors... aucune '\0' n'est copié dans ligne.
Ton erreur est donc ici.

Comme tu connais exactement la taille de temp et de ligne, utilise tout simplement :
strcpy(ligne, temp);

effectivement je n'avais pas pensé au '\0' Mon erreur etait bien la Merci ^^

Message cité 1 fois

Elmoricq

Modérateur

fatloui a écrit :

je savais qu'il fallait ajouter un caractere mais vu que la boucle incrementait une fois le i à la fin, je pensais que j'avais deja mon n+1 nombre de caractere. non?

Sauf que i vaut 0 au départ.

FORUM HardWare.fr

Programmation

Malloc / Lecture fichier

Sujets relatifs
[résolu] Boucle fichier dans un repertoire	importé un fichier excel
Batch avec edition d'un fichier et variable	Lire un fichier dans mon application web
[Résolu] Envoi FTP de fichier Excel avec org.apache.commons.net.ftp	inclure un fichier php avec javascript
Recuprer L'URL du fichier Uploader !!	[VB6] Récupérer un fichier image sur le web => Non résolu
Telecharger un fichier zip sur internet	Recherche/suppression fichier batch
Plus de sujets relatifs à : Malloc / Lecture fichier

Page générée en 0.057 secondes