Discussion :

[dream25]

Bonjour, je débute en programmation C et j'aimerais comprendre un peu plus en détail ce qui se passe lors de l'exécution de ce bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int* create(void)
{
	int b = 3;
	int *a = &b;
	*a = 5;
	return a;
}
 
int main(void)
{
	int* number = test();
	printf("%d", number);
}

Si je fais un printf de number, cela m'affiche 5. Or je ne comprends pas car ma fonction create renvoi un pointeur sur une variable locale de cette fonction (à savoir la variable b).
Mais normalement, étant donné que c'est une variable locale, elle devrait être détruite à la fin de l'exécution de la fonction create. Du coup create va renvoyer un pointeur sur une variable qui va être détruite.
Ma question est, pourquoi alors number vaut 5 ? (Aurais-je fais une allocation dynamique sans m'en rendre compte ?

[diogene]

Mais normalement, étant donné que c'est une variable locale, elle devrait être détruite à la fin de l'exécution de la fonction create. Du coup create va renvoyer un pointeur sur une variable qui va être détruite.

Exact. On ne doit jamais renvoyez l'adresse d'une variable locale.

Ma question est, pourquoi alors number vaut 5 ? (Aurais-je fais une allocation dynamique sans m'en rendre compte ?

Non, tu n'as pas fait de l'allocation dynamique sans le savoir. C'est simplement que la mémoire utilisée par la variable locale b n'a pas été encore réutilisée. Donc, pur hasard.
Dans un tel cas, le comportement est indéfini.

[haraelendil]

non tu n'as pas fait d'allocation dynamique, ne t'inquiète pas.

Ça ne viendrait pas du fait que ta fonction s'appelle create mais que dans ton main tu appelle test?

[dream25]

Ça ne viendrait pas du fait que ta fonction s'appelle create mais que dans ton main tu appelle test?

Oups j'ai mal écrit le message quand j'ai posté de mémoire mon bout de code sur le forum. Dans mon programme j'utilise bien la bonne fonction. (create et non pas test).

Merci diogene pour m'avoir donné la réponse.

Du coup si je veux que mon code soit correct je dois écrire ça ? :

Code :

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

1
2
3
4
5
6
7
int* create(void)
{
int *a = (int*)malloc(1*sizeof(int));
 
a = 5;
return a;
}

faire un malloc pour allouer un seul int me semble un peu superflux. :/

[diogene]

Du coup si je veux que mon code soit correct je dois écrire ça ? :

Oui.

faire un malloc pour allouer un seul int me semble un peu superflux. :/

C'est un exemple pour le principe.

[gl]

Du coup si je veux que mon code soit correct je dois écrire ça ? :
int* create(void)
{
int *a = (int*)malloc(1*sizeof(int));
*a = 5;
return a;
}

faire un malloc pour allouer un seul int me semble un peu superflux. :/

Si tu n'as besoin que d'un int et que tu n'as pas de nécessiter de retourner une adresse, on ferait plus probablement :

Code :

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

1
2
3
4
5
6
int create(void)
{
	int a;
	a = 5;
	return a;
}

qui se simplifie bien évidement en

Code :

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

1
2
3
4
int create(void)
{
	return 5;
}

[dancingmad]

faire un malloc pour allouer un seul int me semble un peu superflux. :/

Pour éviter de faire un malloc, tu peut également écrire cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
void create( int* a ) {
    *a = 5;
}
 
 
int main() {
    int a;
    create( &a );
    printf( "%d", a );
}

EDIT : un * qui trainait dans le printf...

[dream25]

Je suis d'accord avec toutes ces réponses, merci beaucoup.

Sinon j'aurais une autre question concernant les mallocs en C. Si je suis amené à faire des mallocs, faut-il que je garde un pointeur sur toutes les différentes allocations manuelles que j'ai faites afin de les libérer à la fin du programme (avec la fonction free) ? Comment se débrouille-t-on en pratique ? Utilise-t-on un tableau qui regroupe les différents pointeurs à libérer en fin de programme ?

[haraelendil]

Bah deux options, soit tu crées un tableau qui stock tous tes pointeurs au fur et à mesure que tu les crée, soit tu fait directement un tableau dynamique où tu utilise les éléments au fur et à mesure de tes besoins ....

[diogene]

...
Sinon j'aurais une autre question concernant les mallocs en C. Si je suis amené à faire des mallocs, faut-il que je garde un pointeur sur toutes les différentes allocations manuelles que j'ai faites afin de les libérer à la fin du programme (avec la fonction free) ? Comment se débrouille-t-on en pratique ? Utilise-t-on un tableau qui regroupe les différents pointeurs à libérer en fin de programme ?

La question ne devrait pas être posée de cette façon.
Tu crées un objet par allocation dynamique. Lorsque cet objet est devenu inutile dans le déroulement du programme, tu dois le supprimer : il n'y a aucune raison de le conserver plus longtemps et de différer sa libération.
C'est donc dicté par la logique de ton programme. Si tu suis correctement cette logique, tous les objets seront libérés une et une seule fois.