Concatenation de 2 chaines de caractere

Bonjour tous le monde ;
je veut faire un programme qui fait la Concatenation de 2 chaines de caractere
mais sans l'utilisation de la fonction strcat :mrgreen: mais en utilisant les pointeurs :ccool:

voila le code sue j'ai fait ::

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include<stdio.h> #include<conio.h>     int main() { char chaine1[100] , chaine2[50] ,taille1 , taille2 , *p1 , *p2; printf("Entrez successivement les deux chaines : "); gets(chaine1); gets(chaine2); taille1=strlen(chaine1); taille2=strlen(chaine2); printf("la taille 1 = %d et 2 = %d",taille1 , taille2); for(p1=chaine1+taille1,p2=chaine2 ;p2<chaine2+taille2 ;p1++ , p2++) { *p1=*p2; } printf("la chaine concatenée : %s",chaine1); getch(); }

---

: la concatenation marche mais il s'ajoute quelques caractere incomprehensible a la fin de la liste concatenée
et je veux regler ce probleme et merci

La chaine1 n'a peut-être pas suffisamment de place pour recevoir les caractères de chaine2 ?

Citation:

---

Envoyé par fred1599

La chaine1 n'a peut-être pas suffisamment de place pour recevoir les caractères de chaine2 ?

---

fred1599 everywhere:mouarf: si si la chaine resultante a bcp plus d'espace car la chaine concatenée s'affiche et avec des caractere qui s'ajoutent a la fin

Je ne pense pas non plus, si tu veux utiliser les pointeurs, que les longueurs soit utiles pour faire ce que tu veux.

Bonjour,

Tu oublies qu'une chaine de caractères en C se termine par '\0' ;)
Si par exemple on a tapé :

Code:

---

1 2	"Hello " // pour chaine1 "World" // pour chaine2

---

Tu auras dans tes tableaux les caractères :

• chaine1 : {'H', 'e', 'l', 'l', 'o', ' ', '\0', ...}
• chaine2 : {'W', 'o', 'r', 'l', 'd', '\0', ...}

les ... désignant les autres caractères (complètement aléatoires) contenus dans la mémoire qui reste dans les tableaux.

Avec ton code tu ne vas copier que {'W', 'o', 'r', 'l', 'd'} à la suite de chaine1 et vas donc te retrouver avec :

Code:

---

{'H', 'e', 'l', 'l', 'o', ' ', 'W', 'o', 'r', 'l', 'd', ...}

---

Ainsi lors de l'affichage avec

Code:

---

printf("la chaine concatenée : %s",chaine1);

---

printf va continuer dans nos ... jusqu'à rencontrer une caractère '\0' final, qui peut être n'importe où, même après les 100 caractères que peut contenir chaine1, ce qui a de bonnes chances de faire planter ton programme.

:arrow: Ainsi donc rajoute le '\0' à la fin ;)
Tu peux par exemple le faire en rajoutant le = ici :

Code:

---

for(p1=chaine1+taille1,p2=chaine2 ;p2<=chaine2+taille2 ;p1++ , p2++)

---

Remarques :

• Plutôt que gets, utilise fgets à la place, pour une saisie sécurisée qui t'empêchera de saisir trop de caractères par rapport à la taille de tes tableaux.
• vérifie que taille1+taille2 < 100, tu ne voudrais pas écrire au delà de ton tableau ;)
• évite l'utilisation de getch(); et <conio.h> non portable : préfère getchar() dans <stdio.h>
• Le prototype correct de main en C est :
  
  Code:

  ---

  int main(void)

  ---

  et un return qui va avec, c'est mieux ;)

Tout pareil que Winjerome, je ne vois pas à quel moment tu ajoutes le zéro de fin de chaîne.

Hello,

Aussi, j''utiliserais une boucle while au lieu de for :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

int main(){ char chaine1[100], chaine2[50], *p1, *p2; printf("Entrez successivement les deux chaines : "); gets(chaine1); gets(chaine2);   p1 = &chaine1[strlen(chaine1)]; p2 = chaine2; while(*p2 != '\0'){ *p1++ = *p2++; } *p1 = '\0'; printf("la chaine concatenée : %s",chaine1); getch(); }

---