Discussion :

[Chop_chop]

Bonjour à tous,
J'essaie de mettre le contenu d'un fichier de plusieurs MO dans un char* puis dans un string. Mais, le comportement de mon programme est bisare. Voici mon code :

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
string list_str;
char* list_char;
 
ifstream file("c:\\cpp\\copie\\out.txt",ios::binary);
int size;
 
// taille du fichier
file.seekg(0,ios_base::end);	
size= file.tellg();			
file.seekg(0,ios_base::beg);  
 
list_char= new char[size];			
file.read(list_char,size);
list_of_packets.assign(list_char,size);
 
cout<<strlen(list_char)<<endl;       //cela m'affiche 2633
cout<<list_of_packets.size()<<endl; // cela m'affiche 19599000 (qui est la taille exacte du fichier)
cout<<list_of_packets<<endl;
cout<<list_char;

Pourquoi strlen(list_char) fournit-il une valeur différente de list_of_packets.size() alors que c'est justement list_char que j'ai mi dans list_of_packets ?

Quand je vérifie le contenu, list_str contient bien le contenu exact du fichier, mais list_char ne contient qu'une petite partie, et arrête d'afficher lorsque, dans mon fichier , apparait le caractère NUL (identifié avec Notepad+).

Est-cela le problème, comment rectifier et faire en sorte que char* contienne tous le contenu du fichier.

Merci

[Laurent Gomila]

Pourquoi strlen(list_char) fournit-il une valeur différente de list_of_packets.size() alors que c'est justement list_char que j'ai mi dans list_of_packets ?

Parce que strlen s'arrête au premier 0. Donc ton tableau de caractères contient bien tous les caractères, c'est ta façon de les compter qui n'est pas adaptée.

[zais_ethael]

Si tu es sous windows, c'est normal.
Tous tes retours à la ligne sont sur deux bytes dans le fichier mais sont convertis en 1 byte en interne (toute l'utilité d'ouvrir un fichier en mode texte ).
Si tu lis des données pures binaires, je te conseillerais d'ouvrir ton fichier en mode binaire puis de ne pas utiliser les méthodes de stream (les streams sont une surcouche aux buffers orientée chaines de caractères), utilises plutot les méthodes du buffer sous-jacent (la méthode rdbuf() de stream te renvoie un pointeur sur celui-ci). Finalement, n'utilises pas de std::string, ce n'est pas fait pour ça. Par contre un vector<char> fait des merveilles
Si tu lis des données textuelles, laisses tomber ton pointeur de char. Utilises std::getline(stream&,string&) pour importer ton texte ligne par ligne puis concatène le tout dans un stringstream.

[Chop_chop]

Merci Laurent pour l'explication et zais_ethael pour les pistes de solutions plus efficaces. Je vais me tourner vers cela

[Chop_chop]

Je me permet de revenir car j'ai quelques soucis avec le morceau de code modifié suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
  filebuf *pbuf;
  ifstream filestr;
  long size;
  char * buffer1;
 
  filestr.open ("c:\\cpp\\copie\\out.trp");
  pbuf=filestr.rdbuf();
  size=pbuf->pubseekoff (0,ios::end,ios::in);
  pbuf->pubseekpos (0,ios::in);
  buffer1=new char[size];
  pbuf->sgetn (buffer1,size);
  filestr.close();
  cout<<strlen(buffer1);

A la fin lors de l'affichage de strlen(buffer1), la longueur affichée est de 4octets trop grande (dans mon cas : 19 599 004 octets alors que le fichier ouvert fait 19 599 000 octets).

D'ou cela peut-il venir dans mon code ? Merci

[zais_ethael]

Je ne suis pas sur que ce soit ça, mais on dirait que tu n'as toujours pas ouvert ton stream en mode binaire.
Voir ici: http://c.developpez.com/tutcpp/x6779.php#AEN6916, si je me souviens bien, il faut soit le passer en argument du constructeur de stream, soit à la méthode open().
Edit: j'avais pas vu, oublie les strlen quand tu ne traites pas de chaines de caractères!!! D'ailleurs, tu n'ajoutes même pas de 0 à la fin de ton buffer, c'est le plantage aléatoire assuré ça !