Discussion :

[BlackTwix]

Bonjour, je suis sur un problème depuis des heures et je n'arrive pas à le résoudre.

Le problème est le suivant: j'essaye de compiler le programme mais il me dit à chaque fois

Voice l'erreur:



Voici le test.c (c'est le fichier main)



Voici la fonction create



Voici la structur PNM_T (renommé PNM avec un type opaque dans un .h)



Quelqu'un pourrait m'aider car je commence à désespéré ?

Merci d'avance

[Obsidian]

Bonjour,

Le message d'erreur initial me laisse perplexe : je vois bien qu'il manque quelques pixels à gauche de ta capture mais malgré tout, il semble que le type PNM soit également connu sous le nom de « struct NM_t » et pas « struct PNM_t ».

Comme deux stuctures peuvent se faire mutuellement référence via les pointeurs et que, dans ce cas, il faut bien définir l'une avant l'autre, le compilateur peut te laisser déclarer un pointeur faisant référence à une structure pas encore définie.

S'il y a une faute de frappe dans le *.h qui définit ton type opaque, il se peut que l'erreur reste latente jusqu'au moment où l'on en a réellement besoin.

[Matt_Houston]

Le retour de la vengeance du code en captures d'écran..

C'est une remarque de style, mais en général on utilise plutôt le suffixe _t pour le typedef (pour indiquer qu'il s'agit d'un type) et le radical pour le nom de l'enregistrement (qui a justement besoin du mot-clef struct pour être utilisé comme un type, ce qui lève d'office toute ambiguïté).

Tu ne peux pas directement manipuler un type « opaque » (en créant une instance sur la pile, en utilisant sizeof, en faisant référence à ses champs, etc..) pour la simple raison que le compilateur a besoin de connaître sa définition complète - taille et position des champs - afin de réaliser ces opérations (d'où le message mentionnant incomplete type). Tout ce que tu peux faire avec une simple déclaration de struct, c'est manipuler des pointeurs vers ce type sans jamais les déréférencer.

[BlackTwix]

Merci pour vos réponses si rapide. J'ai repensé au problème sur base de vos réponses et je viens juste de trouver la solution. Il suffisait juste de créer des accesseurs pour accéder à mes variables de ma structure opaque. J'ai fait le test et cela fonctionne et je n'ai plus d'erreurs.

[Obsidian]

Merci pour vos réponses si rapide. J'ai repensé au problème sur base de vos réponses et je viens juste de trouver la solution. Il suffisait juste de créer des accesseurs pour accéder à mes variables de ma structure opaque. J'ai fait le test et cela fonctionne et je n'ai plus d'erreurs.

Oui mais ce faisant, tu contournes le problème sans le résoudre et ce n'est pas une bonne chose en soi.
Même si tu décides de choisir cette voie quand même, il faut absolument que tu comprennes à quoi est dû le bug qui t'a intéressé jusqu'ici. Qu'en est-il notamment de cette hypothétique faute de frappe ?

[BlackTwix]

Ce n'est pas une faute de frappe, c'est juste que j'ai fait une mauvaise capture d'écran

[Ngork]

Au passage, l'instruction :

Code :

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

printf("%u\n", *image_cree->nbre_colonnes);

ne peut donner autre chose qu'une erreur dans ton cas, même sans le problème de la structure opaque.

Il faudrait plutôt écrire :

Code :

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

printf("%u\n", image_cree->nbre_colonnes);

ou à la rigueur printf("%u\n", (*image_cree).nbre_colonnes); ou même printf("%u\n", image_cree[0].nbre_colonnes); ...

Evidemment, après ta définition d'accesseurs, je suppose que cela n'a plus d'importance puisque ton instruction doit maintenant ressembler à quelque chose comme :

Code :

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

printf("%u\n", pnm_get_nbre_colonnes(image_cree));

non ?

Par ailleurs, je dis peut-être une ânerie, mais j'ai l'impression que tu veux faire de la POO en C, et si c'est le cas pourquoi ne pas t'orienter vers un langage proche mais plus adapté comme C++ ou Obj-C (c'est une vraie question pas une critique) ?