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.