Discussion :

[lejohn]

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 :

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

1
2
 
typedef struct Case *Case;

mon .c

Code :

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

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 :

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

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

[souviron34]

lis le cours ou les tuto sur les structures et comment accèder aux champs..

[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

[souviron34]

d'abord ce serait :

Code :

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

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

ensuite :

Code :

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

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 :

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

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

[lejohn]

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

Code :

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

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 :

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
 
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 :

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
 
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

[Emmanuel Delahaye]

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...

[lejohn]

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.

[souviron34]

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 :

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

#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 :

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
#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 :

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

#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 ..

[/EDIT]

[diogene]

Si tu procèdes ainsi :
.h :

Code :

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

typedef struct Case *Case;

.c : définition

Code :

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

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 :

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

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

.h :

Code :

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

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.

[lejohn]

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