Discussion :

[Dom_the_quaker]

Salut,
j'ai un problème de déclaration de fstream (accès écriture et lecture d'un fichier) dans une classe
en gros:

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
 
A.h
 
classe A
{
private:
fstream file;
 
public
A(string& filename);
}
 
A.cpp
 
A::A(string& filename)
{
file.open(filename); // peut etre qu'il faut un petit .c_str() mais la n'est pas le probleme.
}

Donc, mon probleme, c'est qu'a la compile il me dit:

Code :

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

1
2
 
error: field 'file' has incomplete type //en pointant sur la ligne déclaration du .h

du coup je ne sais pas comment declarer mon fstream :/
merci d'avance
dom
ps: j'ai bien chercher dans des refs ou des tutos, mais ils ne parlent pas de declaration dans un classe.

[Invité(e)]

Bonjour,

En remplaçant

Code :

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

1
2
private:
file fstream;

par

Code :

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

1
2
private:
fstream file;

ça ne fonctionne pas mieux ?

[Dom_the_quaker]

merci mabu mais en fait cette erreur n'existe que sur mon exemple (ouf) (mais c'est bien mon genre de tout façons )
(je corrige de suite) mon probleme est toujours la...
mais ya t il une regle pour declarer un fstream dans une classe? a prioris il faut le definir tout de suite mais, je veux pouvoir instencier ma classe plusierus fois avec un fichier different je trouve ca louche, est ce que la declaration d'un fstream dans une classe pause probleme en general ou pas du tout?

[Davidbrcz]

#include <fstream> avec std:: devant et ca ira mieux.
Car tu essaie de créer un objet complet de type fstream. Pour y arriver, il faut que tout la déclaration de la classe soit connu. On connait cette déclaration via l'inclusion de l'en tête.

Donc ton code devient:

Code :

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

1
2
3
4
5
6
7
8
9
10
//A.h
#include <fstream>
classe A
{
private:
std::fstream file;
 
public
A(std::string& filename);
}

[Dom_the_quaker]

oh la la la.... le boulet.... quand je disais que c'etait bien mon genre....
merci Davidbrcz!! (beaucoup!)
promis j'arrete ... enfin j'essaye!
dom

[Invité(e)]

Hum comment se fait il qu'il faille std:: pour fstream, mais pas pour string ??

Sinon, aux erreurs de recopie pres, je n'ai pas de problème avec :

Code fs.h :

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

1
2
3
4
5
6
7
8
class A
{
private:
fstream file;
 
public:
A(string& filename);
};

Code fs.cc :

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

1
2
3
4
5
6
7
8
9
#include <fstream>
#include <string>
using namespace std;
#include "fs.h"
 
A::A(string& filename)
{
file.open(filename.c_str()); // peut etre qu'il faut un petit .c_str() mais la n'est pas le probleme.
}

[Davidbrcz]

Hum comment se fait il qu'il faille std:: pour fstream, mais pas pour string ??

L'inattention, je corrige.

Sinon mabu pour ton 2eme code, c'est normal car le pre-processeur copie et colle le code qu'on lui dit d'inclure. Avec les inclusions que tu fait avant et le using, ca passe mais si tu le faisait pas, ca casserai. De plus, tu brise la règle comme moi tout fichier doit être indépendant.

[Dom_the_quaker]

mabu, j'avais juste fait un code basique representant le schema de vrai code, et il se trouve que j'avais oublié d'inclure fstream, mais string y est lui
merci encore!!!
A+ (tard )
dom

[Médinoc]

Au passage, ta référence vers string devrait être une référence constante.
Ou mieux, prendre directement un char const *, puisque c'est ce que demande fstream. Ça évitera, si tu appelles ta classe avec une chaîne littérale, de créer une std::string pour rien...