Discussion :

[Roud9]

Bonjour,

J'ai créé deux structures que je vais ensuite mettre dans une liste, mais lorsque je fais la lecture du fichier et affiche le contenu de ma structure, la dernière donnée est affichée deux fois. Voici mon code :

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
void lireFichierAbonnes(abonne *& listeAbonnes)
{
	int compterReservations = 0;
	ifstream lecture; 
	abonneFichier personne;
	reservationFichier reservation;
	lecture.open("Abonnes.data", ios::binary); 
 
	if(!lecture.fail())
	{
		cout << "Voici la liste des des abonnés : " << endl << endl;
 
 
		while (!lecture.eof())   // Tant qu'on peut lire un numero d'item      
		{
			lecture.read ((char*) &abonne, sizeof(abonneFichier));
 
			cout << personne.nom << endl;
			cout << personne.no << endl;
			cout << personne.adresse << endl;
			cout << personne.telephone << endl;
			cout << personne.nbReservations << endl;	
			while(compterReservations != personne.nbReservations)
			{
				lecture.read ((char*) &reservation, sizeof(reservationFichier));
				cout << reservation.isbn << endl;	
				cout << reservation.dateReserv.jour << endl;
				cout << reservation.dateReserv.mois << endl;
				cout << reservation.dateReserv.an << endl;
				++ compterReservations;
			}	
			compterReservations = 0;	
		} 
		lecture.close();
	}
	else
	{
		cout << "La lecture du fichier n'a pas réussi" << endl;
	}
}

Avez vous une solution?

Merci d'avance

[JolyLoic]

Code :

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

1
2
3
		while (!lecture.eof())   // Tant qu'on peut lire un numero d'item      
		{
			lecture.read ((char*) &abonne, sizeof(abonneFichier));

Déjà, je présume que tu voulais mettre personne et non abonne dans ton read. Ensuite, j'espère que tu es conscient que cette manière de lire est très restrictive. Elle ne marchera pas si ton objet contient des pointeurs sur des sous objets (par exemple des chaînes de caractères), elle ne marchera pas non plus pour relire avec un compilateur des données enregistrées avec un autre (ou le même avec des options de compilation différentes).

Autrement, ton problème est assez classique, je l'avais deviné à la lecture du titre de ton post, avant d'en lire le contenu. En effet, eof ne devient vrai qu'après que l'on ait essayé de lire trop loin dans le fichier. Donc, tu lis le dernier enregistrement une fois, tu passes dans la boucle à nouveau, tu tentes de faire un nouveau read, celui-ci échoue, mais tu es déjà dans la boucle donc tu continues à créer un enregistrement avec les dernière valeurs lues.

A remplacer par :

Code :

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

1
2
3
4
while (lecture.read ((char*) &abonne, sizeof(abonneFichier)))
{
 //...
}

[Roud9]

Merci JolyLoic, cela a résolu mon problème