quelques questions concernant g_free, g_malloc

Version imprimable

Bonsoir,

je voudrais avoir des précisions sur l'utilisation et l'utilité des fonctions g_free et g_malloc.
est-ce que ces deux fonctions peuvent s'utiliser ainsi :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 GtkWidget **monWidget; monWidget = g_malloc(sizeof(GtkWidget **)); for(i = 0; i < 10; i++){ monWidget[i] = g_malloc(sizeof(GtkWidget *)); } //Plus loin dans le code for(i = 0; i < 10; i++){ g_free(monWidget[i]); } g_free(monWidget);
Merci de vos réponses :)

22/03/2007, 20h14
Franck.H

Oui, elles s'utilisent exactement de la même façon que les fonctions standards free et malloc, ce sont que des wrappers rien de plus ;)

si l'utilisation est exactement la même, le problème devant lequel je suis doit concerner les types des variables que je modifie. Je m'explique :
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 26 27 28 29 30 31 32 33 34 35 36 37 38 39 gint i; GtkTextBuffer **textbuf; //tableau de GtkTextBuffer alloué dynamiquement GtkTextBuffer **tmp; //tableau dans lequel on va copier textbuf pour ajouter un nouvel élément à textbuf // mettons que je veuille en allouer 5 textbuf = g_malloc(sizeof(GtkTextBuffer **) * 5); for(i = 0; i < 5; i++){ textbuf[i] = g_malloc(sizeof(GtkTextBuffer *)); } //Fin de l'allocation // .... //Début de la copie dans tmp tmp = g_malloc(sizeof(GtkTextBuffer **) * 5); for(i = 0; i < 5; i++){ tmp[i] = g_malloc(sizeof(GtkTextBuffer *)); tmp[i] = textbuf[i]; g_free(textbuf[i]); } g_free(textbuf); // Allocation d"un 6 eme élément textbuf = g_malloc(sizeof(GtkTextBuffer **) * 6); for(i = 0; i < 6; i++){ textbuf[i] = g_malloc(sizeof(GtkTextBuffer *)); textbuf[i] = tmp[i]; g_free(tmp[i]); } g_free(tmp); //bla bla bla ... :)
Ce code produit une erreur lorsque j'essaie de faire les g_free(textbuf[i]) dans la boucle de recopie du contenu de textbuf dans tmp (la 2eme boucle ;) ) et je ne comprends pas pourquoi :?
Pourriez-vous m'éclairer ?

Merci pour vos réponses

23/03/2007, 17h29
Franck.H

Ce qu'il y'a, c'est qu'on ne créé pas un GtkTextBuffer de cette manière:

Code:

textbuf[i] = g_malloc(sizeof(GtkTextBuffer *));

:sm: Chaque widget possède son propre constructeur, c'est pas pour faire beau qu'ils existent ou pour faire style !

Le mieux serait que tu post le code complet et compilable (réduit au minimum cernant le problème si le code est trop gros) car là j'avoue franchement ne pas voir de problème mis à part ce que je dit ci-dessus :lol:
23/03/2007, 18h50
bit_o

Citation:

Envoyé par Franck.H

Ce qu'il y'a, c'est qu'on ne créé pas un GtkTextBuffer de cette manière:

Code:

textbuf[i] = g_malloc(sizeof(GtkTextBuffer *));

:sm: Chaque widget possède son propre constructeur, c'est pas pour faire beau qu'ils existent ou pour faire style !

:aie:
Je suis un boulet.., merci :)

Citation:

Envoyé par Franck.H

Le mieux serait que tu post le code complet et compilable (réduit au minimum cernant le problème si le code est trop gros) car là j'avoue franchement ne pas voir de problème mis à part ce que je dit ci-dessus :lol:

Je vais essayer ce que tu m'as dit, je reposterais si besoin :)

Bonjour,

J'explique ce que je cherche à faire :
je voudrais être capable d'allouer dynamiquement un nouveau GtkTextView lorsque l'utilisateur clique sur un bouton. En gros, il faut que la fonction ci-dessous ajoute un pTextView et un pScrolled (respectivement) aux variables **pTextView et **pScrolled.

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
void ma_fonction(gint nb, gpointer data){
gint i;
win *pApp = NULL;
 
GtkWidget **tScroll;
GtkWidget **tView;
 
//le gint nb passé en parametre est le nombre d'objets contenus dans **pTextView et //**pScrolled
//pApp est le pointeur sur la structure qui contient 
//les déclarations de **pTextView et **pScrolled
pApp = (win *) data;
 
 
tView = g_malloc(sizeof(GtkWidget **) * nb);
tScroll = g_malloc(sizeof(GtkWidget **) * nb);
 
    for(i = 0; i < nb; i++){
        if(GTK_IS_TEXT_VIEW(pApp->pTextView[i])){
            tView[i] = gtk_text_view_new();
            tView[i] = pApp->pTextView[i];
 
            /* Tous les g_free font planter l'application */
            g_free(pApp->pTextView[i]);
            }
 
        if(GTK_IS_SCROLLED_WINDOW(pApp->pScrolled[i])){
            tScroll[i] = gtk_scrolled_window_new(NULL, NULL);
            tScroll[i] = pApp->pScrolled[i];
 
            g_free(pApp->pScrolled[i]);
            }
        }
 
        g_free(pApp->pTextView);
        pApp->pTextView = NULL;
        pApp->pTextView = g_malloc(sizeof(GtkWidget **) * (nb + 1));
 
        g_free(pApp->pScrolled);
        pApp->pScrolled = NULL;
        pApp->pScrolled = g_malloc(sizeof(GtkWidget **) * (nb + 1));
 
    for(i = 0; i < nb; i++){
        if(GTK_IS_TEXT_VIEW(tView[i])){
            pApp->pTextView[i] = gtk_text_view_new();
            pApp->pTextView[i] = tView[i];
 
            g_free(tView[i]);
            }
 
        if(GTK_IS_SCROLLED_WINDOW(tScroll[i])){
            pApp->pScrolled[i] = gtk_scrolled_window_new(NULL, NULL);
            pApp->pScrolled[i] = tScroll[i];
            g_free(tScroll[i]);
            }
        }
 
    g_free(tView);
    g_free(tScroll);
    tView = NULL;
    tScroll = NULL;
    }
 
}

Merci de vos réponses :)

EDIT : l'erreur dans la console est :

Code:

1
2
3
 
free() :invalid pointer: 0X80xxxxxxx <-- adresse mémoire
//ensuite la backtrace que je ne sais pas décodé ^^

24/03/2007, 17h29
Franck.H
Ok donc tu veux un tableau de widgets c'est bien ca ? Si oui et si c'est bien ce que j'ai compris, tu peux faire ainsi (si tu veux un tableau dynamique):

1.- On alloue le tableau;
Code:

1 2 3 4 5 6 7 8 9 GtkWidget ** wid = NULL; ... wid = g_malloc (n * sizeof (* wid)); if (wid == NULL) { /* Erreur d'allocation */ ... }
Ensuite, tu peux créer tes widgets à dedans mais en utilisant les constructeurs de ces widgets soit, si par exemple tu veux créer un widget de type GtkButton sur l'index 0:

Code:

wid[0] = gtk_button_new ();

Voilà mais tu ne peux pas créer de widgets par une simple allocation dynamique, programmer avec GTK+ c'est comme programmer en C pur, on peut pas le faire en jouant aux devinettes, il y'a des règles précises à respecter pour bien faire les choses ;)

Sinon au pire, passe le code complet (et compilable) réduit au minimum (si trop gros) cernant le problème que je test chez moi.
24/03/2007, 17h35
Franck.H
Citation:
Envoyé par bit_o

EDIT : l'erreur dans la console est :

Code:

1 2 3 free() :invalid pointer: 0X80xxxxxxx <-- adresse mémoire //ensuite la backtrace que je ne sais pas décodé ^^
Et comment est créé le widget:

Code:

g_free(pApp->pTextView[i]);

?
24/03/2007, 18h11
bit_o

pApp->pTextView[i] est créé par l'appel à la fonction gtk_text_view_new() à un autre endroit de mon programme ;)

ce que je cherche à faire avec l'algo que j'ai mis au dessus, c'est ajouter un nouvel emplacement dans mon tableau **pTextView et dans **pScrolled.
est-ce que mon algorithme est faux ?
24/03/2007, 18h14
Franck.H

Citation:

Envoyé par bit_o

pApp->pTextView[i] est créé par l'appel à la fonction gtk_text_view_new() à un autre endroit de mon programme ;)

Ok donc évite un g_free là dessus !

Citation:

Envoyé par bit_o

ce que je cherche à faire avec l'algo que j'ai mis au dessus, c'est ajouter un nouvel emplacement dans mon tableau **pTextView et dans **pScrolled.
est-ce que mon algorithme est faux ?

Pour agrandir ou rétrécir ton tableau, il te faut utiliser la fonction g_realloc ! L'utilisation reste la même qu'avec realloc et donc les données précédentes sont conservées !
24/03/2007, 18h21
bit_o
je n'ai jamais utilisé realloc() en fait.. :D
est-ce que je dois l'utiliser comme ça ?
Code:

1 2 g_realloc (pApp->pTextView, sizeof(GtkWidget *));
EDIT : je viens de trouver comment faire ^^
Code:

1 2 pApp->pTextView = g_realloc(pApp->pTextView , sizeof(GtkWidget *));
Merci Franck :)
24/03/2007, 18h28
Franck.H
Citation:

Envoyé par bit_o

je n'ai jamais utilisé realloc() en fait.. :D
est-ce que je dois l'utiliser comme ça ?

Ok pas de problème je te l'explique. Si pTextView est un pointeur de pointeur, soit:

Code:

GtkWidget ** pTextView

(pour l'exemple hein)

Alors tu peux faire ainsi:
Code:

1 2 3 4 5 6 7 GtkWidget ** tmp = g_realloc (pApp->pTextView, n * sizeof (* pApp->pTextView)); if (tmp != NULL) { /* On met a jour le tableau : */ pApp->pTextView = tmp; }
24/03/2007, 18h33
Franck.H
Citation:
Envoyé par bit_o

EDIT : je viens de trouver comment faire ^^

Code:

1 2 pApp->pTextView = g_realloc(pApp->pTextView , sizeof(GtkWidget *));
Il ne faut pas utiliser g_realloc de cette manière, voir un petit cours sur la bonne utilisation de realloc: http://emmanuel-delahaye.developpez....es.htm#realloc Je t'ai montré juste sur le post au-dessus comment faire "correctement" :mrgreen:
24/03/2007, 18h43
bit_o

ok merci :)
25/03/2007, 14h46
bit_o
Bonjour,

j'ai à nouveau un problème avec l'utilisation de la fonction g_realloc(), elle ne marche plus :(
je m'explique : hier la fonction dans laquelle j'utilise la fonction g_realloc marchait tres bien, et aujourd'hui elle ne marche plus. L'application est compilée sans erreur (ni warnings), seulement à l'exécution j'ai un joli Segmentation fault qui m'indique que la ligne ou je fais l'allocation ne marche pas
Code:

1 2 tmp = g_realloc(pApp->pTextView, nb * sizeof (* pApp->pTextView));
cette ligne la ne marche pas. Or nb est le nombre d'onglets ouverts que je récupère avec la fonction gtk_notebook_get_n_pages(monNotebook) (résultat que je teste), donc pas d'erreur de ce coté la.

D'ailleurs je me pose une question, est-ce que je peux faire un realloc() sur une variable qui n'a pas été allouée avec malloc() ?

Merci de vos réponses
25/03/2007, 15h33
Franck.H

Citation:

Envoyé par bit_o

D'ailleurs je me pose une question, est-ce que je peux faire un realloc() sur une variable qui n'a pas été allouée avec malloc() ?

Oui, en fait si tu passes un pointeur NULL à realloc, la fonction agit comme un malloc tout simplement !

Pour ton problème, bin le mieux serait encore que tu donnes le code complet que je puisse tester ca chez moi.
25/03/2007, 15h44
bit_o

merci pour le realloc()

poster tout mon code c'est bien trop long. Je vais faire une trace de mes algos et je finirais bien par trouver ou ça plante ^^ :)

je ne comprends pas ce qu'il se passe. Quoique je fasse j'ai l'impression que les fonctions realloc() et malloc() ne produisent pas les effets que j'espère. Je vous mets la fonction que j'utilise :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
 
//===================
//déclaration des mes widgets dans la structure :
// GtkWidget **pTextView;
// GtkWidget **pScrolled;
//===================
 
int main(int argc, char *argv[]){
 
 //Version raccourcie, on suppose que toutes les allocations se passent bien ;) 
 pApp->pTextView = g_malloc(sizeof(* pApp->pTextView));
 pApp->pScrolled = g_malloc(sizeof(* pApp->pScrolled));
 creation(NULL, (gpointer) pApp);
 
}
 
void        creation(GtkWidget *widget, gpointer data){
    win             *pApp = NULL;
    gint            nb = 0;
    GtkWidget       **tmp;
    GtkWidget       **tmp2;
 
    pApp = (win *) data;
 
    nb = gtk_notebook_get_n_pages(GTK_NOTEBOOK(pApp->pNotebook));
 
    nb == -1 ? 0 : nb;
 
    /* ajout d'un nouvel élément */
    tmp = g_realloc(pApp->pTextView, sizeof(* pApp->pTextView) * (nb + 1));
    if(tmp != NULL)
        pApp->pTextView = tmp;
 
    tmp2 = g_realloc(pApp->pScrolled, sizeof(* pApp->pScrolled) * (nb + 1));
    if(tmp2 != NULL)
       pApp->pScrolled = tmp2;
 
    pApp->pScrolled[nb] = gtk_scrolled_window_new(NULL, NULL);

Le code bloque sur la création du pScrolled[nb] (la derniere instruction)

Bon j'ai testé comme j'ai pû avec ce que j'avais mais pas de problèmes chez moi mais apparement pas de soucis, d'ailleurs je ne vois pas pour quelle raison il y en aurait !

D'abord une question ... A quoi te sert cette expression ?

Code:

nb == -1 ? 0 : nb;

Telle qu'elle est présentée là elle ne sert strictement à rien:

Citation:

/home/franhec/Documents/Programmation/CodesTests/GTK+/main.c:26: warning: statement with no effect
:: === Build finished: 0 errors, 1 warnings ===

Si c'est pour déterminer le nombre de pages, il faut savoir que la fonction gtk_notebook_get_n_pages renvoie 0 s'il n'y à pas de pages donc je vois pas bien ce que viens faire un -1 dans l'expression :aie: Tu n'as même pas besoin de tester la valeur de nb car s'il n'y pas de d'onglets, tu créé d'office avec ton code qui suit, les widgets pour le premier onglet.

Voici mon code de test:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
#include <gtk/gtk.h>
#include <glib.h>
#include <glib/gprintf.h>
 
 
typedef struct
{
   GtkWidget *  pNotebook;
   GtkWidget ** pScrolled;
}
win;
 
 
static void creation (GtkWidget * widget, gpointer data)
{
   win             *pApp = NULL;
   gint            nb = 0;
   GtkWidget       **tmp2 = NULL;
 
   pApp = data;
 
 
   nb = gtk_notebook_get_n_pages (GTK_NOTEBOOK (pApp->pNotebook));
 
 
   /* ajout d'un nouvel élément */
   tmp2 = g_realloc (pApp->pScrolled, sizeof (* pApp->pScrolled) * (nb + 1));
   if (tmp2 != NULL)
   {
      g_printf ("Re-allocation reussie !\n");
      pApp->pScrolled = tmp2;
   }
 
 
   pApp->pScrolled[nb] = gtk_scrolled_window_new (NULL, NULL);
   if (pApp->pScrolled[nb] != NULL)
   {
      g_printf ("Creation reussie !\n");
   }
 
   (void) widget;
}
 
 
int main (int argc, char ** argv)
{
   GtkWidget * p_win       = NULL;
   GtkWidget * p_vbox      = NULL;
 
 
   gtk_init (& argc, & argv);
 
 
   p_win = gtk_window_new           (GTK_WINDOW_TOPLEVEL);
   gtk_window_set_default_size      (GTK_WINDOW (p_win), 200, 200);
   gtk_container_set_border_width   (GTK_CONTAINER (p_win), 5);
   gtk_window_set_position          (GTK_WINDOW (p_win), GTK_WIN_POS_CENTER);
   g_signal_connect                 (p_win, "destroy", gtk_main_quit, NULL);
 
 
   p_vbox = gtk_vbox_new (FALSE, 5);
   gtk_container_add (GTK_CONTAINER (p_win), p_vbox);
 
 
   /* ----- */
 
   win * pApp = g_malloc (sizeof (* pApp));
 
   if (pApp != NULL)
   {
      pApp->pNotebook = gtk_notebook_new ();
      gtk_box_pack_start (GTK_BOX (p_vbox), pApp->pNotebook, TRUE, TRUE, 0);
 
      pApp->pScrolled = g_malloc (sizeof (* pApp->pScrolled));
 
      if (pApp->pScrolled != NULL)
      {
         g_printf ("Allocation reussie !\n");
         pApp->pScrolled[0] = gtk_scrolled_window_new(NULL, NULL);
      }
 
      creation (NULL, (gpointer) pApp);
   }
 
   /* ----- */
 
 
   gtk_widget_show_all (p_win);
   gtk_main ();
 
 
   return 0;
}

Voilà, je ne peut faire plus sans avoir ton code complet et le tester chez moi, si des fois ont le demande le code, c'est pas pour rien :aie:

25/03/2007, 18h23
bit_o

je croyais que ça renvoyait -1 si le notebook ne contient aucune page... C'est la raison de mon test.. Je vais tester le reste

En tout cas, merci beaucoup Franck d'avoir pris le temps de tester tout ça :)
25/03/2007, 18h36
Franck.H

Citation:

Envoyé par bit_o

je croyais que ça renvoyait -1 si le notebook ne contient aucune page... C'est la raison de mon test.. Je vais tester le reste

En tout cas, merci beaucoup Franck d'avoir pris le temps de tester tout ça :)

De rien ... on est la pour aider alors si on peut .. ;)
26/03/2007, 17h22
bit_o
bon, bon, bon je ne comprends vraiment plus rien à rien la.. Je dois surement mal faire une chose, mais je n'arrive pas à mettre la main dessus :
J'ai fait mes allocations comme dans l'exemple de Franck.H (avant-dernier message).
Les allocations se passent bien, car quand je fais ce test le programme continue.
Code:

1 2 3 4 5 6 7 if (pApp->pScrolled[0] != NULL){ // l'allocation est faite ! // suite du code //... gtk_notebook_append_page(GTK_NOTEBOOK(pApp->pNotebook), pApp->pScrolled[0], gtk_label_new("test")); }
le code plante sur l'instruction gtk_notebook_append_page. J'ai testé tous les widgets utilisés, et qu'ils sont tous bien alloués et créés (j'ai utilisé la macro GTK_IS_NOTEBOOK pour le test de pNotebook et GTK_IS_SCROLLED_WINDOW pour le test de pScrolled[0]).

Any suggestions ? :?

Citation:

Envoyé par bit_o

Any suggestions ? :?

En fait oui, j'en ai une qui permettrait surtout de mieux organiser tes pages :P

Pourquoi tu n'as pas directement fait une structure contenant les widgets à disposer sur une page d'un GtkNotebook soit pour ton cas:
Code:

1 2 3 4 5 6 7 8 9 typedef struct { GtkWidget * pScrolled; GtkWidget * pTextview; /* Pourquoi même d'autres infos concernant les pages ! */ ... } page;
Cette structure, tu peux en faire un pointeur que tu mets éventuellement dans ta structure principale de ta fenêtre soit "win" par exemple:
Code:

1 2 3 4 5 6 7 8 typedef struct { /* Les widgets de ta fenetre... */ ... /* Les pages: */ page * p; } win;
Cela te permet d'économiser des allocations car du coup tu ne te retrouve pas avec un tableau de tableau soit si tu me permet, une fonction de création de page qui commencerais par allouer (ou reallouer) ton tableau puis créer les widgets:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 page * page_add (page * p, size_t size) { page * tmp = NULL; page * _p = p; if (_p != NULL) { tmp = g_realloc (size * sizeof (* tmp)); if (tmp != NULL) { _p = tmp; _p[size].pScrolled = gtk_scrolled_window_new (NULL, NULL); _p[size].pTextView = gtk_text_view_new (); } } return _p; }
Tu pourrais même stocker chaque page dans une liste chainée style une GList par exemple donc, la structure de ta fenetre serait:
Code:

1 2 3 4 5 6 7 8 typedef struct { /* Les widgets de ta fenetre... */ ... /* Les pages: */ GList * pages; } win;
Cela permettrait également de pouvoir facilement retirer des pages car une liste est bien plus souple qu'un tableau, biensûr opter pour une liste comme celle-ci inclus une légère modification de la fonction que je t'au proposé ;)

26/03/2007, 18h20
bit_o

c'est vrai que je trouvais ma structure bien trop lourde. Je vais tester cette façon de faire
merci :)
26/03/2007, 18h22
Franck.H

Citation:

Envoyé par bit_o

c'est vrai que je trouvais ma structure bien trop lourde. Je vais tester cette façon de faire
merci :)

Attention tout de même au fait que si tu utilises une liste chaînée, ce que je recommande si tu dois par exemple supprimer une page en plein milieu du GtkNotebook, la fonction de création change elle aussi car elle n'a nullement besoin alors de construire un tableau mais simplement créer une nouvelle page puis créer les widgets comme là mais à la fin insérer la page dans la liste, liste qu'il est préférable de passer en paramètre ;)
26/03/2007, 20h36
bit_o

j'ai donc pas mal de travail en perspective, merci pour les solutions que tu m'as suggéré ;)
je m'y mets dès ce soir, je ferais signe si besoin est.