probleme lecture ficher cause separateur

Version imprimable

Bonjour à tous,

Voilà j'aimerai lire dans un fichier les données de personne mais ces données sont sous la forme nom;prenom;age;adresse;numero;ville;taille;telephone;sexe

Voici mon code qui ne focntionne pas car je ne trouve pas comment faire pour dire au compilateur Visual que mon séparateur est ;.
Code:

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 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 void Joueur::lire_fichierjoueur() { ifstream entree; entree.open("Joueur.txt",ios::in); entree.seekg (0, ios::beg); if(entree.fail()){ cerr<<"Fail _ Impossible de lire le fichier joueur.txt\n"; exit(8); } if(entree.bad()){ cerr<<"Bad _ Impossible de lire le fichier joueur.txt\n"; exit(8); } while(!entree.eof()) { entree>>nom>>prenom>>age>>adresse>>numero>>ville>>taille>>telephone>>sexe; if(entree.eof()!=1) { cout<<"Nom:"<<nom<<endl; cout<<"Prénom:"<<prenom<<endl; cout<<"Taille:"<<taille<<endl; cout<<"Age:"<<age<<endl; cout<<"Adrese:"<<adresse<<" "<<numero<<endl; cout<<"Ville:"<<ville<<endl; cout<<"Telephone"<<telephone<<endl; if(sexe==1) { cout<<"Sexe: feminin"<<endl; } else if(sexe==0) { cout<<"Sexe: masculin"<<endl; } else { cout<<"Erreur"<<endl; } } } system("pause"); entree.close(); }
Sachant que Joueur est une classe qui a comme donnée membre TAILLE et que elle dérive de la classe Personne qui elle a les autres donnée membre.

Merci pour votre aide

11/12/2009, 11h41
Lavock

Sinon je crois que y a un truc boost (tokenizer) pour ça, mais je connais pas trop boost.
11/12/2009, 11h46
ciic10

...

Merci pour la réponse mais je début en C++ et ce que vous me dit ça me dit absolument rien
11/12/2009, 11h55
Lavock

Un tokenizer, c'est une fonction qui te permet de séparer une chaine de caractère en plusieurs plus petite. Soit tu la fais toi même (pas très dure), soit tu utilise boost::tokenizer (rappelle, je ne sais rien de cette bibliothèque a par que ça a le bon nom).
11/12/2009, 12h17
3DArchi

Bonjour et bienvenu,
La F.A.Q. contient les éléments de réponse à ton problème : Comment découper une chaîne en fonction d'un séparateur ?.
Tu peux aussi regarder du côté de ces discussions où un problème similaire a été abordée :
Fichier CSV
Découper une chaîne

J'ai essayer en faisant comme ceci:

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
void Joueur::lire_fichierjoueur()
{
	ifstream entree;
 
	istringstream iss();
    entree.open("Joueur.txt",ios::in);
	entree.seekg (0, ios::beg);
    if(entree.fail()){
		cerr<<"Fail _ Impossible de lire le fichier joueur.txt\n";
		exit(8);
	}
	if(entree.bad()){
		cerr<<"Bad _ Impossible de lire le fichier joueur.txt\n";
		exit(8);
	}
	while(!entree.eof())
	{
 
		while (getline(iss,';'))
		{
			if(entree.eof()!=1)
			{
				cout<<"Nom:"<<nom<<endl;
				cout<<"Prénom:"<<prenom<<endl;
				cout<<"Taille:"<<taille<<endl;
				cout<<"Age:"<<age<<endl;
				cout<<"Adresse:"<<adresse<<" "<<numero<<endl;
				cout<<"Ville:"<<ville<<endl;
				cout<<"Telephone"<<telephone<<endl;
				if(sexe==1)
				{
					cout<<"Sexe: feminin"<<endl;
				}
				else if(sexe==0)
				{
					cout<<"Sexe: masculin"<<endl;
				}
				else
				{
					cout<<"Erreur"<<endl;
				}
			}
 
 
		}
	}
	system("pause");
	entree.close();
 
}

Mais bon je n'arrive pas à ne plus avoir d'erreur

Merci pour votre aide

Salut,
Il manque deux étapes : lire la chaine depuis le fichier et l'affecter au flux de chaines. Je n'ai pas tester mais ça doit être quelque chose comme ça :
Code:

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 27 28 void Joueur::lire_fichierjoueur() { std::ifstream entree; entree.open("Joueur.txt",std::ios::in); if(entree.fail()){ std::cerr<<"Fail _ Impossible de lire le fichier joueur.txt\n"; throw 8; } if(entree.bad()){ std::cerr<<"Bad _ Impossible de lire le fichier joueur.txt\n"; throw 8; } std::string ligne; while(std::getline(entree,ligne)) { std::istringstream iss(ligne); std::getline( iss, nom, ';' ); // type(nom)==std::string std::getline( iss, prenom, ';' ); // type(prenom)==std::string std::getline( iss, taille, ';' ); // type(taille)==std::string std::getline( iss, age, ';' ); // type(age)==std::string std::getline( iss, adresse, ';' ); // type(adresse)==std::string std::getline( iss, ville, ';' );// type(ville)==std::string std::getline( iss, telephone, ';' ); // type(telephone)==std::string std::getline( iss, sexe, ';' ); // type(sexe)==std::string } entree.close();
A noter, que exit(8) et pause(), c'est pas terrible.
Ensuite, j'aurais eu tendance à mettre lire_fichierjoueur dans une classe dédiée à la lecture de fichier CSV quitte à la faire ami de Joueur. Car si un jour tu dois supporter plusieurs type de fichier, tu auras une fonction par fichier dans ta classe joueur ? Non, ce n'est pas très cohérent. Joueur n'a pas besoin de savoir ce lire dans un fichier.

11/12/2009, 14h52
Lavock

[edit] grilled ><!
11/12/2009, 15h07
ciic10

J'utilise ce qu'on m'apprend à l'école...

Le compilateur nannonce 95erreur l'une des premiere est le fait qu'un string ne peut etre converti en int alors que taille, numero, telephone, sexe sont des int.
11/12/2009, 15h09
Lavock

dans la version d'arch, se sont des strings >< ! Pour les convertir en int, cf. FAQ.

Salut, et bienvenue sur le forum.

Déjà, je voudrais faire quelques remarques (qui n'ont rien à voir avec le problème) sur les quelques premières lignes de code, à savoir la partie
Code:

1 2 3 4 5 6 7 void Joueur::lire_fichierjoueur() { ifstream entree; istringstream iss(); entree.open("Joueur.txt",ios::in); entree.seekg (0, ios::beg);
Il faut, en effet, savoir que, par définition, un flux d'entrée comme ifstream sera ouvert avec le flag... ios::in placé, et que, par définition, le "curseur de lecture" sera placé... au début du fichier.

Il est donc tout à fait inutile de rajouter le deuxième argument (,ios::in) à la fonction open, et, de même, il ne sert à rien de vouloir... déplacer le curseur de lecture au début du fichier avec la fonction membre seekg.

Une autre remarque qui me semble importante à faire sur ces quelques lignes, c'est qu'il existe un constructeur prenant comme premier argument une chaine de caractères "C style" correspondant au nom du fichier à ouvrir et proposant un comportement RAII (Ressource Acquisition Is Initialization).

C'est à dire que ce constructeur va, de manière strictement automatique, chercher à ouvrir le fichier dont le nom est donné, et placer le curseur de lecture en début de fichier.

Il est largement préférable d'utiliser ce constructeur particulier, ne serait-ce que pour avoir la certitude que l'ensemble des actions à mener avant d'entreprendre la lecture auront été effectuées (en plus, je suis de nature fainéante lorsqu'il s'agit d'écrire du code, et, si je peux faire en une ligne ce que je devrais, autrement, faire en 5, je choisi sans hésiter la première solution :D)

Ainsi, ces premières lignes de code pourraient être fortement simplifiées en
Code:

1 2 3 4 void Joueur::lire_fichierjoueur() { ifstream entree("Joueur.txt"); istringstream iss;
Viennent ensuite les lignes
Code:

1 2 3 4 5 6 7 8 if(entree.fail()){ cerr<<"Fail _ Impossible de lire le fichier joueur.txt\n"; exit(8); } if(entree.bad()){ cerr<<"Bad _ Impossible de lire le fichier joueur.txt\n"; exit(8); }
Là encore, il y a parfaitement moyen de se simplifier la vie, étant donné que tu ne semble pas vouloir faire la distinction de la raison pour laquelle le fichier n'a pas pu être ouvert.

Il existe, en effet, un opérateur de conversion qui permet, simplement, en effectuant un test booléen, de savoir si le flux est dans un état valide ou non.

Autrement dit, un seul test sur la variable correspondant à ton flux peut parfaitement suffire pour déterminer si, oui ou non, ton fichier a bien été ouvert.

Ces lignes pourraient donc avantageusement être remplacées par un simple
Code:

1 2 3 4 5 if(!entree) { cerr<<"Fail _ Impossible de lire le fichier joueur.txt\n"; exit(8); }
Vient ensuite le noeud du problème: ta boucle
Code:

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 27 28 29 30 31 while(!entree.eof()) { while (getline(iss,';')) { if(entree.eof()!=1) { cout<<"Nom:"<<nom<<endl; cout<<"Prénom:"<<prenom<<endl; cout<<"Taille:"<<taille<<endl; cout<<"Age:"<<age<<endl; cout<<"Adresse:"<<adresse<<" "<<numero<<endl; cout<<"Ville:"<<ville<<endl; cout<<"Telephone"<<telephone<<endl; if(sexe==1) { cout<<"Sexe: feminin"<<endl; } else if(sexe==0) { cout<<"Sexe: masculin"<<endl; } else { cout<<"Erreur"<<endl; } } } }
Avant de parler de ton while(!entree.eof()), je vais m'attarder un peu sur ton while(getline(iss,';'))

En effet, dans le cas présent, tu fais appel à la fonction libre getline, et non à la fonction membre de la classe istream portant le même nom.

Les deux paramètres obligatoires à fournir sont donc:
le flux à partir duquel récupérer la donnée
une chaine de caractères dans laquelle mettre la donnée récupérée
le paramètre facutlatif étant le séparateur à considérer.

Pour cela, il semble intéressant de disposer... d'une chaine de caractères temporaire pour récupérer cette donnée.

Cela donnerait donc un code plutôt proche de
Code:

1 2 std::string temp; while(getline(entree,temp))
Et cela nous ramène à la ligne while(!entree.eof()) que j'avais laissée en suspend...

En effet, il est souvent préférable (reporte toi à la partie de la FAQ dédiée à la manipulation des fichiers ;)) de ne pas baser sa logique sur le test du resultat renvoyé par la fonction eof, car le flag correspondant n'est placé que... quand la fin de fichier a été dépassée...

Ta séquence de lecture pourrait donc ressembler à
Code:

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 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 std::string temp; // une chaine de caractères à usage temporaire // tant qu'il est possible de lire une ligne du fichier while(getline(entree,temp)) { /* nous allons utiliser un flux de conversion de type dont nous * initialisons directement le contenu avec la variable temporaire */ stringstream iss(temp); /* et au départ duquel nous pouvons sereinement récupérer l'ensemble * des membres intéressants, quel que soit leur type * * (il faut juste ne pas oublier qu'ils sont séprés par ";", et * qu'il faut donc veiller à le supprimer du flux à chaque fois */ char dummy; // pour récupérer le caractère génant /* si la récupération de toutes les informations échoue, c'est que * le format de fichier est invallide */ if(! (iss>>nom>>dummy >>prenom>>dummy >>taille>>dummy >>age>>dummy >>adresse>>dummy >>numero>>dummy >>ville>>dummy >>telephone>>dummy >>sex) ) { cerr<<"Error : format de fichier invalide"; exit(NN); //valeur à décider toi-même, mais, peut être serait il // intéressant de lancer plutôt une exception ;) } /* allez, nous pouvons procéder à un affichage des données */ cout<<"Nom:"<<nom<<endl <<"Prénom:"<<prenom<<endl <<"Taille:"<<taille<<endl <<"Age:"<<age<<endl <<"Adresse:"<<adresse<<" "<<numero<<endl <<"Ville:"<<ville<<endl <<"Telephone:"<<telephone<<endl <<"Sexe :"<< (sexe==0 ? "feminin" : sexe==1 ? "masculin" : "androgine??"); }