Discussion :

[Gueuz]

Bonjour à tous,

Voici j'ai ce code :

Code :

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

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...)

[Neckara]

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

Pourquoi ne pas coder ta propre fonction strtok ?

Code :

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

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 :

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

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);

[Gueuz]

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.

[Neckara]

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.

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^^).

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 :

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
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.

[Gueuz]

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...

[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
Ok, j'avoue, je ne l'ai pas fait exprès