Vérifier la bonne exécution de strtok

Bonjour à tous,

Voici j'ai ce code :

Code:

uneVariable = strtok("une chaine", " ");

Je découpe donc pas chaîne de caractère avec un espace " ".
Admettons que le deuxième paramètre change : "-".

strtok ne fera rien. Comment faire un test pour savoir si il découpe ou pas ?
J'ai besoin de le tester car ma chaîne de caractère passé à strtok et le caractère de découpe sont passé en argument du programme.

Donc, l'erreur est possible.

Merci pour vos réponses.
(j'ai regardé le man mais je ne vois rien à part que sa retourne un type char et le NULL pour réutiliser le premier paramètre. De plus, j'ai testé ma variable qui récupère le résultat avec 0 et NULL, rien...)

Le man dit : "évitez au maximum d'utiliser cette fonction".

Pourquoi ne pas coder ta propre fonction strtok ?

Code:

1
2
3
4
5
6
7
8
9
10
 
char * decouper(char ** chaine, char caractere)
{
           for(int i = 0; (*chaine)[i] && (*chaine)[i] != caractere; i++);
           char * resultat = (char *)malloc(sizeof(char)*i);
           strncpy(resultat, *chaine, i-1);
           resultat[i] = 0;
           *chaine += i;
           return resultat;
}

Utilisation :

Code:

1
2
3
4
5
6
7
8
9
 
char maChaine[] = "toto yyy-9";
char *tmp = maChaine;
char * c1 = decouper(&tmp, " ");
char * c2 = decouper(&tmp, "-");
char * c3 = decouper(&tmp, "-");
 
 
free(c1); free(c2); free(c3);

Citation:

Envoyé par Neckara

Le man dit : "évitez au maximum d'utiliser cette fonction".

Parce que c'est mon prof qui m'en a parlé :)

Mon man (je suis sur Mac) dis ceci par contre :
This interface is obsoleted by strsep(3).

Sinon merci pour ton code, je regarderais de plus près ce soir. Mais par quel moyen je peux vérifier que ça fonctionne ? Autrement dis, être sûr qu'il y'a bien découpage dans toutes mes variables ? (actuellement j'en ai 4 qui reçoivent le résultat de strtok en passant une chaîne et ensuite NULL pour qu'il continu avec la même chaine mais modifié).

EDIT : genre renvoyer NULL, comme le fait strsep si elle ne trouve pas serait parfait.

Citation:

Envoyé par Gueuz

Parce que c'est mon prof qui m'en a parlé :)

Même si strtok est conforme à la norme C ainsi, je trouve que c'est une fonction vraiment sale.
Si un jour tu décides d'ajouter un thread à ton code, ton programme va planter à cause de strtok.

Citation:

Envoyé par Gueuz

Mon man (je suis sur Mac) dis ceci par contre :
This interface is obsoleted by strsep(3).

Je ne connaissais pas cette fonction, c'est à peu près ce que je te proposais.
Je te conseillerais de ne pas utiliser cette fonction et de l'écrire toi-même pour avoir un code portable (et puis comme tu es dans une phase d'apprentissage, ça ne te fera pas de mal^^).

Citation:

Envoyé par Gueuz

Mais par quel moyen je peux vérifier que ça fonctionne ? Autrement dis, être sûr qu'il y'a bien découpage dans toutes mes variables ? (actuellement j'en ai 4 qui reçoivent le résultat de strtok en passant une chaîne et ensuite NULL pour qu'il continu avec la même chaine mais modifié).

EDIT : genre renvoyer NULL, comme le fait strsep si elle ne trouve pas serait parfait.

Après je ne t'ai passé que le code de base.
Je suppose que tu préfère ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 char * decouper(char ** chaine, char caractere) { if(*chaine == NULL) return NULL; for(int i = 0; (*chaine)[i] && (*chaine)[i] != caractere; i++); char * resultat = (char *)malloc(sizeof(char)*i); strncpy(resultat, *chaine, i-1); resultat[i] = 0; if( (*chaine)[i] == 0) *chaine = NULL; //indique que le découpage est terminé else *chaine += (i + 1); //inattention de ma part. if(i == 0) { (*chaine)++; //si deux caractère de délimitation se suivent. return NULL; } return resultat; }
/!\ Je l'ai fait à la va-vite, il peut y avoir quelques erreurs.

Merci pour ta réponse complète, précise et pertinente.
Quand je t'ai répondu j'était sur le départ mais ce soir je vais regardé ça de plus près et je te dirais...

Tu remarqueras que j'ai fait une petite erreur dans mon code.

Il faudra donc que tu comprenne ce que le code fait et ce qu'il doit faire pour la trouver.

On va dire que j'ai fait cette erreur exprès dans un but pédagogique :mouarf:
Ok, j'avoue, je ne l'ai pas fait exprès :aie:

Bonjour,

strtok doit effectivement etre evitee dans la mesure du possible. Si la chaine que tu veux decouper est formatee (par exemple si tu lis un fichier), alors les fonctions de type scanf sont tout a fait conseillees, et beaucoup moins dangeureuses.

Un tuto dessus : http://xrenault.developpez.com/tutoriels/c/scanf/

Citation:

Envoyé par Neckara

Tu remarqueras que j'ai fait une petite erreur dans mon code.

Il faudra donc que tu comprenne ce que le code fait et ce qu'il doit faire pour la trouver.

On va dire que j'ai fait cette erreur exprès dans un but pédagogique :mouarf:
Ok, j'avoue, je ne l'ai pas fait exprès :aie:

C'est pas ta boucle for j'espère :)
Bon sinon c'est pas tout à fais ça mais je pense que je vais pas vérifié de toutes façons sinon ça complique de trop (trop de test pour finalement pas grand chose, c'est plutôt les arguments que je dois changer et le tout deviendra plus simple).

Citation:

Envoyé par gangsoleil

Bonjour,

strtok doit effectivement etre evitee dans la mesure du possible. Si la chaine que tu veux decouper est formatee (par exemple si tu lis un fichier), alors les fonctions de type scanf sont tout a fait conseillees, et beaucoup moins dangeureuses.

Un tuto dessus : http://xrenault.developpez.com/tutoriels/c/scanf/

Je découpe des arguments passé au programme, pas de fichier pris en compte mais merci pour le lien.

Citation:

Envoyé par Gueuz

Je découpe des arguments passé au programme, pas de fichier pris en compte mais merci pour le lien.

Tu as la fonction sscanf pour faire ceci.

Sinon l'erreur :
Code:

1 2 3 4 5 if(i == 0) { (*chaine)++; //si deux caractère de délimitation se suivent. return NULL; }
En fait c'est :
Code:

1 2 if(i == 0) return NULL;