structure, tableau et pointeur

Bonjour a tous, donc je pratique le C depuis quelques mois ainsi que GTK.
Je n'arrive pas à mettre le doigt sur une erreur concernant apparement un pointeur permettant de remonter dans une structure seulement le point où je souhaite remonter est un tableau.
Voici le code:

la structure en question

Code:

---

1 2 3 4 5 6 7 8 9 10 11

typedef struct {   GtkWidget* widget; GtkWidget* image; _Bool libre; short int proprietaire; guint posX; guint posY; struct reversiCase* pPlateau;   }reversiCase;

---

le type guint correspond au type unsigned int défini dans Glib

Le tableau

Code:

---

reversiCase plateau[LGMAX][LGMAX];

---

LGMAX est une constante paire.

La partie du code ou ça ne va pas:

Code:

---

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

for( y = 0; y < LGMAX; y++ ) { pHBoxLignePlateau[y] = gtk_hbox_new(FALSE, 0); gtk_box_pack_start( GTK_BOX( pVBoxPlateau ), pHBoxLignePlateau[y], FALSE, FALSE, 0); for( x = 0; x < LGMAX; x++) { plateau[x][y].widget = gtk_button_new(); plateau[x][y].image = gtk_image_new(); plateau[x][y].libre = TRUE; plateau[x][y].posX = x; plateau[x][y].posY = y; plateau[x][y].pPlateau = &plateau[x][y]; //WARNING ici ! gtk_container_add(GTK_CONTAINER(plateau[x][y].widget), plateau[x][y].image); gtk_widget_set_usize(plateau[x][y].widget, 50, 50); g_signal_connect(G_OBJECT(plateau[x][y].widget), "clicked", G_CALLBACK(clickCase), plateau[x][y].pPlateau); gtk_box_pack_start( GTK_BOX( pHBoxLignePlateau[y] ), plateau[x][y].widget, FALSE, FALSE, 0); } }

---

ici, x et y sont de type int, et voici ma fonction "clickCase"

Code:

---

1 2 3 4 5 6

void clickCase( GtkWidget* pWidget, gpointer* pData ){ gtk_image_set_from_file(GTK_IMAGE(plateau[pData->posX][pData->posY].image),"images/pion_blanc.png"); //ERREUR ici ! printf("fonction executee\n"); }

---

Les erreurs montrées par le compilateur (gcc sous devcpp):

Citation:

---

request for member `posX' in something not a structure or union
request for member `posY' in something not a structure or union

---

g_signal_connect(); ne prend que 4 arguments, dont le dernier est l'argument à passer à la fonction utilisé par le Callback (ici clickCase). Hors j'ai besoin d'en passer plusieurs, par conséquent j'ai crée la structure reversiCase dans laquelle se situe toutes les données que je souhaite passer en argument à clickCase (du moins certains) avec le pointeur pPlateau qui me permet d'avoir accès à tout les champs de la structure (c'est ce pointeur que je souhaite passer en argument à clickCase via le g_signal_connect(); ).

En espèrant etre assez claire, merci d'avance.

Bonjour,

c'est plutot bizzare que la définition de la structure ne génére pas d'erreur à la compilation, normalement il faut écrire :

Code:

---

1 2 3 4 5 6 7 8 9 10

typedef struct reversiCase {   GtkWidget* widget; GtkWidget* image; _Bool libre; short int proprietaire; guint posX; guint posY; struct reversiCase* pPlateau; }reversiCase_s;

---

Par contre tu signale deux erreurs dans le code qui sont différentes des messages du compilateur :koi:

Citation:

---

Envoyé par gege2061

Bonjour,
c'est plutot bizzare que la définition de la structure ne génére pas d'erreur à la compilation....

---

En effet c'est très bizarre.
D'ailleurs pourquoi la définition suivante ne génère pas d'erreur (avec comme option de compilation -Wall uniquement) ? :

Code:

---

1 2 3 4 5

typedef struct { int a; int b; struct nimportequoi *c; } mystruct;

---

Et celle là en génère une !!!

Code:

---

1 2 3 4 5

typedef struct { int a; int b; struct nimportequoi c; } mystruct;

---

Que faudrait-il mettre comme option de compilation pour faire hurler le compilateur dans tout les cas ?

Citation:

---

Envoyé par crocodilex

D'ailleurs pourquoi la définition suivante ne génère pas d'erreur (avec comme option de compilation -Wall uniquement) ?

---

Même avec le mode parano (-W -Wall -O2 -ansi -pedantic) je n'ai pas d'erreur :(

C'est normal : C'est indispensable pour des références croisées: Un pointeur de structure DOIT pouvoir être déclaré même si le compilo ne sait pas ce qu'il y a au bout (il doit juste savoir que c'est une structure).

Votre erreur, elle vient d'une tentative de conversion de pointeur vers tableau à deux dimensions, sans typecast d'aucune sorte.
Le compilateur ne peut pas deviner la seconde dimension...
[ Erreur : j'ai mal lu le code ]

Citation:

---

Envoyé par Médinoc

Votre erreur, elle vient d'une tentative de conversion de pointeur vers tableau à deux dimensions, sans typecast d'aucune sorte.
Le compilateur ne peut pas deviner la seconde dimension...

---

Tu parles de quelle erreur (dans quelle partie du code) ?

J'ai rien dit.

Par contre, le "c'est normal" parlait du comportement que tu signalais dans ton post.



Pour l'erreur: pData est un gPointer, pas un reversiCase * (ou un struct reversicase *, puisqu'on discute sur ce sujet)

Citation:

---

Envoyé par Médinoc

J'ai rien dit.

---

OK, je n'ai rien entendu...

Citation:

---

Envoyé par Médinoc

Par contre, le "c'est normal" parlait du comportement que tu signalais dans ton post.

---

Pour ça OK, j'avais entendu. ;)

Mais pourtant le pointeur est bien déclaré dans la structure comme pointant vers quelque chose de type "reversiCase" donc la ligne:

Code:

---

plateau[x][y].pPlateau = &plateau[x][y];

---

ne devrai pas générer de warning.
Ou bien il y a une méthode particuliere pour le faire pointé vers la partie "parente" de la structure (c'est ce que je cherche a faire), c'est a dire sur plateau[x][y] (sans champs).

Si tu pouvais poster le warning lui-même, ce serait pas mal...

PS: tu as corrigé la structure depuis? en C, reversiCase et struct reversiCase sont deux types différents... (pas en C++, il me semble)


Edit: c'est la fête du cross-post, ici!

PS: Amstrad CPC6128 for ever!

Citation:

---

Envoyé par Phil'

Mais pourtant le pointeur est bien déclaré dans la structure comme pointant vers quelque chose de type "reversiCase" donc la ligne:

Code:

---

plateau[x][y].pPlateau = &plateau[x][y];

---

ne devrai pas générer de warning.
Ou bien il y a une méthode particuliere pour le faire pointé vers la partie "parente" de la structure (c'est ce que je cherche a faire), c'est a dire sur plateau[x][y] (sans champs).

---

As-tu fais la correction proposé par gege2061 ?

Ah c'est bon, un grand merci à Médinoc; je m'explique:

Je pensais que g_signal_connect prenai uniquement les type void* en dernier argument, j'ai donc changé le gpointer* du prototype de ma fonction pour reversiCase*. Resultat le compilo accepte sans broncher.
Cependant le warning subsiste toujours, mais le programme est fonctionnel, sans probleme apparant.

EDIT: voici le warning en question:

Citation:

---

341 D:\Programmation\Projet GTK\2.4.9\main.c [Warning] assignment from incompatible pointer type

---

À mon avis, les callback n'acceptent en effet que les gpointer. (et pas gpointer *)

Mon conseil: dans ta fonction clickcase, tu fais juste reversiCase *pCase = pData : en théorie, ça devrait compiler sans broncher.

Et pour la structure, tu DOIS la corriger pour supprimer le warning.
Et aussi, plutôt que &plateau[x][y], essaie &(plateau[x][y]) car je ne suis pas certain de la priorité des opérateurs, et là ce sera plus explicite.

Citation:

---

Envoyé par Phil'

Ah c'est bon, un grand merci à Médinoc; je m'explique:

Je pensais que g_signal_connect prenai uniquement les type void* en dernier argument, j'ai donc changé le gpointer* du prototype de ma fonction pour reversiCase*. Resultat le compilo accepte sans broncher.
Cependant le warning subsiste toujours, mais le programme est fonctionnel, sans probleme apparant.

EDIT: voici le warning en question:

---

Aïe, aïe, aïe....
Si tu n'as pas corrigé la définition de ta structure comme le proposait gege2061, ton problème est toujours là.
Si j'ai bien compris ce que tu viens de faire, tu as juste masqué le problème par un horrible cast.

Le typedef ne permet pas uniquement de crée un alias pour eviter de taper struct ?

Sinon je n'ai rien casté, j'ai juste modifier le prototype de la fonction pour qu'elle prenne un argument de type reversiCase*, comme est défini le pointeur dans ma structure. Puis sinon le probleme ne viendrait pas du fait que le pointeur pointe vers aucun champ de la structure mais simplement le tableau ?

Je vais corriger la strucuture pour voir si sa résout le probleme.

Je pense que la macro G_CALLBACK fait l'horrible cast dont on t'accuse.

Et ici, tu as deux types différents:
struct { tout plein de trucs }, alias reversiCase
struct reversiCase, non-définie.

Citation:

---

Envoyé par gege2061

c'est plutot bizzare que la définition de la structure ne génére pas d'erreur à la compilation, normalement il faut écrire :

Code:

---

1 2 3 4

typedef struct reversiCase {   struct reversiCase* pPlateau; }reversiCase_s;

---

---

Rien de bizarre.

Code:

---

struct s *p;

---

est une définition valide.
de même que

Code:

---

1 2 3 4

struct s;   struct s *p;

---

Code:

---

1 2 3 4 5

struct s { struct s *p; };

---

Code:

---

1 2 3 4 5 6

typedef struct s { struct s *p; } s_s;

---

et

Code:

---

1 2 3 4 5 6

typedef struct { struct s *p; } s_s;

---

Le dernier cas est, il est vrai un peu étrange, mais valide. Simplement, on a pas le lien (alias) entre struct s et s_s.

Vérification :

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

int main (void) { struct sa *pa; struct sb;   struct sc *pc; struct sd { struct sd *p; }; typedef struct se { struct se *p; } s_se;   typedef struct { struct sf *p; } s_sf;   return 0; }

---