Discussion :

[elishac]

Bonjour,

j'ai un petit doute sur l'utilisation de free(x), lorsque je veux que ma fonction retourne x. Doit-on faire free(x) puis return(x) (à mon avis ce return ne marche pas), ou return(x) puis free(x) (x est-il vraiment libéré?) ou faut-il créer une variable locale y=x, ou existe-t-il un moyen plus élégant?
Voici ce que je souhaite faire (tout en libérant res au bon moment) :

Code :

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

1
2
3
4
5
6
7
char *time_t_to_string (time_t t)
{
	time_t *pt = &t;
	char *res = (char*) calloc (16, sizeof(char));
	res[16] = '\0';
	strncpy (res, ctime (pt) + 4, 15);
}

Merci pour votre aide.

[Pouet_forever]

Regarde dans la <== Click on me !

[elishac]

Je connaissais l'idée de passer un pointeur à la fonction, mais j'aurais voulu l'éviter.
J'avais commencé par une chaîne statique, mais ça ne marche pas :

Code :

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

1
2
3
4
5
6
7
const char *time_t_to_string (time_t t)
{
	time_t *pt = &t;
	char res[16];
	res[16] = '\0';
	strncpy (res, ctime (pt) + 4, 15);
}

[Pouet_forever]

Avec une chaîne déclarée en 'static' ça fonctionne :

Code :

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

1
2
3
4
5
6
7
8
const char *time_t_to_string (time_t t)
{
	time_t *pt = &t;
	static char res[16];
	res[16] = '\0';
	strncpy (res, ctime (pt) + 4, 15);
	return res;
}

[diogene]

j'ai un petit doute sur l'utilisation de free(x), lorsque je veux que ma fonction retourne x. Doit-on faire free(x) puis return(x) (à mon avis ce return ne marche pas)

Il ne marche pas : tu retournerais l'adresse d'une variable détruite (même problème avec char res[16]).

ou return(x) puis free(x) (x est-il vraiment libéré?)

Non, il ne sera pas libéré parce que tu quitteras la fonction avant l'exécution du free().

-Tu ne peux donc faire le free() qu'après avoir retourné le x, donc hors de la fonction.

-Il est aussi possible de laisser la question de création/destruction du tableau au code appelant la fonction :

Code :

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

1
2
3
4
5
void time_t_to_string (time_t t, char* res)
{
	strncpy (res, ctime (&t) + 4, 15);
	res[16] = '\0';
}

[elishac]

Bonjour,
merci à vous deux pour ces réponses.
Diogène, pouvez-vous expliquer plus en détail pourquoi c'est le "même problème avec char res[16]' ?