Discussion :

[Ganondorf]

bonsoir à tous,

J'ai une petite question qui me tracasse ...
Quelle méthode vous parait la plus optimisée pour la gestion de la mémoire

Code :

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

1
2
3
4
5
 
char chaine[50];
 
strncpy(chaine, "un simple exemple", 50);
/* on laisse la mémoire être vidée à la terminaison du programme */

ou bien

Code :

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

1
2
3
4
5
 
char *chaine;
 
chaine = strdup("un simple exemple");
free(chaine);

Ma question vous parait peut être idiote mais dans le cas du 1er exemple on alloue un espace mémoire de 50 caractères, ceci même si le mot qu'on y copie n'en fait pas autant, puis dans l'exemple 2, seul l'espace mémoire dont j'ai besoin va être alloué grâce a strdup.
Donc pour vous, quelle méthode est la plus propre, la plus optimisée en terme de gestion de la mémoire.

Merci à vous pour vos futurs explications

[pfeuh]

Salut,

Ca dépend de l'usage que tu comptes faire de cette chaine. Si c'est une constante, il faut la déclarer en tant que telle. Si c'est le résultat d'une saisie, il faut réserver l'espace pour la plus grande saisie autorisée. Si elle fait partie d'un dictionnaire lu d'un fichier, strdup me parait bien.

A+

Pfeuh

[Médinoc]

Options non-indiquées dans le premier message:

Code C :

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

1
2
3
4
5
/*Constante pure*/
char const * chaine = "un simple exemple";
 
/*Buffer avec pile la bonne taille*/
char chaine[] = "un simple exemple";

[diogene]

Options dans un cadre un peu différent puisqu'il s'agit alors d'initialisations et non d'affectations.

[Médinoc]

Ah, en effet.

D'un autre côté, la première peut être une affectation (mais pas la seconde).

[Ganondorf]

merci pour vos réponses

Dans le cas du 1er exemple, l'espace pré-alloué est limité a 50 caractères, si je n'en copie que 10 avec un strncpy(), l'espace restant il fini ou ? il est vidé automatiquement ? si oui, du coup il n'y a pas vraiment d'avantage a utiliser strdup() , a part s'il y a une taille maxi imposée ? J'me suis dit que je pourrai combiner les 2 en écrivant une fonction strdup qui prendrai également comme argument une taille maximum

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
 
char
*string_copy(char *dest, const char *src, size_t size)
{
	const char *end = dest + size;
	char *copy = dest;
 
	while (*src && dest < end)
		*dest++ = *src++;
 
	*dest = 0;
	return copy;
}
 
 
char
*string_dup(const char *src, size_t maxsize)
{
	char *ret = NULL;
	size_t size = strlen(src);
 
	if (size > maxsize) size = maxsize;
 
	ret = malloc(size + 1);
 
	if (ret) string_copy(ret, src, size);
	else abort();
 
	return ret;
}

Code :

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

1
2
 
chaine = string_dup("un simple exemple", 50);

au final es-ce que ca en vaut la peine d'après vous ? J'essai simplement de gérer la mémoire au mieux, si vous me dites qu'il n'y a pas réellement d'intêret et que les 2 méthodes citées en début de poste sont plus ou moins equivalentes je lacherai l'affaire ...

Merci à vous