Discussion :

[Le Furet]

Bonjour,

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

Code :

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

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 :

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

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 ?

[diogene]

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

[Médinoc]

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

[diogene]

Médinoc :

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

[Emmanuel Delahaye]

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

Code :

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

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..

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.

Code :

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

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 :

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

1
2
3
4
5
6
 
char *ver(void)
{
   char s[] = "1.2";
   return s;
}

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

Code :

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

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 :

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

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;
}

[Le Furet]

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 :

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
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 ?

[gege2061]

Bonjour,

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é)

[Emmanuel Delahaye]

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

Code :

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

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.

[Le Furet]

Je vous remercie, je suis rassuré