concaténation cd chaine saisie au clavier

Bonjour,

Je sais que ce sujet a déja été traité maintes et maintes fois, j'ai donc essayé toutes les solutions que j'ai pu lire sur le net, ou sur ce forum, mais rien toujours ma segmentation fault sur le strcat.

Voici mon code :

Code:

---

1 2 3 4 5 6 7

char *cheminX, *chemin, *nomX;   scanf("%s",&chemin); //pareil pour nomX   memcpy(&cheminX,&chemin,sizeof(char *)); strcat(cheminX,nomX);

---

a l'aide d'un printf j'ai pu constater que le memcpy fonctionne sans probleme

J'ai essayé en faisant un malloc mais dans ce cas la le memcpy de passe plus

Code:

---

cheminX=malloc((strlen(chemin)+strlen(nomX)+1)*sizeof(char*));

---

Peut-etre avec un calloc mais je ne sais pas l'utilisé, je vais me pencher sur la question maintenant, on s'est jamais...

J'ai aussi lu cette solution :

Code:

---

sprintf(cheminX,"%s%s",chemin,nomX);

---

Cette ne fonctionne ni avec un malloc ni sans.

Voila je pense j'ai a peu près tout dis.
Je m'en remets a vos lumières

Ouh la la, consulte d'urgence un cours sur les pointeurs en C 8O
Pratiquement tout ce que tu écris est incorrect en C (la syntaxe est correc te mais le programme plante ou a un comportement imprévisible).

deja dans ton malloc au lieu de faire sizeof(char*) remplace par sizeof(char) qui en fait vaut 1octet donc inutile de le mettre a part pour prevenir une modif future . Car autrement tu demandes la taille d'un pointeur de caractere or ce n'est pas des pointeurs que tu veux stocker c'est des caracteres

Citation:

---

Envoyé par juls64

deja dans ton malloc au lieu de faire sizeof(char*) remplace par sizeof(char) qui en fait vaut 1octet donc inutile de le mettre a part pour prevenir une modif future . Car autrement tu demandes la taille d'un pointeur de caractere or ce n'est pas des pointeurs que tu veux stocker c'est des caracteres

---

Oui j'avais testé mais ça ne marche pas non plus donc je pensais c'était comme ça ... donc je vais remettre vu que apparement c'est le reste qui est faux.

Citation:

---

Envoyé par Trap D

Ouh la la, consulte d'urgence un cours sur les pointeurs en C

---

Bin le truc c'est que j'en ai eu des cours de C en DUT, mais franchement ya un fossé énorme entre ce qu'on t'apprends en cour et ce que tu dois faire après kan tu es au boulot.
:?

meme remarque pour memcpy le dernier parametre devrait etre strlen(src) je trouve bizarre que ca marche avec sizeof(char*) vu que la fonction attend un nombre d'octets à copier

petite remarque pour ton scanf avec &chaine tu veux passer l'adresse mais en fait quand tu declares *chaine chaine represente l'adresse donc pas besoin de mettre un & devant

Mye,

Citation:

---

Envoyé par psylox

Code:

---

1 2 3 4 5 6 7

char *cheminX, *chemin, *nomX;   scanf("%s",&chemin); //pareil pour nomX   memcpy(&cheminX,&chemin,sizeof(char *)); strcat(cheminX,nomX);

---

---

Bon, plus loin tu réserves de la mémoire pour cheminX, mais les autres ?

Le scanf : comme chemin est déjà un pointeur, il faut écrire

Code:

---

scanf("%s",chemin); //pareil pour nomX

---

Pour memcpy, même remarque:

Code:

---

memcpy(cheminX,chemin,ici_la_bonne_taille_svp);

---

Rarement vu tant d'erreurs en si peu de lignes :D

voici un code qui tourne

Code:

---

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

char *test; char *test2; char *concatene;   test =(char*) malloc(10); test2 =(char*) malloc(10); concatene =(char*) malloc(20); scanf("%s",test); scanf("%s",test2);   strcat(test,test2);   printf("test = %s\n",test); memcpy(concatene,test,strlen(test) ); printf("concatene = %s\n",concatene);

---

une petite remarque quand tu declares un pointeur l'adresse qu'il pointe n'est pas valide donc si tu essaies d'ecrire dedans tu as un seg fault donc utiliser un malloc permet de lui affecter un pointeur valide pour une taille donnée si tu t'amuses avec ce code d'ecrire plus de 10 caracteres une chance sur deux que ca plante et la chaine concatene a le meme resultat que test donc vaut eviter d'utiliser des variaboles qui ne servent a rien

Citation:

---

Envoyé par droggo

Rarement vu tant d'erreurs en si peu de lignes :D

---

Merci je pensais appelé le livre des records :mrgreen:

Non j'ai rien dis c'est pathétique.

Plus j'avance et plus je recul ..

Code:

---

1 2 3 4 5 6 7 8

chemin=malloc(256*sizeof(char)); //pareil pour nomX if (chemin==NULL) { printf("Echec allocation mémoire !!!\n"); exit(EXIT_FAILURE); } scanf("%s",&chemin); //le & est obligatoire   cheminX=malloc((strlen(chemin)+strlen(nomX)+1)*sizeof(char)); ... memcpy(cheminX,chemin,strlen(chemin));

---

C'est mieux? pasque chez moi ça marche toujours pas ..

(o faites merci de votre aide :P)

Citation:

---

Envoyé par psylox

Plus j'avance et plus je recul ..

Code:

---

1 2 3 4 5

chemin=malloc(256*sizeof(char)); //pareil pour nomX if (chemin==NULL) { printf("Echec allocation mémoire !!!\n"); exit(EXIT_FAILURE); } scanf("%s",&chemin); //le & est obligatoire

---

---

Déjà c'est faux ici..

Relis tes cours sur les pointeurs. On ne programme pas en faisant n'importe quoi..

que prends la fonction scanf ? un pointeur.
Qu'est-ce que chemin ? un pointeur
Donc comment code-t-on scanf en voulant le résultat dans chemin ?

Code:

---

1 2 3 4 5 6

char *chemin=NULL ; chemin = malloc ( 256); if ( chemin == NULL ) return ERROR ; scanf("%s",chemin);

---

Ensuite :

Citation:

---

Envoyé par psylox

Code:

---

1 2 3

cheminX=malloc((strlen(chemin)+strlen(nomX)+1)*sizeof(char)); ... memcpy(cheminX,chemin,strlen(chemin));

---

---

Si tu veux utiliser memcpy, ça peut marcher.

Mais :
1) il faut tester que l'allocation a marché
2) le mieux serait strcpy/strcat, ou sprintf.

Code:

---

1 2 3 4 5 6 7

cheminX=malloc((strlen(chemin)+strlen(nomX)+1)*sizeof(char)); if ( cheminX == NULL ) return ERROR ;   strcpy ( cheminX, chemin); strcat ( cheminX, nomX);

---

ou

Code:

---

1 2 3 4 5 6

cheminX=malloc((strlen(chemin)+strlen(nomX)+1)*sizeof(char)); if ( cheminX == NULL ) return ERROR ;   sprintf ( cheminX, "%s%s", chemin, nomX);

---

Et enfin si tu veux mettre un séparateur entre les 2, ajouter un 1 à l'allocation...

C'est bon ça marche :lol:

Citation:

---

Envoyé par souviron34

1) il faut tester que l'allocation a marché

---

Je le fais tout le tps meme si je le mettais pas dans les postes pour pas surcharger.

Ok donc je retiens de pas mettre le & dans un scanf avec les char*:P


Merci à vous tous