[C]Passage de tableau de structures ds 1 fctn

Résolu/Fermé

Mumu - 17 nov. 2007 à 19:13
Mumu - 18 nov. 2007 à 13:52

Bonjour,

J'essaye en vain de créer une fonction qui travaille sur un tableau de structure.
On m'a expliqué qu'il était mieux de créer un tableau de pointeurs sur des structures, mais bien que je comprenne la chose, je n'arrive pas à la coder. En effet, je me retrouve avec quelque chose comme ça :

int loadDict(FILE *dictFile, struct symbolStruct** symbolList, int wordCount){
	short int i = 0;

	if ( (dictFile = fopen("dict", "r")) == NULL)
	{
		fprintf(stderr,"Dictionnary file can't be opened!\n");
		return 0;
	} 
	
	symbolList = malloc(wordCount*sizeof(struct symbolStruct*));
	for ( i = 0; i < wordCount; i++){
		symbolList[i] = malloc(sizeof(struct symbolStruct));}


	dictFile = fopen("dict", "r");*/

	symbolList = malloc(wordCount*sizeof(struct symbolStruct*));
	for ( i = 0; i < wordCount; i++)
		symbolList[i] = malloc(sizeof(struct symbolStruct));
..
}

int main(int argc, char* argv[]){
	
	FILE* dictDescriptor;
	int wordCount;
	struct symbolStruct** symbolList;

	loadDict(dictDescriptor, symbolList);

..
}

Ca compile moyennement bien, surtout au niveau des mallocs (j'ai du mal à cerner ce que je fais avec un malloc, surtout dans la fonction), les structures que j'affecte sont utilisables à l'intérieur de la fonction, mais dès que j'en sors, c'est fini...

Merci d'avance, je planche dessus depuis quelque temps déjà :(

A voir également:

11 réponses

Réponse 1 / 11

Mumu
17 nov. 2007 à 19:27

Je me suis trompé dans le quote, il n'y a évidemment qu'une seule allocation :

int loadDict(FILE *dictFile, struct symbolStruct** symbolList, int
wordCount){
	short int i = 0;

	if ( (dictFile = fopen("dict", "r")) == NULL)
	{
		fprintf(stderr,"Dictionnary file can't be opened!\n");
		return 0;
	} 
	
	symbolList = malloc(wordCount*sizeof(struct symbolStruct*));
	for ( i = 0; i < wordCount; i++){
		symbolList[i] = malloc(sizeof(struct symbolStruct));}
..
}

int main(int argc, char* argv[]){
	
	FILE* dictDescriptor;
	int wordCount;
	struct symbolStruct** symbolList;

	loadDict(dictDescriptor, symbolList);

..
}

Réponse 2 / 11

mype Messages postés 2435 Date d'inscription jeudi 1 novembre 2007 Statut Membre Dernière intervention 16 août 2010 436
17 nov. 2007 à 20:08

tu as oublié de faire le cast pour le malloc c'est surement pour ça que tu as des erreur a ce niveau

essaye ça

symbolList = (struct symbolStruct **) malloc(wordCount*sizeof(struct symbolStruct*));

symbolList[i] = (struct symbolStruct *) malloc(sizeof(struct symbolStruct));

Réponse 3 / 11

Mumu
17 nov. 2007 à 21:16

Merci de te pencher sur mon problème !

Avec ta solution, cela compile bien, et le malloc ne génère pas d'erreur. Cependant j'ai un segmentation fault lorsque j'essaye d'affecter les structures, par exemple :

(*symbolList[i]).firstPx = 2;

Réponse 4 / 11

mype Messages postés 2435 Date d'inscription jeudi 1 novembre 2007 Statut Membre Dernière intervention 16 août 2010 436
17 nov. 2007 à 21:31

en general c'est un debordement de memoire tu es sure d'avoir alloué assez de memoire ?

Vous n’avez pas trouvé la réponse que vous recherchez ?

Posez votre question

Réponse 5 / 11

Mumu
17 nov. 2007 à 21:43

Ca plante à la toute première affectation, pour i=0 :(
Soit je ne fais pas l'allocation avec malloc de la bonne façon, soit je n'affecte pas de la bonne façon... ?

Réponse 6 / 11

mype Messages postés 2435 Date d'inscription jeudi 1 novembre 2007 Statut Membre Dernière intervention 16 août 2010 436
17 nov. 2007 à 21:46

essaye sans le "*"

(symbolList[i]).firstPx = 2;

Réponse 7 / 11

Mumu
17 nov. 2007 à 22:24

request for member ‘firstPx’ in something not a structure or union..

Il est pas venu le jour où les pointeurs seront clairs pour moi :/

Réponse 8 / 11

mype Messages postés 2435 Date d'inscription jeudi 1 novembre 2007 Statut Membre Dernière intervention 16 août 2010 436
17 nov. 2007 à 23:48

c'est peu etre une question bete mais as tu placer l'affectation au bon endroit ?

for ( i = 0; i < wordCount; i++){
		symbolList[i] = malloc(sizeof(struct symbolStruct));
                (symbolList[i]).firstPx = 2;}

Mumu
18 nov. 2007 à 11:33

C'était peut-être bête mais c'était mon erreur à peu de chose près...
Je faisais d'abord une bouche pour allouer puis une seconde pour affecter et j'avais oublié de réinitialiser i à 0.. Je m'excuse :)

Ceci dit j'ai toujours un problème : à l'intérieur de la fonction où je fais l'allocation et l'affectation tout se passe bien, mais une fois sorti de celle-ci j'ai bien un segmentation fault

Réponse 9 / 11

lami20j Messages postés 21331 Date d'inscription jeudi 4 novembre 2004 Statut Modérateur, Contributeur sécurité Dernière intervention 30 octobre 2019 3 567
18 nov. 2007 à 09:43

Salut,

(*symbolList[i]).firstPx = 2;
c'est plutôt comme ça mais je préfère les flèches.

regarde cet exemple

lami20j@debian:~/trash$ cat ccm.c
#include <stdio.h>
#include <stdlib.h>

int main(void)
{
  struct symbolStruct {
          int firstPx;
          int firstPy;
  };

  struct symbolStruct** symbolList;

  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[0] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  (*symbolList[0]).firstPx = 2;
  symbolList[0]->firstPy = 20;

  printf("firstPx = %d\n",symbolList[0]->firstPx);
  printf("firstPy = %d\n",symbolList[0]->firstPy);

  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[1] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  (*symbolList[1]).firstPx = 5;
  symbolList[1]->firstPy = 50;

  printf("firstPx = %d\n",symbolList[1]->firstPx);
  printf("firstPy = %d\n",symbolList[1]->firstPy);

  return 0;
}
lami20j@debian:~/trash$ gcc ccm.c
lami20j@debian:~/trash$ ./a.out
firstPx = 2
firstPy = 20
firstPx = 5
firstPy = 50

Mumu
18 nov. 2007 à 11:47

Du coup c'est exactement ce que je fais, mais dans une fonction qui prend comme paramètre struct symbolStruct** symbolList.

Mon problème est que j'appelle cette fonction dans le main, qu'à l'intérieur de celle-ci les structures sont bien initialisées, mais que si je fais, par exemple

		printf("%d %d %d %d %c\n", symbolList[0]->firstPx, symbolList[0]->secondPx, symbolList[0]->thirdPx, symbolList[0]->fourthPx, symbolList[0]->symbol);

Les valeurs affichées sont erronées !?

Si je reprends ton exemple, voilà ce que j'essaye de faire :

#include <stdio.h>
#include <stdlib.h>

struct symbolStruct {
          int firstPx;
          int firstPy;
};

int load(struct symbolStruct** symbolList)
{


  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[0] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  symbolList[0]->firstPx = 2;
  symbolList[0]->firstPy = 20;

  printf("firstPx = %d\n",symbolList[0]->firstPx);
  printf("firstPy = %d\n",symbolList[0]->firstPy);

  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[1] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  symbolList[1]->firstPx = 5;
  symbolList[1]->firstPy = 50;

  printf("firstPx = %d\n",symbolList[1]->firstPx);
  printf("firstPy = %d\n",symbolList[1]->firstPy);

  return 0;
}

int main(void)
{
	struct symbolStruct** symbolList;
	load(symbolList);
  	printf("firstPx = %d\n",symbolList[1]->firstPx);
  	printf("firstPy = %d\n",symbolList[1]->firstPy);
	return 0;
}

Et bien entendu c'est aux printf du main que cela plante (seg fault).

Réponse 10 / 11

lami20j Messages postés 21331 Date d'inscription jeudi 4 novembre 2004 Statut Modérateur, Contributeur sécurité Dernière intervention 30 octobre 2019 3 567
18 nov. 2007 à 12:25

c'est normal puisque ta fonction retourne un entier mais tu as besoin qu'elle retourne un pointeur sur la structure

lami20j@debian:~/trash$ cat ccm.c
#include <stdio.h>
#include <stdlib.h>

struct symbolStruct {
          int firstPx;
          int firstPy;
};

struct symbolStruct ** load(struct symbolStruct** symbolList)
{


  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[0] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  symbolList[0]->firstPx = 2;
  symbolList[0]->firstPy = 20;

  printf("firstPx = %d\n",symbolList[0]->firstPx);
  printf("firstPy = %d\n",symbolList[0]->firstPy);

  symbolList = (struct symbolStruct**) malloc (sizeof(struct symbolStruct*));
  symbolList[1] = (struct symbolStruct*) malloc (sizeof(struct symbolStruct));

  symbolList[1]->firstPx = 5;
  symbolList[1]->firstPy = 50;

  printf("firstPx = %d\n",symbolList[1]->firstPx);
  printf("firstPy = %d\n",symbolList[1]->firstPy);

  return symbolList;
}

int main(void)
{
        struct symbolStruct** symbolList;
        symbolList = load(symbolList);

        printf("Dans main\n");

        printf("firstPx = %d\n",symbolList[1]->firstPx);
        printf("firstPy = %d\n",symbolList[1]->firstPy);
        return 0;
}
lami20j@debian:~/trash$ gcc ccm.c
lami20j@debian:~/trash$ ./a.out
firstPx = 2
firstPy = 20
firstPx = 5
firstPy = 50
Dans main
firstPx = 5
firstPy = 50

Réponse 11 / 11

Mumu
18 nov. 2007 à 13:52

Aaaah d'accord !
Ca m'embête un peu car le retour d'entier j'en avais besoin mais j'ai divisé la fonction en 2 du coup.
Ben tout fonctionne correctement, merci beaucoup mype et lami20j pour vos lumières ;)

Discussions similaires

Problème Chronopost : Pas d'avis de passage !!

ou sont les bananes ?

generateur de combinaison du loto 5/90

Nom de champs du tableau croisé dynamique non valide

liste de noms par ordre alphabétique

Discussions similaires

Newsletters