Discussion :

[Mimidoux]

Bonjour.
Voici mon interrogation.
Je me demande la nuance entre 2 codes.
dans un cas on déclare un pointeur et on malloc(), dans l'autre on déclare un contenu et on renvoie le pointeur.

Au niveau du mécanisme du programme, est-ce que dans le cas ou on ne fait pas de malloc, le programme gère un 'malloc' automatique qui doit être free() ? Parce que pour moi on alloue une certaine mémoire mais sans malloc.
Qu'en pensez vous ?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
struct color *color_new(char newred, char newgreen, char newblue)
{
  struct color new;
 
  new.red = newred;
  new.green = newgreen;
  new.blue = newblue;
  return (&new);
}

Code :

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

1
2
3
4
5
6
7
8
9
10
 
struct color *color_new(char red, char green, char blue)
{
     struct color *new;
new = malloc(sizeof(*new));
new->red = red;
new->blue = blue;
new->green = green;
return (new);
}

[ram-0000]

La 1ere forme est absolument à proscrire car c'est le crash assuré. En effet, tu retourne l'adresse d'une variable locale qui n'existe plus après la fin de la fonction. Donc tu retournes un pointeur fou.

La seconde forme est correcte par contre, elle t'oblige à gérer à la main la libération de mémoire avec free() lorsque tu n'en a plus besoin.

[Mimidoux]

Merci.

[vitoubien]

La seconde forme est correcte par contre, elle t'oblige à gérer à la main la libération de mémoire avec free() lorsque tu n'en a plus besoin.

Il y aurait moyen de faire autrement ?

[Médinoc]

Eh bien, il reste toujours l'option de déclarer la structure color hors de la fonction:

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void color_constructor(struct color *This, char red, char green, char blue)
{
	This->red = red;
	This->blue = blue;
	This->green = green;
}
 
int main(void)
{
	struct color c;
	color_constructor(c, 244, 243, 255);
	return 0;
}

D'ailleurs, tu peux utiliser cette même fonction dans le color_new:

Code C :

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

1
2
3
4
5
6
7
8
struct color *color_new(char red, char green, char blue)
{
	struct color *This;
	This = malloc(sizeof(*This));
	if(This != NULL)
		color_constructor(This, red, green, blue);
	return This;
}

[Jean-Marc.Bourguet]

Tu es sur de vouloir manipuler des pointeurs pour ca?

Code :

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

1
2
3
4
5
6
7
8
9
struct color color_new(char newred, char newgreen, char newblue)
{
  struct color new;
 
  new.red = newred;
  new.green = newgreen;
  new.blue = newblue;
  return new;
}

[Mimidoux]

oui.

[Mac LAK]

Beurk, faire une allocation pour un truc qui va "peser" 3 octets... 32 bits de pointeur pour 24 bits de données, c'est pas franchement une bonne idée !!! Tu vas bouffer deux fois plus de mémoire en faisant ça, alors que tu peux parfaitement effectuer l'opération donnée par Jean-Marc.Bourguet... Et en plus, tu n'auras pas d'erreurs d'allocation/libération, au moins !!

Ou, vu la simplicité de la structure, carrément mettre ça dans un entier 32 bits via des macros ou un bitfield...