Discussion :

[alves1993]

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 :

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

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);
	}
}

maintenant voici le prototype

Code :

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

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

quand je débogueur il me signal ces erreurs.

Code :

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

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.

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

[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

[Ehonn]

Bonjour, peux-t-on avoir ces nouvelles erreurs ?

[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 ), 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 ) 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 ) 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 :

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

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

et

Code :

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

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

[alves1993]

super sa passe .
ok merci pour le conseil a propos du fichier d’entête using namespace std;

[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 :

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

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.

[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 )

[Astraya]

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

[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 ) du fait que tu te trouve dans CEL, dans le pire, le compilateur se plaindra du conflit

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