Discussion :

[Davide]

Bonjour,

Devant faire rentrer dans différents fichiers des listes de structures différentes ( ex : fichier1 -> listeA (contenant x structZ), fichier2 -> listeB(contenant x structY), .. ) j'ai voulu faire abstraction du type de la structure pour ne devoir créer qu'une fonction d'écriture et une de lecture. Pour la version ecriture pas de problème.. je passe le nom du fichier, la taille de la structure et la liste chainée et l'écriture dans le fichier est nickel à l'aide de la fonction fwrite.

Par contre lors de la lecture, je dois passer un buffer à fread mais je ne connais que la taille de la structure mais je n'ai pas son type sous main!
Comment puis-je réserver suffisement d'espace mémoire un pointeur sur void avec un malloc égal à la taille des structures?

Voici un bout de code qui ressemble au mien (je n'ai pas mon code sous la main..)

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
 
..
....
......
#define MAXITEMS 200
liste *FichierLecture(char *FichierPath, size_t structSize)
  FILE *f;
 
  STRUCTINCONNU mem = (STRUCTINCONNU *)malloc(MAXITEMS * structSize);
 
 
  f = fopen(FichierPath,"r");
 
  fread(mem, structSize, MAXITEMS, f)
 
  fclose(f);
 
 // Mise en liste utilisant des pointeurs sur void
  ..
  ....
  ......
 
  liste *l  = ...;
  ..
  ....
  .......
 
  free(mem);
 
  return l;
}

J'espère que quelqu'un pourra m'aider..

En vous remerciant,

Davide

[Médinoc]

Déjà, je ne sais pas s'il est très judicieux ou non de faire ça. Mais je ne suis pas assez doué en génie logiciel pour répondre à ça...

<edit: supprimé des bêtises>
Ben si tu connais la taille, où est le problème pour le malloc?

[Trap D]

Si tu fais du fread, il faut ouvrir en binaire,

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
char *FichierLecture(char *FichierPath, size_t structSize) 
  FILE *f; 
 
  // le cast est inutile (et à la limite nuisible).
  char *mem = malloc(MAXITEMS * structSize); 
 
  // teste le retour de malloc, on ne sait jamais
  f = fopen(FichierPath,"rb"); 
 
  fread(mem, structSize, MAXITEMS, f) 
 
  fclose(f); 
/* ce n'est pas à cette fonction de faire les transformations 
 // Mise en liste utilisant des pointeurs sur void 
  .. 
  .... 
  ...... 
 
  liste *l  = ...; 
  .. 
  .... 
  ....... 
 
  free(mem); 
 
  return l; 
*/
  return mem;
}

Maintenant, pour ton problème, je retournerais l'adresse du tableau mem (mise sous la forme d'un char *) et c'est la fonctionn appelante qui créerais la liste en utilisant les casts appropriés.

[Davide]

Je vais tester ta solution.. je te remercie

[Davide]

Malheureusement ca ne fonctionne pas..
Une fois le cast de char* vers ma structure faite ou le pointeur est nul ou il pointe n'importe où et ca ne donne rien..

[diogene]

Trap D

je retournerais l'adresse du tableau mem (mise sous la forme d'un char *)

La logique voudrait que ceci soit mis sous la forme d'un void *
Davide

Une fois le cast de char* vers ma structure faite ou le pointeur est nul ou il pointe n'importe où et ca ne donne rien..

Mais je ne pense pas que le problème vienne de ce char*. On n'a pas assez de code pour conclure sur ce problème

[Davide]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	size_t nbrItems;
	void *p =  FichierLire(sizeof(competiteur), szFichierCompetiteur, &nbrItems);
 
	liste *l = ListeInitialiser();
 
	// Remise en liste du tableau
	for(int i = 0; i<nbrItems; i++ )
	{
		item *it = (item*)malloc(sizeof(item));
		competiteur *comp = (competiteur*)p[i];
		it->data = comp;
		ListeAjouterItemFin(l, it);
	}

Il me donne une erreur "unknow size" sur cette ligne..

Code :

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

1
2
 
		competiteur *comp = (competiteur*)p[i];

[Emmanuel Delahaye]

Il me donne une erreur "unknow size" sur cette ligne..

Code :

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

		competiteur *comp = (competiteur*)p[i];

p étant de type void*, p[i] n'a aucun sens. Il faut définir un pointeur typé et l'initialiser avec p...

Code :

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

1
2
3
 
competiteur *a_comp = p;
competiteur *p_comp = a_comp + i;

[diogene]

Il me donne une erreur "unknow size" sur cette ligne..

Voir le message d'Emmanuel ou encore :

Code :

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

competiteur *comp = ((competiteur*)p)[i];

[Davide]

Malheureusement celà ne fonctionne pas.. oui à la compilation oui.. mais au runtime je ne réceptionne que des gribouillages dans ma structure.. pourtant le fread me retourne bien le bon nombre d' "instances" de la structure.. Et la fonction d'écriture logiquement fonctionne vu que je l'ai testée avant..

[Davide]

Voilà c'est rêglé.. c'était ma fonction d'écriture qui foirait.. lol

Je vous remercie pour votre aide