Erreur de compilation sur une fonction utilisant les flux

Version imprimable

Salut

J'ai un problème au niveau de ma fonction.en fait je voudrais récupéré les mots d'un fichier.txt pour les mettre dans un tableau, les mots du fichier.txt sont disposés de cette manière
MOT1
MOT2
MOT3
voici mon code qui pose un problème

Code:

1
2
3
4
5
6
7
8
9
 
void recueilleMot(ifstream fichier,  vector <string> &tabMot)
{
	string mot;
	while (getline(fichier, mot))
	{
		tabMot.push_back(mot);
	}
}

:roll:

maintenant voici le prototype

Code:

1
2
 
void recueilleMot(ifstream fichier, vector <string> &tabMot);

quand je débogueur il me signal ces erreurs.

Code:

1
2
3
4
5
6
7
8
9
10
 
Erreur	1	error C2065: 'ifstream'*: identificateur non déclaré mesfonc.h	4	1	test
Erreur	2	error C2146: erreur de syntaxe*: absence de ')' avant l'identificateur 'fichier'   mesfonc.h	4	1	test
Erreur	3	error C2182: 'recueilleMot'*: utilisation non conforme du type 'void'	 mesfonc.h	4	1	test
Erreur	4	error C2059: erreur de syntaxe*: ')'	mesfonc.h	4	1	test
Erreur	5	error C2065: 'ifstream'*: identificateur non déclaré   mesfonc.h	4	1	test
Erreur	6	error C2146: erreur de syntaxe*: absence de ')' avant l'identificateur 'fichier'	mesfonc.h	4	1	test
Erreur	7	error C2182: 'recueilleMot'*: utilisation non conforme du type 'void'	mesfonc.h	4	1	test
Erreur	8	error C2059: erreur de syntaxe*: ')'	mesfonc.h	4	1	test
Erreur	9	error C2365: 'recueilleMot'*: redéfinition*; la précédente définition était 'variable de données'    ajoutedeux.cpp	13	1	test

bon j'ai essayer d'autre solution mais toujours le même problème pouvez vous m'aider:calim2:.

16/07/2013, 02h48
Iradrille

Hello,

L'erreur est qu'il faut passer le flux par référence, les flux ne sont pas copiables.

(Tu as dû oublier d'inclure fstream aussi.)
17/07/2013, 14h12
alves1993

il s’avère que je n'ai pas oublier de mettre le <fstream> et j'ai passé les flux par référence mais rien de concret le débogueur me signal qu'il y a toujours des erreurs :calim2:
17/07/2013, 23h51
Ehonn

Bonjour, peux-t-on avoir ces nouvelles erreurs :) ?
18/07/2013, 15h11
koala01
Salut,

si tu n'as pas utilisé la directive using namespace std; ( que je ne recommande personnellement absolument pas d'utiliser, et dont je t'interdirai l'usage dans les fichiers d'en-tête :D), il faut garder en mémoire que tout ce qui vient de la bibliothèque standard se trouve dans l'espace de noms std.

Cela signifie que, pour pouvoir utiliser la classe ifstream, il faut utiliser son "nom complet" (on parle de "nom pleinement qualifié"), à savoir std::ifstream.

Il en ira de même pour vector, pour string et pour getline, bref, pour tout ce qui nécessite (on va faire simple :D) l'inclusion d'un fichier d'en-tête sans extension.

Si tu es sur d'avoir inclus les fichiers d'en-tête requis (j'en dresserai la liste tout de suite :D) dans ton propre fichier d'en-tête (parce que c'est nécessaire aussi pour le prototype), tu devrais au final te retrouver avec quelque chose comme

Code:

void recueilleMot(std::ifstream fichier, std::vector <std::string> &tabMot);

et
Code:

1 2 3 4 5 6 7 8 void recueilleMot(std::ifstream fichier, std::vector <std::string> &tabMot) { std::string mot; while (std::getline(fichier, mot)) { tabMot.push_back(mot); } }
Nota: Pour que ton code puisse compiler, il faudra penser à inclure les fichiers d'en-tête
1. <string> (pour... std::string)
2. <vector> (pour std::vector) et
3. <fstream> (pour std::ifstream
18/07/2013, 17h23
alves1993

super sa passe :).
ok merci pour le conseil a propos du fichier d’entête using namespace std; :ccool:
18/07/2013, 17h34
Astraya
Je plusoi koala sur ce sujet.

N'utilise pas using namespace std;. C'est dangereux (que ce passe t il si tu créer une classe string et que tu fait un using namespace std ou n'importe quel classe du standard)

Une solution que je trouve très élégante est le typedef.
Code:

1 2 typedef std::string String typedef std::ifstream IFStream
Le mieux est de mettre ca dans une entête que tu inclus en premier dans le projet et dans une entête précompiler. et Hop plus de souci, et tu peux facilement faire du #define #if defined etc.. pour choisir quel classe tu désire a la compilation comme par exemple passer de std a boost facilement avec une variable de compile.
18/07/2013, 20h43
koala01

Et encore, je ne suis vraiment pas fan de cette solution...

Autant je n'ai pas grand chose contre le fait de créer un alias d'espace de noms quand les noms pleinement qualifiés deviennent vraiment trop long (les bibliothèques de boost, par exemple), autant je suis vraiment réfractaire à tout ce qui permettrait d'utiliser le nom de manière non pleinement qualifiée.

La raison est, sommes toutes, bien simple:

Si, d'une manière ou d'une autre, tu t'arranges pour ne pas avoir à utiliser le nom pleinement qualifié, tu peux avoir, à la lecture du code, l'impression que tu as la possibilité d'aller modifier le type manipulé, alors que si tu garde le nom pleinement qualifié (fusse-t-il simplifié par un alias d'espace de noms), tu gardes une information primordiale à la lecture du code: ce type provient d'ailleurs et tu dois partir du principe qu'il fait ce qu'on attend de lui.

En plus, le typedef provoque, finalement, exactement le même problème que la directive using: Si tu veux, à un moment particulier, créer ta propre classe String, le compilateur va t'envoyer paitre (et sans doute d'ailleurs plus violemment qu'avec la directive using :aie:)
19/07/2013, 10h05
Astraya

Citation:

En plus, le typedef provoque, finalement, exactement le même problème que la directive using: Si tu veux, à un moment particulier, créer ta propre classe String, le compilateur va t'envoyer paitre (et sans doute d'ailleurs plus violemment qu'avec la directive using )

Héhé, en fait, je ne l'utilise jamais hors namespace :)
Exemple, j'ai un moteur perso ou je définis mes types de base (int String, Flux, etc...) le tout est dans un namespace CEL. Donc, CEL::String n'est pas la même chose que std::string et si la fonction attends un CEL::String elle n'acceptera pas un std::string. Jusque la j'ai jamais eu de problème avec un projet de 5 librairie différente utilisant le même header precompiler. Après cela répond a mes besoins :p
19/07/2013, 11h20
koala01

Jusqu'au jour où, le jeu d'inclusion en cascade aidant, tu auras, dans un fichier d'implémentation de CEL, inclut le fichier d'en-tête contenant le typedef sur std::string et ta déclaration perso de CEL::String.

Dans le meilleur des cas, ce sera CEL::String qui sera automatiquement sélectionné (alors que tu voulais peut etre utiliser std::string :aie:) du fait que tu te trouve dans CEL, dans le pire, le compilateur se plaindra du conflit ;)
19/07/2013, 11h56
Astraya

Justement, je veux que le compilateur crie au loup lorsque l'on utilise autre chose que la classe que j'aurais définit, je ne laisse pas le choix de certains type pour assurer leur fonctionnement. Bon actuellement, CEL::String est un std::string donc pas de problème... mais std::string à des défauts et sur certaines plateforme pose problème donc j'ai changé le typedef à la compile avec une autre classe respectant la logique de la programmation par règle. Exemple, j'avais du changer tr1 en boost et 0 souci, les 5 libraires fonctionnait en moins d'une heure avec une autre classe.