Discussion :

[ZeRevo]

Bonjour,

J'ai besoin de découper une chaine avec un séparateur, j'ai essayé avec la commande strtok mais impossible de la faire tourner quand je n'utilise pas de tableau à taille fixe.

Code :

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

1
2
3
 
char *ma_string = "test";
pch = strtok_r(ma_string, sep, saveptr1);

Erreur de segmentation

Code :

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

1
2
3
 
char ma_string[] = "test";
pch = strtok_r(ma_string, sep, saveptr1);

Correct

La méthode strtok est deprecated car elle modifie le premier argument, j'ai alors essayé strtok_r mais j'ai le même résultat.

Il doit bien avoir une possibilité de passer un char * à un char[] facilement, sinon une solution que je voudrais éviter c'est d'utiliser une méthode d'une lib c++ mais je voudrais éviter.

[SofEvans]

Je crois que

Code :

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

1
2
 
char *ma_string = "test";

ne gere pas l' "\0".
C'est peut etre la source de ton erreur car s'il parcours la chaine jusqu'a \0, tu va deborder en memoire.

[orfix]

Code :

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

1
2
 
char *ma_string = "test";

Ton pointeur pointe sur une chaine littérale, souvent placée dans une zone mémoire en lecture seule, d'où l'erreur de segmentation, d'ailleurs une écriture plus correcte serais :

Code :

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

1
2
 
const char *ma_string = "test";

[orfix]

Je crois que

Code :

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

1
2
 
char *ma_string = "test";

ne gere pas l' "\0".
C'est peut etre la source de ton erreur car s'il parcours la chaine jusqu'a \0, tu va deborder en memoire.

Non, la chaîne littérale est stockée comme une suite de caractères suivis d'un '\0'.

[SofEvans]

Okay, je ne savais plus.
C'est bon a savoir ^^

[Médinoc]

La méthode strtok est deprecated car elle modifie le premier argument, j'ai alors essayé strtok_r mais j'ai le même résultat.

strtok_r() modifie aussi le premier argument. strtok() est dépréciée non pas pour cette raison, mais parce qu'elle utilise une variable statique.

Le mieux, c'est d'utiliser malloc() pour allouer un nouveau buffer de la taille de la chaîne, et l'y recopier. Si tu es sous un système POSIX, tu as accès à la fonction strdup() qui fait ça toute seule. Dans les deux cas, n'oublie pas le free() à la fin.

PS: Si ton compilateur est gcc, ajoutes-y l'option -Wwrite-strings. Cela te donnera un warning pour tout char *s="toto", te rappelant que les chaînes littérales sont non-modifiables. Ainsi, seul const char *s="toto" compilera sans warning.

[ZeRevo]

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
/* strtok example */
#include <stdio.h>
#include <string.h>
 
int main ()
{
  const char *str ="- This, a sample string.";
  char * pch;
  printf ("Splitting string \"%s\" into tokens:\n",str);
  pch = strtok (str," ");
  while (pch != NULL)
  {
    printf ("%s\n",pch);
    pch = strtok (NULL, " ");
  }
 
  return 0;
}

main.c: In function ‘main’:
main.c:10: attention : passing argument 1 of ‘strtok’ discards qualifiers from pointer target type
Splitting string "- This, a sample string." into tokens:
Erreur de segmentation (core dumped)

J'ai ajouté le mot clé const mais maintenant la compilation ne fonctionne plus.

Ce qu'il me faudrait c'est créer une chaine de la forme char mavariable[] mais je ne sais pas comment le faire à partir d'une chaine de type char *machaine

Désolé mais je n'ai pas compris comment je peux utiliser strdup, est-ce que vous pouvez donner un exemple de code svp

[Médinoc]

Code C POSIX :

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
/* strtok & strdup example */
#include <stdio.h>
#include <string.h>
 
int main(void)
{
  const char *str = "- This, a sample string.";
  char * tmp;
 
  tmp = strdup(str);
  if (tmp == NULL)
    puts("Memory error.");
  else
  {
    char * pch;
    printf("Splitting string \"%s\" into tokens:\n", tmp);
    pch = strtok(tmp, " ");
    while (pch != NULL)
    {
      printf("%s\n",pch);
      pch = strtok(NULL, " ");
    }
    free(tmp);
  } 
  return 0;
}

Note: Généralement, on met un espace entre mot-clé et parenthèse ou entre fonction et parenthèse, mais jamais les deux à la fois.

[ZeRevo]

Merci bien, toutefois j'ai une erreur à la compilation sur la ligne free(tmp)

main.c:23: attention : incompatible implicit declaration of built-in function ‘free’

[Médinoc]

Code C :

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

#include <stdlib.h>

[ZeRevo]

Arf à force d'utiliser Eclipse, je fais plus attention aux imports ~~

Le code fonctionne mais je ne le comprend pas. Je suis allé cherché le code source du strdup.c http://www.diplom.org/~njudge/doxyge...8c-source.html
mais ça ne me fait pas plus comprendre.

Pourquoi d'un côté quand on fait char *variable = "toto"; on ne peut pas la passer à la méthode strtok, alors que si on utilise strdup ça fonctionne ?

Code :

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

strtok_r() modifie aussi le premier argument. strtok() est dépréciée non pas pour cette raison, mais parce qu'elle utilise une variable statique.

Comment on sait qu'une variable est statique ? je comprend pas la différence entre allouer un espace mémoire (malloc) et faire char *variable = "toto"; ? Quand on fait char *variable = "toto";, ça fait bien appele à malloc ?

J'ai trouvé ça :
http://nicolasj.developpez.com/articles/libc/string/

Code :

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

1
2
3
char s1[11] = "Developpez";
char s2[] = "Developpez";
char *s3 = "Developpez";

La première méthode réserve une zone de 11 bytes et la chaîne "Developpez" y est stockée. La seconde méthode laisse le compilateur calculer la taille appropriée (11 bytes dans notre cas). La dernière méthode ne réserve pas de mémoire, elle se contente de stocker l'adresse d'une chaîne de caractères qui peut se trouver dans un endroit de la mémoire non modifiable par le programme, dans ce cas toute modification de la chaîne s3 se conclura par un comportement indéterminé. il est donc conseillé d'écrire :

Code :

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

1
2
 
const char *s3 = "Developpez";

mais je ne comprend pas bien, pourquoi toute modification de la chaine s3 se conclura par un comportement indéterminé. Comme ça se passe en mémoire quand on crée char *s3 ou const char *s3 ?

[Médinoc]

je comprend pas la différence entre allouer un espace mémoire (malloc) et faire char *variable = "toto"; ? Quand on fait char *variable = "toto";, ça fait bien appele à malloc ?

Ben non, justement.
La chaîne "toto" est stockée dans une zone mémoire en lecture seule, initialisée lors du chargement du programme.
Donc, variable pointe sur de la mémoire en lecture seule. Si on tente d'y écrire, on a une erreur mémoire (segfault sous Linux, Access Violation sous Windows).

Par contre, malloc() alloue dans une zone mémoire accessible en lecture & écriture. On peut donc y recopier la chaîne (ce que fait strdup) et modifier cette copie.

Le tableau dans une fonction alloue directement sur la pile, qui est elle aussi accessible en lecture et écriture.

[jabbounet]

Code C POSIX :

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
/* strtok & strdup example */
#include <stdio.h>
#include <string.h>
 
int main(void)
{
  const char *str = "- This, a sample string.";
  char * tmp;
 
  tmp = strdup(str);
  if (tmp == NULL)
    puts("Memory error.");
  else
  {
    char * pch;
    printf("Splitting string \"%s\" into tokens:\n", tmp);
    pch = strtok(tmp, " ");
    while (pch != NULL)
    {
      printf("%s\n",pch);
      pch = strtok(NULL, " ");
    }
    free(tmp);
  } 
  return 0;
}

Note: Généralement, on met un espace entre mot-clé et parenthèse ou entre fonction et parenthèse, mais jamais les deux à la fois.

Attention le free sur tmp ne marchera pas tres bien car la fonction strtok va faire bouger le pointeur sir je me souviens bien.

il faut conserver un pointeur sur la chaine initiale qui passe pas dans strtok si tu ne veux pas de fuite mémoire.

enfin si je me souviens bien cela fait longtemps que je ne m'en suis pas servi.

[ZeRevo]

merci bien

[Sve@r]

Quand on fait char *variable = "toto";, ça fait bien appele à malloc ?

Non. Ca stocke 't', 'o', 't', 'o', '\0' qq part dans une zone mémoire read-only et ça met l'adresse de cette zone dans "variable".

mais je ne comprend pas bien, pourquoi toute modification de la chaine s3 se conclura par un comportement indéterminé.

Ben justement parce que tu veux modifier une zone non modifiable !!!

Comme ça se passe en mémoire quand on crée char *s3 ou const char *s3 ?

Quelle que soit ta déclaration, la zone pointée par "s3" est constante. Donc la syntaxe "const char* s3" (l'élément pointé par s3 est constant) est la plus appropriée.

Attention le free sur tmp ne marchera pas tres bien car la fonction strtok va faire bouger le pointeur sir je me souviens bien.

Depuis quand une fonction qui reçoit (par recopie évidemment) une simple variable est capable de modifier ladite variable ???
strtok() modifie la zone pointée, pas le pointeur. tmp aura la même valeur avant et après l'appel !!!

[Médinoc]

Depuis quand une fonction qui reçoit (par recopie évidemment) une simple variable est capable de modifier ladite variable ???
strtok() modifie la zone pointée, pas le pointeur. tmp aura la même valeur avant et après l'appel !!!

+1

[jabbounet]

comme je disais cela fait longtemps que je ne l'ai pas utilisé,

mais il faut quand même faire gaffe au premier argument.