Discussion :

[merouenx]

bonjour tout le monde j'essaie de faire un programme en c qui permet de fair la concatenation de deux chaine ch1 et ch2; mais j'ai une erreur de segmentation ;
voila le programme:

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
#include<stdio.h>
#include<ncurses.h>
 
char *concat(char *ch1, char *ch2)
{
char *p;
p=ch1;
while(*ch1) ch1++;
 
while(*ch2)
{ 
*ch1=*ch2; ch1++; ch2++;
}
*ch1='\0';
 
return p;
}
 
 main()
{
char *s1,*s2;
 
printf("donner la 1er chne :");scanf("%s",s1);
puts("donner la 2eme chne :");scanf("%s",s2);
puts(concat(s1,s2));
return 0;
}

[zooro]

bonjour tout le monde j'essaie de faire un programme en c qui permet de fair la concatenation de deux chaine ch1 et ch2; mais j'ai une erreur de segmentation ;

Regardes attentivement la deuxième ligne de ton main()...

[diogene]

Code :

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

1
2
3
4
5
6
int main(void)
{
char *s1,*s2;
... 
scanf("%s",s1);
...

Allouer un pointeur n'alloue pas d'espace pour l'objet pointé. s1 et s2 doivent être des tableaux de char pour permettre de stocker les chaînes entrées par scanf

[corentin59]

il faut que :
dans ton main
1) tu alloues deux tableaux de char pour y mettre tes deux chaines
2) tu vérifies que le scanf ne te renvoie pas plus de caractères que ne peuvent en contenir tes tableaux (pour cela, utiliser fgets en lieu et place de scanf)
dans ta fonction
tu alloues un tableau de char qui contiendra les deux chaines concaténées, il doit donc être au moins égal à la somme du nombre de caractères de tes deux chaines (utiliser strlen)