Discussion :

[Mr_Chut]

bonsoir

j ai fait un code assez simple qui a pour but de recupere plusieurs informations dans une chaine de caracteres.

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
char *get_info(char *info) {
     char *info_tmp;
     char *tmp;
 
     info_tmp = info;
 
     if (info_tmp == NULL) {
         return NULL;
     }
 
     tmp = strtok(info_tmp, " ");    
     printf("info : %s\n", tmp);
     if (tmp == NULL && strcmp(tmp, r) != 0) {
         printf("info invalid\n");
         return NULL;
     }
 
     return NULL;
}

Je ne sais pas d ou le probleme peut venir.

Merci.

[gnto]

le pointeur info contient bien un caractère de fin de chaine ?

strtok est une fonction très dangereuse

É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.
La fonction strtok() utilise un buffer statique et n'est donc pas sûre dans un contexte multithread. Dans ce cas il vaut mieux utiliser strtok_r().

man strtok

[Mr_Chut]

oui il contient un caharactere de fin de chaine.

par exemple :

Code :

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

info = "blabla toto titi tata";

Il y a bien un caractere de fin.


j ai meme fais un exemple encore plus simple :

Code :

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

1
2
3
4
int main() {
     printf("%s\n", strtok("ee jj\0", " \0"));
     return 1;
}

ca fait toujours segmentation fault !!!

PS : j ai mis '\0' pour vraiment etre sur ....

[gnto]

Code :

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

info = "blabla toto titi tata";

ca, ca marche pas meme dans les reves les plus fous

je pense que tu as surement voulu dire

Code :

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

char *info = "blabla toto titi tata";

ou alors

Code :

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

1
2
char info[20];
sprintf(info,"blabla toto titi tata");

[Mr_Chut]

oui j ai voulu marquer char *info = ...... of course
J avais suppose que info avait deja ete declaree.

Mais la n est pas le probleme

[fearyourself]

strtok modifie la chaîne passée en paramètre... En écrivant :

Code :

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

1
2
 
char *s = "blabla";

ou

Code :

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

f("blabla");

Tu passeras une chaîne qui peut se retrouver en zone lecture seule (d'où le problème).

Il faudrait faire ceci :

Code :

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

1
2
char info[20];
strcpy(info,"blabla");

ou

Code :

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

1
2
 
char s[] = "blabla";

Jc

PS: strtok est une fonction à éviter si possible...

[Pouic]

Cette fonction ne doit pas être invoquée sur une chaîne constante.

Et tu fais

Code :

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

 printf("%s\n", strtok("ee jj\0", " \0"));

Cela ne marche évidemment pas...

[Mr_Chut]

ok merci pour votre aide.

Vous me conseillez vivement donc de ne pas l utiliser et plutot d utilser d autres fonction pour arriver au meme resultat.

[Franck.H]

Juste une petite remarquer sur ta fonction !

J'ai pû remarquer qu'en cas d'erreur, ta fonction retourne une valeur NULL mais tu retourne aussi NULL explicitement en fin normale de la fonction ... le mieux serais de retourner au moins une chaine étant donné que c'est ce qu'elle doit retourner d'après son entête