Discussion :

[Christophe]

Bonjour,

Je cherche à créer une fonction split.

Voici mon code actuel :

Code :

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

1
2
3
4
5
6
temp=strtok(chaine,":");
while (temp!=NULL)
{
 coupure=strdup(temp);
 temp=strtok(NULL,":");
}

Je souhaite ensuite stocker les adresses de coupure dans un tableau comme ceci :

[tableau 0]=1ère adresse coupure
[tableau 1]=2éme adresse coupure
etc ..
la dernière entrée de tableau devant être NULL.

c'est là que je bloque.

Mes essais n'ont pas été fructueux.

Je dois pas être loin de la solution.

Merci pour votre aide.

[BufferBob]

salut,

la fonction strtok() renvoit NULL s'il n'y a plus d'occurence du délimiteur ou qu'elle rencontre le terminateur de chaine, ça a tendance à nous arranger ici
d'autre part on va troquer la boucle while(){...} pour une boucle do {...} while().

on aurait alors quelque chose du genre :

Code :

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

1
2
3
4
do {
   p = strtok(chaine, ":");
   tableau[i++] = p;
} while (p != NULL)

[Sve@r]

Bonjour

Attention à strtok() car elle utilise un pointeur statique pour mémoriser ses étapes (pour pouvoir repartir du même point lors de l'appel suivant). Ca marche jusqu'au moment où on veut l'appeler dans des boucles imbriquées comme par exemple exploiter un PATH avec ses ":" puis exploiter chaque chemin trouvé avec ses dossiers séparé par "/". Et là le truc s'envoie joyeusement en l'air.

Il vaut mieux lui privilégier strtok_r() à laquelle on passe en 3° paramètre l'adresse d'un pointeur dédié à la sauvegarde de l'état en cours. On peut alors, dans le cas d'appels imbriqués, utiliser différents pointeurs de travail...

[Christophe]

Merci pour vos réponses.
@sven : dans mon cas pas de prob. strtok ou strtok_r répond à mon besoin.

Je n'ai pas de soucis avec strtok, je récupère bien toutes les occurrences dans temp et recopiés via dtrdup dans mon pointeur nommé coupure.

@bufferbob : je vais tester ton bout de code, à vu d'oeil, ça me parait bon.

je vous tiens au courant.

[Sve@r]

Je n'ai pas de soucis avec strtok

Oui parce que là tu ne l'appeles qu'une seule fois et ta fonction traite toute la boucle strtok() avant de sortir.

Appelles-là dans cette configuration

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
chemin=strtok(path, ":")
while (chemin != NULL)
{
    dossier=strtok(chemin, "/")
    while (dossier != NULL)
    {
        ...
       dossier=strtok(NULL, "/")
    }
    chemin=strtok(NULL, ":")
}

Et tu verras comment ton truc partira dans le décors...

...et recopiés via strdup dans mon pointeur nommé coupure.

Pourquoi dupliquer les chaines ? La solution de bufferBob est plus sympa (il stocke les pointeurs de la chaine d'origine et non de nouvelles chaines recopiées).
Ceci dit, si vraiment c'est utile (modif ultérieure etc) il te faudra bien penser à les libérer en fin de travail...

[Christophe]

oui Sve@r,

ce que je voulais dire, c'est que je ne suis pas dans cette config, je peux donc utiliser soit l'un soit l'autre. Mais sur le principe tu as raison, il suffit que je change mon code on faisant des imbrications et je devrais remplacer strtok par strtok_r.