Discussion :

[Florent23]

Bonjour, je programme actuellement une fenêtre pour récupérer le nom de 2 joueurs, et j'ai un problème de compilation... invalid use of flexible array member qu'il me dit le méchant compilateur!

Voici le signal que je connecte au bouton blancer dans ma fenêtre:

Code :

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

1
2
3
4
 
g_signal_connect ((gpointer) blancer, "clicked",
                    G_CALLBACK (on_blancer_clicked),
                    joueura, joueurb, commencerpartie);

et voici la fonction on_blancer_clicked appelée :

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
 
void on_blancer_clicked                     (GtkButton       *button,
                                        GtkWidget *nom1, GtkWidget *nom2, 
                                        gpointer user_data)
{
 Joueur joueur1;
 Joueur joueur2;
 
 joueur1.nom = gtk_entry_get_text(GTK_ENTRY(nom1));
 joueur2.nom = gtk_entry_get_text(GTK_ENTRY(nom2));
 
 initialiser (joueur1, joueur2);
 gtk_widget_destroy(user_data);
 lancerPartie(joueur1, joueur2);
 
}

enfin voici la définition du type Joueur dont je veux récupérer les noms

Code :

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

1
2
3
4
5
6
7
 
typedef struct{
  int couleur;
  int compteur;
  ListeDeCartes jeux;
  char nom[];
    }  Joueur;

C'est donc dans les lignes où j'appelle gtk_entry_get_text que j'ai un problème. Pourtant j'ai bien des char de chaque côté, donc je ne vois pas ce qui cloche...

Merci d'avance pour l'aider que vous pourrez m'apporter!

[Kicker]

Code :

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

1
2
3
g_signal_connect ((gpointer) blancer, "clicked",
G_CALLBACK (on_blancer_clicked),
joueura, joueurb, commencerpartie);

Ça tu vois, c'est ce qu'on rêverait tous de faire
Malheureusement, tu ne peux passer qu'un seul et unique argument dans ta Callback (généralement, on utilise une structure, un tableau, ou une liste pour passer plusieurs variables).

De plus, le résultat de gtk_entry_get_text doit être copié dans une zone mémoire que tu auras allouée préalablement (y a un topic récent à ce propos).


PS : oublies pas les balises [CODE]

[Moiban]

Salut,

Pense à utiliser la balise [code], ce n'est vraiment pas très lisible là.

Pour ce qui est de ton code... excuse-moi, mais il est bourré de fautes

En C, on ne peux initialiser un tableau statique avec = qu'à la déclaration, dans ton cas il faut que tu utilise soit des tableau alloués dynamiquement, soit des tableaux à taille fixe et une boucle pour assigner la chaîne.

Ensuite, gtk_entry_get_text() renvoie un const char* qui est explicitement défini par la documentation comme privé au widget, et ne devant pas être stocké, modifié ou supprimé ; donc copie la chaîne, avec g_strdup() par exemple.

Ensuite, g_signal_connect() ne prends que 4 paramètres, le premier n'étant d'ailleurs pas un gpointer mais un GObject.
Et pareil, la fonction de callback de l'évènement clicked est définie comme suit :

Code c :

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

void user_function (GtkWidget* widget, gpointer data);

Je te conseille de potasser un petit tuto avant d'aller plus en avant

Cordialement,
moiban


[edit]
Grilled

[Florent23]

hum désolé pour la présentation, j'avais oublié ces balises :p et puis j'ai toujours eu des problèmes avec la manipulation de chaines de caractères, je sais pas pourquoi mais c'est pas du tout intuitif...

bon j'ai essayé de modifié ma structure joueur pour pouvoir modifier le nom plus facilement:

Code :

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

1
2
3
4
5
6
7
 
typedef struct{
  int couleur;
  int compteur;
  ListeDeCartes jeux;
  char *nom;
    }  Joueur;

et de remplir les noms de cette manière (en oubliant l'histoire du g_signal_connect() et ses paramètres :p je pense créer une structure pour y remédier)

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
 
void
on_blancer_clicked                     (GtkButton       *button,
                                        GtkWidget *nom1, GtkWidget *nom2, 
                                        gpointer user_data)
{
 Joueur joueur1;
 Joueur joueur2;
 const gchar *cible1;
 const gchar *cible2;
 
 cible1 = gtk_entry_get_text(GTK_ENTRY(nom1));
 cible2 = gtk_entry_get_text(GTK_ENTRY(nom2));
 
 joueur1.nom = (char *)malloc(sizeof(char)*20);
 strcpy(joueur1.nom,cible1);
 
 initialiser (joueur1, joueur2);
 gtk_widget_destroy(user_data);
 lancerPartie(joueur1, joueur2);
 
}

mais cette fois c'est au niveau du malloc que le compilateur n'est pas content... pourtant j'ai bien regardé les tuto, et il me semble que j'ai bien fait les choses est ce que le problème vient de la structure joueur?

[teuf13]

mais cette fois c'est au niveau du malloc que le compilateur n'est pas content...

Si les messages d'erreur de ton compilateur sont du type "Je suis le compilateur, et je ne suis pas content!!!", je te suggère de changer de compilateur... La plupart des compilateurs donnent des msgs pour (tenter de) décrire l'erreur qu'ils ont rencontrée...

[Florent23]

erf pardon j'avais oublié de préciser le message d'erreur: c'est toujours le même qu'avant, invalid use of flexible array member

[teuf13]

T'es sûr d'avoir modifié ton .h ? Un flexible array, c'est un char toto[]; a priori, un char *toto; c'est plus un flexible array, donc le warning ne devrait pas mentionner ça.

[Florent23]

oui en fait c'était ça mon problème, je travaille avec dev cpp et mon .h n'était pas modifié au bon endroit... d'où le problème
et j'ai réussi à faire ma structure pour passer tous mes arguments, donc merci beaucoup pour votre aide

[Moiban]

Salut,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
...
 
 Joueur joueur1;
 Joueur joueur2;
 const gchar *cible1;
 const gchar *cible2;
 
 cible1 = gtk_entry_get_text(GTK_ENTRY(nom1));
 cible2 = gtk_entry_get_text(GTK_ENTRY(nom2));
 
 joueur1.nom = (char *)malloc(sizeof(char)*20);
 strcpy(joueur1.nom,cible1);
 
...

Plutôt que d'allouer la place manuellement, et peut-être pas assez (imagine que cible1 fasse 21 cractères ? ), tu peux utiliser les fonctions bien pratique de la GLib, comme g_strdup()

Code :

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

joueur1.nom = g_strdup (gtk_entry_get_text(GTK_ENTRY(nom1)));