Discussion :

[chfakht]

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 mais en utilisant les pointeurs

voila le code sue j'ai fait ::

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
 
#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

[fred1599]

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

[chfakht]

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

fred1599 everywhere 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

[fred1599]

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

[Invité]

Bonjour,

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

Code :

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

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 :

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

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

Ainsi lors de l'affichage avec

Code :

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

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.

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

Code :

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

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 :

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

  int main(void)

  et un return qui va avec, c'est mieux

[Franck.H]

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

[boboss123]

Hello,

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

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
 
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();
}