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...

[Dom_the_quaker]

il faut que je m'oblige a utiliser const, il est seulement la pour proteger le contenu? ou il y a un coté performance ou autre a prendre en compte? sinon c'est vrai qu'un char const * serait plus judicieux une conversion de moins a faire .
++

[Médinoc]

Dans le cas présent, c'est surtout parce que le C++ n'accepte pas les références non-constantes vers les temporaires non-nommées, tu n'aurais donc carrément pas pu appeler ton constructeur avec une chaîne littérale...

Mais oui, la const-correctness est importante. Pas seulement pour protéger tes données, mais aussi pour faciliter la lecture du code (une référence const vers un objet accentue le fait que c'est un paramètre en entrée et surtout pas en sortie, etc.)

[Dom_the_quaker]

ok merci pour ces conseils, je vais tacher de les appliquer
A+ dom