Discussion :

[pacyang]

Bonjour a tous.
J'ai déjà pas mal utilisé les fonctions d'allocations, malloc, calloc et free. Par contre j'ai jamais utilisé realloc mais logiquement on doit pouvoir s'en sortir avec un free et un malloc non ?

En tout cas, j'ai un probleme dans mon code, je comprends vraiment pas, je marque deux fois la meme chose, juste que la deuxieme fois je passe par une fonction, et là j'ai un gros soucis.

Le code C est en piece jointe.

Le resultat a la console donne :

stralloc: "un premier texte"
stralloc: "un texte modifie manuellement"
stralloc: "un premier texte"
realloc: ""

Merci de votre aide

Edit de gruik : je copie colle car le code est pas long

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
#include<stdio.h>
#include<stdlib.h>
 
char * stralloc (const char * str)
{
	int i,n_char;
	char * s;
	for(n_char=0;str[n_char] != '\0';n_char++);
	s= malloc((n_char+1)*sizeof(char));
	for(i=0;i<n_char;i++)
		s[i] = str[i];
	s[n_char]='\0';
	return s;
}
 
void strrealloc (char * str_dest, const char * str_src)
{
	if(str_dest != NULL)
	{
		free(str_dest);
		str_dest = NULL;
	}	
	str_dest=stralloc(str_src);	
}
 
int main(void)
{
	char * str = NULL;
 
	str = stralloc ("un premier texte");
	printf("stralloc: \"%s\"\n",str);
 
	if(str != NULL)
	{
		free(str);
		str=NULL;
	}
	str=stralloc("un texte modifie manuellement");
	printf("stralloc: \"%s\"\n",str);
 
	free(str);
	str = NULL;
 
	str = stralloc ("un premier texte");
	printf("stralloc: \"%s\"\n",str);
	strrealloc(str,"texte modifier avec realloc");
	printf("realloc: \"%s\"\n",str);
 
 
	return EXIT_SUCCESS;
}

[Gruik]

Salut,

C'est simplement qu'après ton strrealloc, le pointeur str lui meme n'est pas modifié, il pointe toujours sur l'ancienne zone où yavait la chaine, zone qui n'est plus valide après le free.

Sinon, ton stralloc() peut être remplacé par la fonction POSIX strdup(). Si tu veux rester en C ANSI, c'est aussi implementable de façon plus optimale avec un strlen+malloc+memcpy

[souviron34]

d'autant plus que

• primo c'est plus rapide
• secondo, le code ci-dessus ne teste pas pour les allocations échouées..

[pacyang]

Merci a vous !
C'est etrange mais en remplaçant mon ancienne fonction stralloc par strdup() dans mon strrealloc, ca fonctionne :
[EDIT : non autant pour moi ça fonctionne toujours pas]

En revanche j'ai une erreur :

text.c:20: attention : incompatible implicit declaration of built-in function ‘strdup’

juste pour ca :

Code :

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

1
2
3
 
    char * str = NULL;    
    str = strdup ("un premier texte");

C'est bizarre, pourtant au niveau des types il me semble pas y avoir de soucis :
char *strdup (const char *s);

PS: sinon pour mon strrealloc, comment je pourrais resoudre mon probleme ? L'idee c'est de pouvoir modifier un texte a l'appel de cette fonction quelque soit sa taille (taille raisonnable tout de meme).


EDIT :

Le moyen que j'ai trouve est de faire ca :

Code :

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

1
2
3
4
5
6
7
8
9
char *  strrealloc (char * str_dest, const char * str_src)
{
	if(str_dest != NULL)
	{
		free(str_dest);
		str_dest = NULL;
	}	
	return str_strdup(str_src);	
}

C'est pas très propre.
Je devrai peut-etre utiliser realloc finalement non ?
Ou bien je fais un pointeur de pointeur et je passe l'adresse de mon char* à strrealloc.

Dans ce cas je pourrais faire :

Code :

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

1
2
3
4
5
6
7
8
9
void  strrealloc (char ** str_dest, const char * str_src)
{
	if(*str_dest != NULL)
	{
		free(*str_dest);
		*str_dest = NULL;
	}	
	*str_dest = str_strdup(str_src);	
}

Mais je crois que c'est encore moins propre
Vos avis m'interessent, merci

[souviron34]

déjà ça :

Code :

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

1
2
3
4
5
6
 
	if(str_dest != NULL)
	{
		free(str_dest);
		str_dest = NULL;
	}

ne peut pas marcher, car str_dest n'est pas modifiable (donc str_dest=NULL est faux).

[pacyang]

Mmh oui en effet, merci de la remarque. Grace a toi, j'ai trouvé un probleme dans mon projet Je vais revoir toute mes fonctions de free. (auquel je passais directement une structure au lieu de passer par les pointeurs).

J'aurai une question a ce propos : Existe t-il un moyen pour verifier que le free a bien fait son travail ?

Finalement, je vais rester sur ma dernière fonction :

Code :

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

1
2
3
4
5
6
7
8
9
void  strrealloc (char ** str_dest, const char * str_src)
{
	if(*str_dest != NULL)
	{
		free(*str_dest);
		*str_dest = NULL;
	}	
	*str_dest = str_strdup(str_src);	
}

Je crois qu'elle fonctionne.