PROBLÈME affichage contenu d'un fichier !

Question

Bonjour à tous,

Dans mon programme je dois ouvrir et lire le contenu d'un fichier et l'afficher à l'écran pour l'instant aucun problème sa marche...

voici le contenu du fichier:

charles
AdminCharles
boss
Luka
luka2
marco

lorsque que je lis le fichier l'affichage à l'écran me renvoie les informations mais des caractères superflu s'ajoute à la fin...

affichage à l'écran:

charles
AdminCharles
boss
Luka
luka2
marco
IOON  <---- Caractère étrange ?!?

je vous laisse la source en espérant que vous puissiez m'aider.

Merci d'avance

pFile = fopen( "C:\Windows\DBAcess\User\Accounts.acc" , "rb");
 
  
   fseek (pFile, 0 , SEEK_END);  
   lsize = ftell (pFile);
   rewind (pFile);
  
   buffer = (char*) malloc (lsize);   
  

   fread (buffer, 1, lsize, pFile); 
   
   fclose (pFile);  
   printf("%s",buffer);
   free(buffer);
   
   printf("

");

cptpingu · Answer

Essaie en changeant un morceau de ton code:


  buffer = malloc(lsize + 1);
  memset(buffer, '\0', lsize + 1);


Ici on intialise tout à zéro, ce qui évite d'avoir des caractères "pourris" qui trainent.
(Il faut inclure string.h).

_____________________________________________
Historique de mes créations, et quelques articles:[ http://0217021.free.fr/portfolio 
http://0217021.free.fr/portfolio]

cs_Arnotic · Answer

Je sais bien que la performance n'a pas l'air de trop vous inquiéter, cependant mettre un zero final en fin de mémoire serait quand même plus judicieux.

Changer la valeur d'un octet semble en toute logique plus rapide qu'un memset sur un buffer.


@+
Arnotic,
Admin CS

tpoinsot · Answer

C'est vrai mais il est plus judicieux de mettre toute la zone à zéro plutôt que de parier sur le contenu futur.
Ne mettre à zéro qu'un caractère ne devrait se faire qu'après la copie du contenu désiré, quand on est certain de la quantité.

BruNews · Answer

Mais enfin, une fonction de lecture donne le nbr d'octets réellement lus, il n'y a donc qu'1 seul octet à mettre à 0.
Faire un memset est un amuse CPU, inutile et donc nuisible.

ciao...
BruNews, MVP VC++

charleslemagicien · Answer

Merci pour toutes vos réponses !

PROBLÈME affichage contenu d'un fichier !

5 réponses

Votre réponse

Discussions similaires