Discussion :

[Mr_Chut]

Bonjour j essaie de creer une fonction concat qui me concat des chaines de caracteres.
Voila comment je fais :

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
char *concat(int nb, ...) {
     va_list vl;
     int i;
     char *res = "";
     char *tmp;
 
     va_start(vl, nb);
 
     for (i = 0; i < nb; i++) {
          tmp = va_arg(vl, char *);
          res = (char *) realloc(res, sizeof(tmp) + sizeof(res));
          sprintf(res, "%s%s", res, tmp);
     }
     return res;
}

cela me dit

Code :

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

*** glibc detected *** realloc(): invalid pointer: 0x0804860c ***

d ou vient le problem ?

Merci

[Mr_Chut]

c est peut etre pas tres propre, mais j ai creer une variable intermediaire pour ensuite faire un malloc.

[Médinoc]

Pour le pointeur invalide, je dois avouer que je ne vois pas trop. Mais pour le reste, ce n'est pas étonnant que ça plante:
Tu utilises des sizeof sur des pointeurs, il te faut utiliser strlen() pour cela.
Quant au sprintf(), c'est vraiment déconseillé de l'utiliser comme ça. La fonction strcat() est encore préférable...

Code C non-testé :

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
/* On ne doit pas affecter directement 
   le retour de realloc() au pointeur original :
   Sinon, en cas d'échec, on le perd.*/
/* Retourne 0 si OK, -1 si échec
   (comme les fonctions standard/POSIX) */
int MyReallocChar(char **ppC, int nTaille) {
	char *pTmpC = realloc(*ppC, nTaille);
	if(pTmpC != NULL) {
		*ppC = pTmpC;
	}
	return ((pTmpC != NULL) ? 0 : -1);
}
 
char *concat(int nb, ...) {
	va_list vl;
	int i;
	/* ou NULL, mais toujours un pointeur du même type:
	   ici, un pointeur libérable avec free(). */
	char *pRes = strdup(""); 
	char const *pcTmp = NULL;
 
	va_start(vl, nb);
 
	for (i = 0; i < nb; i++) {
		pcTmp = va_arg(vl, char const*);
		if(MyRealloc(&pRes, strlen(pcTmp) + strlen(pRes) + 1)) {
			return pRes; /* N'a pas pu agrandir : stop (ou erreur) */
		}
		strcat(pRes, pcTmp);
	}
	va_end(vl);
	return pRes;
}

Je n'ai pas testé ce code, mais ça devrait marcher...

[Mr_Chut]

le strlen c est une erreur de frappe (j ai recopier la fonction qui etait sur un ordinateur non relie au net :p).

Merci pour le conseil du sprintf.

pour le realloc tu rajoute "+ 1" pour la fin de la chaine ?
si oui, le realloc initialise implicitement le nouvel espace memoire a '\0' ?

[Médinoc]

Oui je rajoute +1 pour le \0, et pour le realloc(), je ne sais plus, je ne sais même plus si c'est spécifié dans la norme.

Mais ce n'est pas grave, puisque le \0 de fin de la chaîne précédente est gardé.

[fearyourself]

si oui, le realloc initialise implicitement le nouvel espace memoire a '\0' ?

Non, voir le man de realloc : man

Le contenu de la zone de mémoire nouvellement allouée n'est pas initialisé

Jc