Fonction STRTOK ne marche pas

**grenadinette** · 21/12/2008, 16h53

Bonjour,

J'essaye d'utiliser la fonction strtok mais ça plante ( une fenêtre windows s'affiche pour dire qu'il y ait eu une erreure)

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
19
20
21
22
23
24
25
26
int main ()                     
{                                                                
char *pTemp ;                                                           
char *pGCXItems;                                                        
char *pGCXItem ;
char *pOwner ; 
char *pGCX;                                                        
int iGCXSize , LnTemp ;                                                 
pTemp = NULL;                                                        
pGCXItems = NULL;                                                    
pGCXItem = NULL;                                                     
GCXStruct GCX;
GCXStruct * pGCXValues;
pGCXValues =&GCX;
pOwner = "RIM"; 
char	*separateur = { "+" };  
pGCX = "GCX+43+RIM:ADAM:RIMA:ADAMA*RIM1:ADAM1:RIMA1:ADAMA1";                                                                           
pTemp = strtok(pGCX, separateur);   
 
cout << "Le pTemp est  = "
    << pTemp 
     << endl;
system("PAUSE");
return 0;  
 
}

Merci pour votre aide.

**ram-0000** · 21/12/2008, 17h26

Tout d'abord, le code que tu fournis n'est pas complet, il a fallut que je commente 3 lignes pour arriver à compiler

Code :

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

1
2
3
4
 
//GCXStruct GCX;
//GCXStruct * pGCXValues;
//pGCXValues =&GCX;

Sinon, ton problème vient du fait que strtok() modifie la chaine passé en paramètre et que cette chaine et du type non modifiable

Code :

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

1
2
 
pGCX = "GCX+43+RIM:ADAM:RIMA:ADAMA*RIM1:ADAM1:RIMA1:ADAMA1";

Pour que ce que tu veux faire fonctionne, il faut passer cette chaine en modifiable en la recopiant dan un tableau de caractères

Code :

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

1
2
3
4
 
char buf[1024];
strcpy(buf, pGCX);
pTemp = strtok(buf, separateur);

Je l'ai fais à la hache, à toi d'arranger le code

**grenadinette** · 21/12/2008, 17h27

Merci pour ta réponse (la première), j'ai essayé avec ta correction mais ça ne marche pas, j'utilise DEVC++ pour progrmmer , peut être qu'il ne comprend pas la fonction strtok, mais à ce moment là il l'aurait dit pendant la compilation...?

Merci ram-0000 je vais essayer ta solution.

**ram-0000** · 21/12/2008, 17h28

Je l'ai fait sous Visual Studio 2005

Invité(e) · 21/12/2008, 17h50

Bonsoir,

Envoyé par grenadinette

Merci pour ta réponse (la première), j'ai essayé avec ta correction mais ça ne marche pas, j'utilise DEVC++ pour progrmmer , peut être qu'il ne comprend pas la fonction strtok, mais à ce moment là il l'aurait dit pendant la compilation...?

strtok fait parti du standard.
Le problème doit donc venir de son utilisation :

Envoyé par http://man.developpez.com/man3/strtok.3.php

Évitez au maximum d'utiliser cette fonction. Sinon, prenez note des informations suivantes :

Cette fonction modifie son premier argument.
Les caractères de séparation sont surchargés, leur identité est donc perdue.
Cette fonction ne doit pas être invoquée sur une chaîne constante.

Sinon, cout c'est du c++, pas du c et system("PAUSE"); n'est pas portable.

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
#include <string.h>
#include <stdio.h>
 
int main(void)
{
    char *pTemp;
    char pGCX[256] = "GCX+43+RIM:ADAM:RIMA:ADAMA*RIM1:ADAM1:RIMA1:ADAMA1";
    char separateur[] = "+";
 
    pTemp = strtok(pGCX, separateur);
 
    printf("Le pTemp est = %s\n", pTemp);
 
    return 0;
}

Code :

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

1
2
$ ./a.out 
Le pTemp est = GCX

**grenadinette** · 21/12/2008, 17h55

Merci ram-0000 ça marche maintenant, mais j'ai une question à poser pour bien comprendre, le tableau Buf doit-il contenir plus de caractères que pGCX, car le contenu de mon pGCX peut atteindre 40000 caractères?

Merci

**ram-0000** · 21/12/2008, 17h55

Envoyé par mabu

Le problème doit donc venir de son utilisation

Je ne comprends pas dans le man:

Envoyé par http://man.developpez.com/man3/strtok.3.php

Évitez au maximum d'utiliser cette fonction. Sinon, prenez note des informations suivantes :

Cette fonction modifie son premier argument.
Les caractères de séparation sont surchargés, leur identité est donc perdue.
Cette fonction ne doit pas être invoquée sur une chaîne constante.

Cela veut dire quoi "Les caractères de séparation sont surchargés, leur identité est donc perdue." ?

**ram-0000** · 21/12/2008, 17h59

Envoyé par grenadinette

Merci ram-0000 ça marche maintenant, mais j'ai une question à poser pour bien comprendre, le tableau Buf doit-il contenir plus de caractères que pGCX, car le contenu de mon pGCX peut atteindre 40000 caractères?

Euh oui, sinon il va y avoir des dégats

Une question, ces 40000 caractères, ils viennent d'où. Ils seront toujours déclarés comme cela (char *pGCS = "40000 caractères...") ou bien ils proviennent par exemple de la lecture d'un fichier ou que sais je encore.

Ma question, c'est pour savoir si ces 40000 caractères sont vraiment constant ou bien s'il sont devenus constants en passant dans une fonction ou pour les besoins de la démo (j'espère être clair mais c'est pas sûr)

Invité(e) · 21/12/2008, 18h07

Envoyé par ram-0000

Cela veut dire quoi "Les caractères de séparation sont surchargés, leur identité est donc perdue." ?

Si je ne dis pas de bêtises, cela veut dire qu'on ne peut pas faire la différence entre deux caractères séparateurs :

Code :

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

1
2
3
    char chaine[256] = "A+B-C";
    char separateur[] = "+-";
    char * pTemp = strtok(chaine, separateur);

Il est impossible de savoir après coup si la chaine analysée était "A+B+C", "A+B-C", "A-B+C" ou "A-B-C".

**ram-0000** · 21/12/2008, 18h11

Envoyé par ram-0000

Cela veut dire quoi "Les caractères de séparation sont surchargés, leur identité est donc perdue." ?

Envoyé par mabu

Si je ne dis pas de bêtises, cela veut dire qu'on ne peut pas faire la différence entre deux caractères séparateurs :

Drôle de manière (un peu incompréhensible?) de le dire

Invité(e) · 21/12/2008, 18h26

Envoyé par ram-0000

Drôle de manière (un peu incompréhensible?) de le dire

En fait, j'ai compris cette phrase le jour où j'ai justement voulu faire la différence entre deux séparateurs...

**grenadinette** · 21/12/2008, 20h16

Re ram-0000,

en fait, je reçois un message sur un pointeur et je dois extraire une partie ( qui peut atteindre 400000 charactères), cette opération est faite à l'aide d'une fonction qui me donne en sortie un pointeur (pGCX) contenant la partie voulue.

**ram-0000** · 21/12/2008, 20h57

Il faut voir une chose, c'est que strtok() va modifier la chaine de caractères que tu lui fournies en entrée.

Donc soit tu as besoin de la conserver intacte pour d'autre traitements ultérieures et alors il faut dupliquer cette chaine,

Soit tu n'en as plus besoin et tu peux te permettre de laisser strtok() modifier cette chaine et donc pas de duplication nécessaire.

Autre chose, je ne sais pas si cela te concerne mais strtok() n'est pas multithread. C'est à dire que si tu commence dans un thread, tout le reste du parsing doit être fait dans le même thread, interdit de sous traiter :-)

**grenadinette** · 21/12/2008, 21h43

Merci ram-000, je crois qu'il vaut mieux que je trouve un autre moyen de découper la phrase.

**Emmanuel Delahaye** · 22/12/2008, 01h08

Envoyé par ram-0000

Je ne comprends pas dans le man:

Cela veut dire quoi "Les caractères de séparation sont surchargés, leur identité est donc perdue." ?

C'est mal rédigé (traduction à la hache, comme d'hab, toujours préférer la V.O...).

Ça signifie que la chaine originelle est modifiée (overwritten ou 'écrite par-dessus'). Les séparateurs sont remplacés par des 0.