strchr qui ne fonctionne pas!

**BFH** · 29/12/2005, 11h53

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 :

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

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.

**Skyounet** · 29/12/2005, 12h35

Code :

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

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 :

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

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

Pareil

Code :

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

1
2
 
	strncpy(balise,ligne+alpha, 15);

Comprend pas pourquoi tu fais ca

Regarde peut etre du cote de strtok

**D[r]eadLock** · 29/12/2005, 21h29

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 :

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

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.

**Skyounet** · 29/12/2005, 23h24

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

**D[r]eadLock** · 30/12/2005, 08h45

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.

strchr qui ne fonctionne pas!

C

Discussions similaires

Partager

Partager