Fonction STRTOK ne marche pas

Version imprimable

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:

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.

21/12/2008, 17h26
ram-0000
Tout d'abord, le code que tu fournis n'est pas complet, il a fallut que je commente 3 lignes pour arriver à compiler
Code:

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:

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:

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
21/12/2008, 17h27
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...?

Merci ram-0000 je vais essayer ta solution.
21/12/2008, 17h28
ram-0000

Je l'ai fait sous Visual Studio 2005

Bonsoir,

Citation:

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 :

Citation:

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:

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:

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

21/12/2008, 17h55
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?

Merci
21/12/2008, 17h55
ram-0000

Citation:

Envoyé par mabu

Le problème doit donc venir de son utilisation

Je ne comprends pas dans le man:

Citation:

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." ?
21/12/2008, 17h59
ram-0000

Citation:

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)
21/12/2008, 18h07
Invité(e)
Citation:

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:

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".
21/12/2008, 18h11
ram-0000

Citation:

Envoyé par ram-0000

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

Citation:

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
21/12/2008, 18h26
Invité(e)

Citation:

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... :aie:
21/12/2008, 20h16
grenadinette

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.
21/12/2008, 20h57
ram-0000

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 :-)
21/12/2008, 21h43
grenadinette

Merci ram-000, je crois qu'il vaut mieux que je trouve un autre moyen de découper la phrase.
22/12/2008, 01h08
Emmanuel Delahaye

Citation:

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.
22/12/2008, 09h08
ram-0000

Citation:

Envoyé par Emmanuel Delahaye

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

OK, je comprend mieux maintenant !!
22/12/2008, 13h26
souviron34

sinon c'est pas trop compliqué de se faire son MyStrtok :D