Discussion :

[psylox]

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 :

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

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 :

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

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 :

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

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

[Trap D]

Ouh la la, consulte d'urgence un cours sur les pointeurs en C
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).

[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

[psylox]

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.

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.

[juls64]

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

[juls64]

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

[droggo]

Mye,

Code :

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

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 :

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

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

Pour memcpy, même remarque:

Code :

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

memcpy(cheminX,chemin,ici_la_bonne_taille_svp);

Rarement vu tant d'erreurs en si peu de lignes

[juls64]

voici un code qui tourne

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

[psylox]

Rarement vu tant d'erreurs en si peu de lignes

Merci je pensais appelé le livre des records

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

[psylox]

Plus j'avance et plus je recul ..

Code :

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

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 )

[souviron34]

Plus j'avance et plus je recul ..

Code :

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

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 :

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

1
2
3
4
5
6
 
char *chemin=NULL ;
chemin = malloc ( 256);
if ( chemin == NULL )
   return ERROR ;
scanf("%s",chemin);

Ensuite :

Code :

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

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 :

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

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 :

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

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

[psylox]

C'est bon ça marche

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*


Merci à vous tous