Fonction retournant un pointeur

Bonjour,

jusqu'à présent, lorsque j'écrivais une fonction retournant un pointeur, je faisais ça :

Code:

---

1 2 3 4 5 6

char *ma_fonction(...) char *result ;   // traitement return result ; }

---

Mais on m'a dit qu'il fallait déclarer le pointeur de retour en static :

Code:

---

1 2 3 4 5 6

char *ma_fonction(...) static char *result ;   // traitement return result ; }

---

Pourtant, j'avais jamais eu de problèmes avec ma façon de faire. Alors ?

Citation:

---

Mais on m'a dit qu'il fallait déclarer le pointeur de retour en static :

---

Non, ce n'est pas nécessaire. La première formulation renvoie la valeur de result
La déclaration static permet que la variable ne soit pas créée à chaque appel de la fonction et détruite en sortie de fonction mais possède la durée de vie du programme après sa création. result désigne alors toujours le même objet . on utilisera static si on veut que cette variable conserve sa valeur entre deux appels de la fonction.
Tout cela n'est pas spécifique aux pointeurs

Pour ce que tu retournes, à première vue, il n'y a aucun problème (je suppose que tu alloues le pointeur en question avec malloc())

Le problème, c'est quand tu retournes l'adresse d'une variable locale: Là, la variable locale en question doit être déclarée en static, sinon elle est détruite quand la fonction retourne (et là, le pointeur retourné est donc invalide)

À savoir aussi, losqu'on retourne l'adresse d'une variable statique, cela peut causer des problèmes en programmation multithread, car la variable est la même pour tous les threads

Médinoc :

Citation:

---

Le problème, c'est quand tu retournes l'adresse d'une variable locale: Là, la variable locale en question doit être déclarée en static, sinon elle est détruite quand la fonction retourne (et là, le pointeur retourné est donc invalide)

---

Tout à fait
Ce qui signifie, pour la situation proposée par Le Furet, que dans ce cas ce n'est pas le pointeur qui doit être static mais la variable dont l'adresse est dans le pointeur

Citation:

---

Envoyé par Le Furet

jusqu'à présent, lorsque j'écrivais une fonction retournant un pointeur, je faisais ça :

Code:

---

1 2 3 4 5 6

char *ma_fonction(...) char *result ;   // traitement return result ; }

---

---

Correct si le pointeur a été correctement initialisé, notamment avec une adresse encore valide après exécution..

Citation:

---

Mais on m'a dit qu'il fallait déclarer le pointeur de retour en static :

---

Soit tu a mal compris, soit 'on' est un ane.

Citation:

---

Code:

---

1 2 3 4 5 6

char *ma_fonction(...) static char *result ;   // traitement return result ; }

---

Pourtant, j'avais jamais eu de problèmes avec ma façon de faire. Alors ?

---

Alors l'important est que l'adresse retrournée soit valide après exécution de la fonction :
Incorrect (adresse d'une variable non persistante, car locale)

Code:

---

1 2 3 4 5 6

char *ver(void) { char s[] = "1.2"; return s; }

---

Correct (adresse d'une variable persistante, car static)

Code:

---

1 2 3 4 5 6

char *ver(void) { static char s[] = "1.2"; return s; }

---

Correct (adresse d'une variable persistante, car alloué dynamiquement)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

char *ver_dyn(void) { char s[] = "1.2"; char *p = malloc (strlen (s) + 1);   if (p!=NULL) { strcpy (p, s); } return p; }

---

Merci de m'avoir répondu et fixé les idées sur ces cas-là.

La portion de code qu'on m'a critiquée est celle-ci (c'est du GTK/GDK/Glib) :

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

GdkPixbuf *gdk_pixbuf_new_from_text ( gchar *text, GtkWidget *widget ) { GdkPixmap *temp_pixmap ; GdkPixbuf *temp_pixbuf = NULL ; PangoLayout *layout ; GdkGC *gc ; GdkColormap *cmap ; GdkColor *fg, *bg ; int sx, sy ;   /* foreground and background colors */ fg = gdk_color_new_from_RGB(255, 0, 0) ; bg = gdk_color_new_from_RGB(0, 0, 0) ;   /* render layout to temp_pixmap */ layout = gtk_widget_create_pango_layout(widget, text) ; pango_layout_get_pixel_size(layout, &sx, &sy) ; temp_pixmap = gdk_pixmap_new(NULL, sx, sy, 24) ; gc = gdk_gc_new(temp_pixmap) ;   gdk_gc_set_foreground(gc, fg) ; gdk_gc_set_background(gc, bg) ; gdk_draw_layout( temp_pixmap, gc, 0, 0, layout) ;   /* render temp_pixmap to temp_pixbuf */ cmap = gdk_colormap_get_system() ; if (cmap==NULL) printf("cmap NULL\n") ; if (temp_pixmap==NULL) printf("temp_pixmap NULL\n") ; temp_pixbuf = gdk_pixbuf_get_from_drawable(NULL, temp_pixmap, cmap, 0, 0, 0, 0, sx, sy) ;   /* free */ g_object_free(layout) ; g_object_free(cmap) ; g_object_free(gc) ;   return temp_pixbuf ; }

---

On m'a conseillé de déclarer le temp_pixbuf en static, pour la raison que l'adresse de temp_pixbuf serait locale. Je suis prêt à le croire, mais comment aurais-je pu le deviner ?

Bonjour,

Citation:

---

Envoyé par Le Furet

On m'a conseillé de déclarer le temp_pixbuf en static, pour la raison que l'adresse de temp_pixbuf serait locale. Je suis prêt à le croire, mais comment aurais-je pu le deviner ?

---

Il n'y a aucun problème, l'adresse que tu retourne est donnée par Gtk et est valable tant que l'objet correspondant n'ai pas détruit (lors de la destruction du GdkPixmap associé)

Citation:

---

Envoyé par Le Furet

La portion de code qu'on m'a critiquée est celle-ci (c'est du GTK/GDK/Glib) :

Code:

---

1 2 3 4 5 6 7

GdkPixbuf *gdk_pixbuf_new_from_text ( gchar *text, GtkWidget *widget ) { GdkPixbuf *temp_pixbuf = NULL ; <...> temp_pixbuf = gdk_pixbuf_get_from_drawable(NULL, temp_pixmap, cmap, 0, 0, 0, 0, sx, sy) ; <...> return temp_pixbuf ; }

---

On m'a conseillé de déclarer le temp_pixbuf en static, pour la raison que l'adresse de temp_pixbuf serait locale. Je suis prêt à le croire, mais comment aurais-je pu le deviner ?

---

C'est confirmé, 'on' est un ane. Ton code est correct.

Je vous remercie, je suis rassuré :D