Discussion :

[Pragmateek]

Salut!

J'impl�mente une classe fichier mais j'ai des probl�mes pour afficher un fichier bufferis�:

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
 
class fichier{
public:       string path;//adresse absolue du fichier
              fstream file;
              char *buffer;
              int sizeofbuffer;
 
              fichier(string s):path(s),file(NULL){}//constructeur
 
              void open(){
                          file.open(path.c_str());
                          }
 
              void bufferize(int n){
                                     buffer=new char;          
                                     file>>buffer;
                                     sizeofbuffer=n;                                      
                                     }
 
              void printbuffer(){
                                 for(int i=0;i<sizeofbuffer;i++){
                                         cout<<buffer[i];
                                         }
                                 }
 
                          };

Voil� le main:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
int main(int argc, char *argv[])
{
 
fichier f("C:/a.txt");
f.open();
 
cout<<f.file.is_open()<<endl;
 
f.bufferize(100);
f.printbuffer();

Ca compile,le test "f.file.is_open()" me renvoie 1 indiquant que le fichier est correctement ouvert,mais l'appel � la m�thode "printbuffer()" m'affiche des signes qui n'ont rien � voir avec le fichier.

Le probl�me est s�rement � l'initialisation de "buffer";j'ai essay� avec un "buffer" constant:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

char buffer[200];

sans faire d'allocation dynamique mais le probl�me demeure.

Merci de votre aide.

[Herode]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 void bufferize(int n){ 
                                     buffer=new char;          
                                     file>>buffer; 
                                     sizeofbuffer=n;                                      
                                     }

:
Ne serait-ce pas plut�t :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 void bufferize(int n){ 
                                     buffer=new char[ n ];          
                                     file>>buffer; 
                                     sizeofbuffer=n;                                      
                                     }

[Higestromm]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
 
              void bufferize(int n){
                                     buffer=new char;          
                                     file>>buffer;
                                     sizeofbuffer=n;                                      
                                     }

D�ja ton new char alloue 1 seul octet de m�moire... Si tu veux allouer un tableau de 100 il faut ecrire :
buffer = new char[n];

Ensuite je trouve cette ligne bizard :
file>>buffer;
ca n'est pas la bonne m�thode pour remplir un buffer selon moi

Enfin pour finir :
sizeofbuffer=n;
Meme si dans l'absolue cela fonctionne, je pense qu'il existe des m�thodes plus �l�gantes.

[Pragmateek]

Merci de vos r�ponses mais le probl�me demeure.

Cependant je croyais que "new char[n]" renvoyait "un pointeur sur un tableau de n caractere" alors que ici il renvoie "un pointeur sur un caract�re":"char *buffer".
De m�me que:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

int a=new int;

est invalide mais:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

int *a=new int;

est bon;car "new int" renvoie "un pointeur sur un entier" et pas un entier.

Quel est donc le fonctionnement de "new"?

Merci.

[Herode]

1 - je ne crois pas que fstream prenne NULL � la construction.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
  fichier( const string& s) : path(s)
  {} //constructeur

devrait suffire.
NB : passage d'objet par ref constante, plus �conomique que par copie

2 - la ligne

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

    m_file >> buffer;

lit dans m_file jusqu'� tomber sur un s�parateur (blanc, tabulation, saut de ligne, etc...).
Si ton buffer fait 100 caract�res, que le premier item fait moins de 100 caract�res et que le buffer n'a pas �t� initialis� proprement, alors il peut se passer pas mal d'octets avant qu'on ne tombe sur un 0 terminal...

3 - un new renvoie un pointeur sur le type demand�. Donc :
new int renvoie un int*
new char[10] un char* pointant au d�but d'une zone allou�e pour 10 char
new double[956] un double* pointant au d�but d'une zone allou�e pour 956 doubles
etc...

[Pragmateek]

Merci de ces conseils.

J'ai fait des modifications mais il ne m'affiche que le premier caract�re du fichier,les autres n'ayant rien � voir.

Quelle est la bonne m�thode pour copier un fichier dans un tableau ou un autre conteneur facilement manipulable?

[Higestromm]

en complement de ce que te dit Herode je te conseille d'utiliser la fonction

file.read (buffer,n);

Pour lire ton fichier... Cela peux eviter des erreurs de segmentation si la zone � lire est sup�rieur � la taille de ton buffer.

[Pragmateek]

En effet avec la m�thode "read(...)" il affiche sans probl�me le contenu du fichier.

En fait plus car le buffer est trop grand;d'ailleurs quelle est la m�thode pour d�terminer la taille du fichier?

[Herode]

L'op�rateur >> fonctionne pour les fichiers texte ASCII. Si tu as un fichier binaire, il faut le lire en mode binaire (cf. Higestromm)

NB : si tu as un fichier Unicode, UTF8 ou autre codage sur >1 octet... je ne sais plus, mais �a devrait se compliquer. Le sujet a �t� �voqu� dans un fil il n'y a pas tr�s longtemps, je crois.

Quelle est la bonne m�thode pour copier un fichier dans un tableau ou un autre conteneur facilement manipulable?

Si tu veux le lire d'un coup, tu peux passer �r des bytearrays que tu tron�onnes et transtypes apr�s � la vol�e. C'est rapide et remarquablement casse-gueule d�s que la situation se complique
Sinon, tu peux utiliser std::copy sur un std::vector par exemple. Ou faire toi m�me la boucle � la main.

[Pragmateek]

L'op�rateur >> fonctionne pour les fichiers texte ASCII.

C'est un fichier texte notepad.
C'est bien de l'ASCII?

[Herode]

Oui. Ca devrait donc fonctionner.

[Pragmateek]

Merci � tous.

[Luc Hermitte]

L'op�rateur >> fonctionne pour les fichiers texte ASCII.

C'est un fichier texte notepad.
C'est bien de l'ASCII?

Encodage 8bits, oui probablement. ASCII cela d�pendra de ce que tu y mets. Au premiers caract�re accentu� tu sors de la table ASCII (limit� � 127 caract�res!), et dans mes souvenirs notepad a la facheuse habitude d'encoder en OEM (CP-850 sous nos tropiques). Cela serait compatible avec la console de windows, mais pas avec celles en iso-8859-1 (aka latin1) (/15 <-> latin 9).

Bref, de toutes fa�ons tu t'y prends mal. Cf ton autre topic o� l'on te montre le chargement avec rdbuf() signal� dans la FAQ.

PS: pour ce qui est d'utiliser >> pour lire des fichiers utf-8 ou autres, le probl�me est un chouilla plus complexe et peut demander des code convertion facets. boost.serialize et adobe.ASL (adam&eve) en fournissent, ou alors il y a des techniques plus manuelles, ou encore des lib � IHM.