strchr qui ne fonctionne pas!

Version imprimable

Bonjour, j'ai une fonction qui doit memoriser des balises Html, c'est a dire des lettres contenues entre < et >
ou entre < et ' ' (l'espace c'est au cas où il y aurait des proprietés a la balise). Enfin bref voila ma fonction:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char *Memoriser_Balise(char *ligne, int alpha) { char *balise=malloc(15); char *finnom; finnom =strchr(&ligne[alpha],' '); if (!finnom) finnom = strchr(&ligne[alpha],'>'); *finnom='\0'; strncpy(balise,ligne+alpha, 15); *finnom='>'; return(balise); }
Or si j'ai par exemple <HTML><Body> la balise memorisée sera : HTML><Body
Pareil si j'ai <i>espaceBlabla , la balise memorisée sera : i>

Quelqu'un peut il m'expliquer pourquoi?

Surtout que si j'ai <i>retour chariot la balise memorisée sera : i
Ce qui est correct.

29/12/2005, 12h35
Skyounet

Re: strchr qui ne fonctionne pas!
Code:

1 2 finnom =strchr(&ligne[alpha],' ');
Tu passes a strchr l'adresse d'un caractere alors que strchr demande une chaine de caractere (c'est quand meme mieux pour rechercher un caractere dans une chaine)
Code:

1 2 finnom = strchr(&ligne[alpha],'>');
Pareil
Code:

1 2 strncpy(balise,ligne+alpha, 15);
Comprend pas pourquoi tu fais ca

Regarde peut etre du cote de strtok

Differentes remarques:
- ton if (!finnom) cherche le > si tu n'as pas trouve d'espace. Donc si t'as un espace vers la fin, tu ne trouveras pas ton > qui est juste a cote.
- tu mets > a la place de ton char trouve or si tu trouve un espace tu mets un > en trop

Sinon en testant ca marche pour moi:
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 #include <stdio.h> #include <string.h> #include <malloc.h> char *Memoriser_Balise(char *ligne, int alpha) { char *balise=malloc(15); char *finnom; finnom =strchr(&ligne[alpha],' '); if (!finnom) finnom = strchr(&ligne[alpha],'>'); *finnom='\0'; strncpy(balise,ligne+alpha, 15); *finnom='>'; return(balise); } int main(void) { char *astring=strdup("<HTML><Body>"); char *balise=Memoriser_Balise(astring,0); printf("String:%s\nBalise:%s\n",astring,balise); return 0; }
Citation:

Envoyé par ./BFH

String:<HTML><Body>
Balise:<HTML

Remarques:
- [Skyrunner] pour l'appel strchr c'est correct, l'addresse d'un caractere (un char *) est bien une chaine de caractere.
- fais attention d'avoir le droit de modifier ligne, perso je prefere un const char * et je ferais le strncpy avec le nombre de caracteres a copier (genre strncpy(balise, ligne+alpha, finnom-ligne+alpha)
- pour ton probleme, tu donne bien le bon alpha (je pense que tu dois mettre 1 au lieu de 0 pour le <i>, et pour le HTML/Body je sais pas trop, mais regarde de ce cote.

29/12/2005, 23h24
Skyounet

Citation:

Envoyé par D[r

eadLock]

- [Skyrunner] pour l'appel strchr c'est correct, l'addresse d'un caractere (un char *) est bien une chaine de caractere.

Exact je n'ai rien dis :oops: :oops:
30/12/2005, 08h45
D[r]eadLock

En fait pour le <HTML><Body> tout vient de ton algo, et je pense que tu dois avoir un espace apres le Body> donc il s'arrette a l'espace.
Tu devrais chercher dans l'autre sens:
- chercher un > (la fin de la balise)
- si tu as trouver, alors cherche un espace (pour les proprietes a l'interieur de ta balise)
Tu n'as qu'a inverser tes deux tests (' ' vs '>'), inverser ton test (if (finnom1=NULL)), ajouter un cas d'erreur quand meme (si tu ne trouve pas de >) et ca devrait marcher ;)

Rq: pour le +1, je crois que je comprends, c'est pour enlever le <, donc c'est bon.