Discussion :

[badack]

Bonjour à tous,

voici mon problème, j'ai une fonction qui me permet de sauvegarder dans un fichier binaire les données de ma structure. Cela se passe sans problème, par contre pour la lecture de se fichier, là ca coince. Si je comprends bien mon debugger, chaque nouvelle lecture est assignée à la variable nSkieur qui est toujours à la même adresse et pareil pour ma variable data.
Comment puis je faire pour que cela ne se passe pas?

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
 
void ChargementListe(bornes *lst){
 
	FILE *fChargement;
	PElement data = (PElement) malloc (sizeof(element));
	skieur *nSkieur = (skieur*) malloc (sizeof(skieur));
 
	fChargement = fopen("d:\\data.bin","rb");
 
	if( !fChargement ){
		printf("Erreur d'ouverture de fichier\n");
	}
	else{
		while(fread(nSkieur, sizeof(skieur), 1, fChargement)){
			//debug
			printf("1...%d\n",nSkieur->idLicenceFis);
			printf("2...%s\n",nSkieur->nom);
			_getch();
			//end debug
			data->type = nSkieur;
			AjouterEnFinDeListe(lst,data);
			//PElement data = (PElement) malloc (sizeof(element));
			//skieur *nSkieur = (skieur*) malloc (sizeof(skieur));
		}
	}
	printf("Fichier lut\n");
	fclose(fChargement);

[Trap D]

Ben oui, c'est normal : que fais-tu

Code :

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

data->type = nSkieur;

Tu donnes à type toujours la même adresse nSkieur

Il faut faire

Code :

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

1
2
3
4
data->type = malloc((sizeof skieur); 
// sans cast, on est en C, enfin je supose (pas de .cpp ...)
if (data->type != NULL)
  memcpy(data->type, nskieur, sizeof skieur);

[badack]

Merci, je vais essayer de ce pas.

[badack]

Trap D, ca n'a pas marché ta solution. Par contre, encore une fois, j'ai fais le boulet, car je me suis rendu compte ce matin qu'en remplacant ceci dans ma boucle while:

Code :

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

1
2
3
 
PElement data = (PElement) malloc (sizeof(element));
skieur *nSkieur = (skieur*) malloc (sizeof(skieur));

par:

Code :

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

1
2
3
 
data = (PElement) malloc (sizeof(element));
nSkieur = (skieur*) malloc (sizeof(skieur));

Mais j'ai du mal à comprendre. Si une bonne âme pouvait me rendre plus intélligent sur ce point.

[Trap D]

Tu pourrais détailler les structures pElement et skieur.
Attention, si tu as besoin du cast c'est que tu fais du C++ pas du C, donc c'est la porte à côté.

[souviron34]

ben pourquoi tes 2 allocs en fin du while sont en commentaires ??

Il n'y aurait plus les commentaires, ça marcherait mieux....

Par contre, il faut tester (y compris au début) le fait que les allocs se soient bien passées...