problème pointeur sur structure

Version imprimable

Bonjour tout le monde

J'ai un problème de pointeur au sein d'une structure et je ne comprend pas l'erreur, un peu d'aide serait la bienvenue...
Merci

mon .h
Code:

1 2 typedef struct Case *Case;
mon .c
Code:

1 2 3 4 5 6 7 struct Case{ int RangCol; int NumLigne; Case CaseSuivante; void *Contenu; };
l
enfin mon main avec l'erreur: error: dereferencing pointer to incomplete type
Code:

1 2 3 4 5 6 Case c; /* appel d une fonction allouant la case c */ /*erreur au moment de recuperer le contenu de ces deux champs */ printf("Affichage de la case:\n"); printf("Case(%d,%d)",c->NumLigne,c->RangCol);
J utilise gcc comme compilo

27/04/2007, 18h34
souviron34

lis le cours ou les tuto sur les structures et comment accèder aux champs..
27/04/2007, 18h39
lejohn

J'ai regardé mais ma structure Case étant une structure Case* je ne comprend pas pourquoi je n'y ai pas accès alors que je le fais de la même manière dans une autre fonction et qu'aucune erreur n'est signalée

d'abord ce serait :

Code:

1
2
3
4
5
6
7
 
typedef struct PtrCase{
	int RangCol;			
	int NumLigne;			
	Case CaseSuivante;		
	void *Contenu;			
} Case ;

ensuite :

Code:

1
2
3
4
5
6
7
 
 
Case c;
/* appel d une fonction allouant la case c */
/*erreur au moment de recuperer le contenu de ces deux champs */
printf("Affichage de la case:\n");
	printf("Case(%d,%d)",c.NumLigne,c.RangCol);

et sinon si tu passes dans une fonction :

Code:

1
2
3
4
5
6
 
void MaFonction ( Case *c )
{
printf("Affichage de la case:\n");
	printf("Case(%d,%d)",c->NumLigne,c->RangCol);
}

J'ai modifié ma structure comme suit, elle me sert a construire une liste chainée
Code:

1 2 3 4 5 6 7 struct ptrCase{ int RangCol; int NumLigne; Case *CaseSuivante; void *Contenu; }Case;
J'utilise la fonction suivante pour l'allocation mémoire, aucun problème de compilation ni d'éxécution
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 Case* CaseAllouer(int lig,int col, void *formule) { Case *c,*tmp; tmp->Contenu=formule; tmp->RangCol=col; tmp->NumLigne=lig; tmp->CaseSuivante=NULL; if ((c=malloc(sizeof(tmp)))==NULL) { fprintf(stderr,"Erreur de la creation de case\n"); return NULL; } else { *c=*tmp; return c; } }
Voici le code complet de mon main que je viens de modifier, je suis debutant sur le forum et ai du mal expliqué mon probleme, l'erreur est toujours au même endroit lorsque je cherche à accéder au contenu
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 int main() { Case *c; int coord[2],i; int contenu; printf("Test de l allocation d une case\n"); printf("Entrez les coordonnees de la case: "); for(i=0;i<2;i++) scanf("%d",coord+i); printf("\nEntrez l entier que vous voulez inserer comme contenu"); scanf("%d",&contenu); printf("c parti pour caseallouer\n"); c=CaseAllouer(coord[0],coord[1],&contenu); printf("\nFin de la creation de la case\n"); printf("Affichage de la case:\n"); printf("Case(%d,%d)",c->NumLigne,c->RangCol); exit(0); }
Lorsque je compile toutes mes fonctions dans un seul et même fichier il n'y a aucun problème, je n'arrive pas à comprendre pourquoi ça na marche pas en compilation séparée

27/04/2007, 20h23
Emmanuel Delahaye

Citation:

Envoyé par lejohn

Lorsque je compile toutes mes fonctions dans un seul et même fichier il n'y a aucun problème, je n'arrive pas à comprendre pourquoi ça na marche pas en compilation séparée

Mets tout tes fichiers en pièce jointes. Il doit y avoir un bug dans les headers...
28/04/2007, 09h57
lejohn

3 pièce(s) jointe(s)

Voila ci joint mes 3 fichiers, la compilation se déroule sans aucun problème sous gcc pour matrice.c et matrice.h, lorsque je compile matrice.o avec test_matrice, il y a erreur dereferencing pointer to incomplete type au niveau du dernier printf, lorsque je cherche à atteindre les champs de ma cellule.

bon je crois qu'il faudrait sérieusement que tu révises ton chapitre sur les pointeurs, sur les include, etc...

il y a pas mal de choses incorrectes, et en plus tu ne tiens pas compte de ce qu'on t'a dit :

Déjà tes définitions de structures DOIVENT être dans ton .h, pas juste les pointeurs.

D'autre part, tu confonds les typedef, les pointeurs, et les structures... Regarde ton cours..

Voici ce que devrait être ton .h :

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

#ifndef _MATRICE_H
#define _MATRICE_H

#include <stdlib.h>

typedef struct PCase {
	int RangCol;			/* numero de la colonne ou est la case */
	int NumLigne;			/* numero de la ligne ou est la case */
	struct PCase *CaseSuivante;		/* pointeur sur la case NON VIDE suivante, NULL si derniere ou seule case de la ligne */
	void *Contenu;			/* champ ou sera rentree la formule  */
} Case ;

typedef struct PLigne {
	int NbrCases;				/*  nbr de cases constituant au moment le chainage */
	int NumLigne;				/* numero de la ligne */
	struct PLigne *LigneSuivante;		/* pointeur sur la ligne suivante, NULL si pas d autres lignes */
	Case *PemiereCase;			/* pointeur sur la premiere case NON VIDE de la ligne */
} Ligne;	

typedef struct PMatrice {
	int NbrLignesMax;	    /*  nbr de lignes maximum que poura contenir le tableau  */
	int NbrColMax;		    /*  nbr de colonnes maximum...                                            */
	Ligne PremiereLigne;    /* Pointeur sur la premiereligne du tableau                        */
	int NbrLignesEnCours;   /* utile ??? nbr de ligne en cours dans le tableau              */
} Matrice ;




extern Case *CaseAllouer(int lig,int col, void *formule);

#endif

Ensuite, comme tu ne tiens pas compte de ce qu'on te dit dans les réponses, voici ce que devrait être ton matrice.c :

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
#include "Matrice.h"




/************************* CaseAllouer ************************************* 
*		# renvoi l'adresse de la case que l on vient de creer, NULL si probleme 
*		# rang: correspond au rang de la case au sein de la ligne= numero de colonne
*		# formule: contenu de la case, de type void
*		# Utilisation d une case temporaire tmp pour allouer la bonne quantite  d espace memoire, probleme du au contenu generique -> on ne connait pas la taille d une formule... 
*/
Case *CaseAllouer(int lig,int col, void *formule)
{
	Case *c=NULL;		
	if ((c=malloc(sizeof(Case)))==NULL)
	{
		fprintf(stderr,"Erreur de la creation de case\n");
		return NULL;
	}
	else
	{
	  (*c).Contenu=formule;
	  (*c).RangCol=col;
	  (*c).NumLigne=lig;
	  (*c).CaseSuivante=NULL;
	  return c;
	}
}

Et enfin, ton test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23

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

int main()
{
	Case *c=NULL;
	int coord[2],i;
	int contenu;
	printf("Test de l allocation d une case\n");
	printf("Entrez les coordonnees de la case:   ");
	for(i=0;i<2;i++) scanf("%d",coord+i);
	printf("\nEntrez l entier que vous voulez inserer comme contenu");
	scanf("%d",&contenu);
	printf("c parti pour caseallouer\n");
	c=CaseAllouer(coord[0],coord[1],&contenu);
	printf("\nFin de la creation de la case\n");
	printf("Affichage de la case:\n");
	printf("Case(%d,%d) contenu:%d",(*c).NumLigne,(*c).RangCol,*((int*)(*c).Contenu));
	exit(0);
}

[EDIT]

Je ne l'ai pas mentionné avant, mais ce code-là compile sans erreur, et donne les bons résultats .. :P

[/EDIT]

30/04/2007, 13h58
diogene

Si tu procèdes ainsi :
.h :

Code:

typedef struct Case *Case;

.c : définition

Code:

struct Case{....};

Tu masques complètement le contenu de la struct Case au code extérieur au .c .
Ce peut être voulu, si tu veux rendre opaque le contenu de struct Case aux utilisateurs et les empècher de lire ou de modifier directement les champs de la structure. Dans ce cas, il faut écrire des fonction d'accès pour les champs auxquels l'utilisateur a la permission d'accéder (en lecture ou/et en écriture) dont le prototype sera dans le .h
Exemple :
.c :

Code:

int GetNumLigne(Case p) {return p->NumLigne);

.h :

Code:

int GetNumLigne( Case p);

permet la lecture mais pas la modification du champ NumLigne.

Si tu n'as pas besoin d'une telle protection, mets la définition de struct Case dans le .h.
01/05/2007, 13h17
lejohn

Merci Diogène, maintenant je comprends mon erreur...