Discussion :

[rewi9]

Bonjour,

Pour un projet d'école, je dois faire une application. J'ai besoin de récupérer plusieurs entrées gtk_entry...
Je vous joins l'appel de ma fonction, ma fonction et quelques éléments en plus.

Mon problème est que je n'arrive pas à compiler... J'avais essayé d'initialiser à nouveau "variableSimulation* variableS; variableS=malloc(sizeof(variableSimulation));" en local dans la fonction "affiche_production" afin de récupérer les entrées dans ma structure sauf que j'avais une erreur segmentation lorsque j'appuyais sur mon bouton sur mon interface. Après de multiples recherches je suis bloqué

A bientôt et merci à vous!

Rémi

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
35
 
typedef struct variableSimulation {
	GtkWidget *SimuWindow;
	GtkWidget *SimuTable;
	GtkWidget *SimuButton[5];
	GtkWidget *SimuLabel[29];
	GtkWidget *sEntry[5];
}variableSimulation;
 
void affiche_production (GtkWidget *widget, sgpointer *variableS);
 
int main( int argc, char **argv)
{
        variableSimulation* variableS;	
 
	variableS=malloc(sizeof(variableSimulation));	
	gtk_init(&argc,&argv);
 
//fonction qui créer une fenetre et qui affiche les 5 entrées
 
        g_signal_connect(G_OBJECT(variableS->SimuButton[4]),"clicked", G_CALLBACK(affiche_production), (gpointer) variableS);
 
        gtk_main();
 
void affiche_production (GtkWidget *widget, gpointer *variableS)
{
	const gchar *sText;
	int i;
 
	for(i=0;i<5;i++)
	{
		sText = gtk_entry_get_text(GTK_ENTRY(variableS->sEntry[i]));
		printf("%s\n",sText);
	}
}

[liberforce]

Bonjour,

ton code est trop incomplet. On ne voit pas la création des widgets. Je te conseille donc soit de mettre une version plus proche de ce que tu essaies de compiler. Si tu as une erreur de segmentation, je te conseille aussi l'usage d'un débogueur. Il s'arrêtera directement à l'endroit de l'instruction qui pose problème.

[gerald3d]

Bonjour.

Histoire de discuter tout de même du bout de code que tu nous as transmis, tu déclares dans la fonction main(); un pointeur variableS. Tout d'abord, si ton programme finit par avoir 10000 lignes de codes, le nom variableS te fera penser à une variable, et non à un pointeur. Essaye de trouver des noms explicites à la première lecture, ou bien à une forme de typage qui peut être de ton propre crû. Par exemple, et ceci n'est bien sûr qu'un exemple, tu pourrais nommer les pointeurs avec la lettre "P". Ainsi, toute donnée commençant par "P" te dirait "pointeur".

Ensuite, variableS est déclaré dans le main();. Donc, tant que le programme ne sort de cette fonction, ce pointeur existe. Dans quel cas le programme sort du main(); ? Lorsque le programme s'arrête. Comme tu alloues dans le tas avec malloc();, cette espace mémoire doit être libéré avant la sortie de main();. Il manque donc un free(variableS);. D'aucuns me diront que cette libération ici sera effectuée par le système d'exploitation. Soit ! Mais c'est un principe de programmation. Si la libération est oubliée ici, elle le sera sûrement pour d'autres allocations au cœur du programme.

Pour finir, et c'est là que les choses deviennent intéressantes , comme variableS existe tout le long de l'exécution, il n'est pas nécessaire de faire une allocation dans le tas. Il suffit de déclarer une simple variable et de transmettre l'adresse de celle-çi au reste du programme. Ainsi, pas besoin de désallouer et le nom "variableS" lui va cette fois çi comme un charme.

Elle est pas belle la vie ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
int main( int argc, char **argv)
{
        variableSimulation variableS;
 
	variableS=malloc(sizeof(variableSimulation);
	gtk_init(&argc,&argv);
 
//fonction qui créer une fenetre et qui affiche les 5 entrées
 
        g_signal_connect(G_OBJECT(variableS->SimuButton[4]),"clicked", G_CALLBACK(affiche_production), &variableS);
...