Discussion :

[remi77]

Bonjour a tous

Dans le code ci-dessous

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
 
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
 
main()
{
	char *add="i";
	char *chaine="rem";
	char *copychaine=malloc(10);
 
	if(copychaine==NULL)
	{
		fprintf(stderr,"\nErreur memeoire");
		exit(EXIT_FAILURE);
	}
 
	strcpy(copychaine,chaine);
 
	printf("\n%s",copychaine);
	strcat(copychaine,add);
 
	return 0;
 
 
}

Je souhaite concatener deux chaines de caracteres dont la chaine de destination a préalablement ete alloue avec malloc. Et cela pose quelques petits soucis, puisque la chaine n'est plus assez longue.

Merci pour votre aide

A bientôt

Rémi

[homeostasie]

e souhaite concatener deux chaines de caracteres dont la chaine de destination a préalablement ete alloue avec malloc. Et cela pose quelques petits soucis, puisque la chaine n'est plus assez longue.

Merci pour votre aide

A bientôt

Rémi

Elle est assez longue dans ton cas.
Disons que tu devrais afficher ton résultat avec un printf après l'appel de ta fonction strcat pour oberver le résultat!

[zooro]

Elle est assez longue dans ton cas.
Disons que tu devrais afficher ton résultat avec un printf après l'appel de ta fonction strcat pour oberver le résultat!

Et dans la mesure du possible, essaie d'utiliser strncpy et strncat de préférence à strcpy et strcat. Ca réduit justement les risques de débordement en cas de chaînes pas assez longues.

[Emmanuel Delahaye]

Et dans la mesure du possible, essaie d'utiliser strncpy et strncat de préférence à strcpy et strcat. Ca réduit justement les risques de débordement en cas de chaînes pas assez longues.

Mauvaise idée. strncpy(), par exemple, ne fait pas ce que tu crois. Pour commencer, si la chaine est plus courte, elle complète de 0 ce qui est une perte de temps. Ensuite, si la chaine est trop longue, elle ne met pas de 0 du tout, ce qui est une catastrophe.

De plus, ici, ce n'est absolument pas justifié, puisque la taille est mesurable a coup de strlen(). Il n'ya donc aucun risque de débordement si on écrit le code correctement.

[remi77]

Voila je reposte le code

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
 
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
 
main()
{
	char *add="i";
	char *chaine="rem";
	char *copychaine=malloc(10);
 
	if(copychaine==NULL)
	{
		fprintf(stderr,"\nErreur memeoire");
		exit(EXIT_FAILURE);
	}
 
	strcpy(copychaine,chaine);
 
	strcat(copychaine,add);
	printf("\n%s",copychaine);
 
	return 0;

Ce fut une petite erreur de ma part, mais le fond de ma question reste toutefois la meme, est il nécessaire de reallouer de la memoire supplémentaire avant une concatenation, si cette derniere a préalablement été alloué avec malloc?

Merci à tous

Rémi

[homeostasie]

est il nécessaire de reallouer de la memoire supplémentaire avant une concatenation, si cette derniere a préalablement été alloué avec malloc?

Non, sauf si tu sais que tu vas dépasser la taille que tu as initiallement alloué.
-> soit comme dit dans le post de zooro, tu utilises strncpy et strncat pour éviter les débordements mais il te manquera des caractères
-> soit tu réalloues un espace mémoire plus grand avec realloc en prenant les précautions nécessaires vis à vis de l'utiisation de cette fonction.

[Trap D]

Le mieux est encore de faire un malloc(strlen(add)+strlen(chaine)+1) pour le zéro terminal, et de ne pas oublier de libérer la mémoire à la fin du programme : veuillez laisser ce programme aussi propre que vous auriez souhaité le trouver en entrant

[Jean-Marc.Bourguet]

le fond de ma question reste toutefois la meme, est il nécessaire de reallouer de la memoire supplémentaire avant une concatenation, si cette derniere a préalablement été alloué avec malloc?

Oui. strcat ne va pas détecter que c'est de la mémoire allouée avec malloc et que la zone allouée ne suffit pas, allouer une zone assez grande, faire la copie puis la concaténation, libérer la zone originelle et changer toutes les autres références à cette zone par une référence à la mémoire nouvellement allouée.

[Emmanuel Delahaye]

<...> est il nécessaire de reallouer de la memoire supplémentaire avant une concatenation, si cette derniere a préalablement été alloué avec malloc?

Si la taille est insuffisante, oui. Encore faut-il penser à gérer la taille correctement, ce qui n'est pas le cas ici. Pourquoi ne pas allouer tout de suite la bonne taille ?

Là, tu mets 10. Pourquoi ? C'est parce que tu as 10 doigts ? Essaye 13, c'est un vrai nombre magique...

[zooro]

Là, tu mets 10. Pourquoi ? C'est parce que tu as 10 doigts ? Essaye 13, c'est un vrai nombre magique...

non, mets 42 ! ça c'est un vrai nombre magique !

[Emmanuel Delahaye]

non, mets 42 ! ça c'est un vrai nombre magique !