Discussion :

[coyotemk]

Bonjour, j'ai essayer de copier tous les argv qui sont passe comme arguments de mon executable, dans un char*
voila la code

Code :

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

1
2
3
4
5
6
 
  char *s =  malloc( 3000*sizeof(char));
  for (int i  = 0 ;i<argc; i++)
    strcat(s,argv[i]);
  printf("%s\n",s);
  free(s);

mais ca marche pas, qqun peut m'aide???
merci

[kevinou55]

chez moi ça marche parfaitement,
peux tu préciser ton problème ??

[coyotemk]

oops dsl,
ca marche tres bien sur ma machine aussi
avan quand j'etais en train de l'executer il m'afichier "segmentation fault".
merci comme meme et dsl pour le derengment
bon journe

[diogene]

oops dsl,
ca marche tres bien sur ma machine aussi
avan quand j'etais en train de l'executer il m'afichier "segmentation fault".
merci comme meme et dsl pour le derengment
bon journe

Il plantait parce qu'il y a effectivement une erreur ! Il ne plante plus parce que tu n'as pas de chance et que tu crois aux miracles

Code :

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

strcat(s,argv[i]);

suppose que s et argv[i] sont des chaînes de caractères donc terminées par 0. C'est vrai pour argv[i] , mais pas sûr du tout pour s au début. Si tu n'as pas de chance, la première case du tableau est 0, et ça semble marcher, sinon le résultat est faux ou ça plante effectivement.
Or, malloc n'initialise pas le tableau créé et tu dois explicitement le remplir avec une chaîne vide avant d'utiliser strcat.

Code :

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

1
2
3
4
5
6
7
8
  char *s =  malloc(3000); // sizeof(char) vaut toujours 1
  if(s!= NULL) // toujours tester le retour de malloc
  {
     s[0] = '\0'; // met une chaîne vide dans s
     for (int i  = 0 ;i<argc; i++) strcat(s,argv[i]);
     printf("%s\n",s);
     free(s);
   }