probléme de manipulation de chaine

29 septembre 2007 à 21:25:21

Bonjour,
bon voila j'ai besoin d'une fonction parser (de mise en forme pour retirer des balise) dans mon programme j'ai donc créer cette fonction à par :

main.cpp

#include <iostream>
using namespace std;
char *parser(char *buffer,const char *balise, int decalage);
char *parser(char *buffer,const char *balise, int decalage)
{
//cout<<buffer;
char *balise_debut, *balise_fin;
balise_debut = new char[sizeof(balise)+2]; //+2 = '<' et '>'
balise_fin = new char[sizeof(balise)+3]; //+3 = '<', '/' et '>'
sprintf(balise_debut, "<%s>", balise); //on créer la balise de début
sprintf(balise_fin, "</%s>", balise); //on créer la balise de fin
//printf("balise debut : %s\nbalise fin : %s",balise_debut, balise_fin);
char *chaine_debut, *chaine_fin, *chaine;
chaine_debut = (strstr((buffer + decalage), balise_debut) + sizeof(balise_debut)); // pointeur vers le debut de la chaine
chaine_fin = strstr((buffer + decalage), balise_fin); //pointeur vers la fin de la chaine
chaine = new char[(chaine_fin - chaine_debut)]; // on alloue la taille de la chaine
strncpy(chaine, chaine_debut,(chaine_fin - chaine_debut)); // et on insere la chaine dans la mémoire alloué
decalage =(chaine_fin + sizeof(balise_fin)) - buffer; // autre...
printf("%s", chaine); // test
}
int main (int argc, char* argv[])
{
char *chaine;
chaine = parser("<app>bonjour je ne veut pas de toi </app>\n"
"<text>en effet tu est très moche</text>\n"
"<app>au eh puis si en fait</app>\n","app",0);
return 0;
}

Le probleme c'est qu'elle ne retourne pas ce qu'il faut :

>bonjour je ne veut pas de toi

et si on remplace app par text dans l'appelle a la fonction elle renvoie ceci :

t>en effet tu est très moche

minirop

29 septembre 2007 à 21:57:07

tu te compliques la vie pour rien, en C++ autant utiliser std::string puis string::find + string::substr

Bull_

30 septembre 2007 à 11:35:37

Et pourquoi sa marche pas

Bull_

1 octobre 2007 à 20:36:30

Up , toujours besoin d'aide merci

Anonyme

1 octobre 2007 à 21:29:39

Recherche du côté de pointeur et tableau, problème déjà abordé

Bull_

2 octobre 2007 à 21:11:00

Pardon

mais j'ai chercher sur le forum et je trouve pas tu peux être un peux plus précis sur ce que je doit chercher , s'il te plait. Merci

Anonyme

2 octobre 2007 à 21:20:15

C'est demandé si gentillement que je ne peux qu'accepter

.
http://www.siteduzero.com/forum-83-128177-p1-cha-ne-de-caract-res.html

PS : il y a eut d'autre topic plus ressent mais je ne les trouve plus...

Bull_

3 octobre 2007 à 21:20:08

J'ai lu le sujet que tu ma donné

Mais je comprend toujours pas, je ne crois pas entrer dans un cas ou m'est variable ne sont ps initialisé, enfin je veux dire j'alloue la mémoire pour chaque chaine qui va être utilisé.
Et pour ce qui est de chaine_debut et chaine_fin strstr renvoie un char* donc c'est bon .... :euh:

non ?

Anonyme

3 octobre 2007 à 23:24:36

Citation : Bull_

Et pour ce qui est de chaine_debut et chaine_fin strstr renvoie un char* donc c'est bon .... non ?

Justement pas.

Si qqn peut expliquer car j'ai pas mal de peine aussi la dedans (vive les string!).

MatteX

4 octobre 2007 à 6:01:15

Ce qu'on essai de te dire c'est qu'en C++ on a des moyens beaucoup plus puissants de manipuler les chaines que de simple tableau de char et des fonction C comme sprintf, strstr, strcmp, strcpy, etc.

Bienvenue dans le C++, bienvenue dans la STL, bienvenue dans la POO : http://www.cppreference.com/cppstring/index.html

Le lien présente les méthodes de la classe string, à toi d'apprendre

Bull_

4 octobre 2007 à 18:53:56

mais je ne doute pas de l'éfficacité des string en c++ je voulait juste comprendre ce qui n'allait pas dans mon programme ...

PS si sa interesse quelqu'un la solution en string :

#include <iostream>
#include <string>
using namespace std;
string parser(string buffer,string balise);
string parser(string buffer,string balise)
{
string balise_debut, balise_fin;
balise_debut = "<"+balise+">";
balise_fin = "</"+balise+">";
size_t chaine_debut, chaine_fin;
chaine_debut = buffer.find(balise_debut) + balise_debut.size();
chaine_fin = buffer.find(balise_fin);
string chaine;
chaine.assign(buffer, chaine_debut, (chaine_fin - chaine_debut));
return chaine;
}
int main (int argc, char* argv[])
{
string chaine;
chaine = parser("<app>bonjour je ne veut pas de toi </app>\n"
"<text>en effet tu est tres moche</text>\n"
"<app>au eh puis si en fait</app>\n","text");
cout<<chaine;
return 0;
}

minirop

4 octobre 2007 à 19:01:19

tu peux (avantageusement ?) remplacer "assign" par "substr".
donc çà remplacerait :

string chaine;
chaine.assign(buffer, chaine_debut, (chaine_fin - chaine_debut));
return chaine;

par

return buffer.substr(chaine_debut, (chaine_fin - chaine_debut));

Bull_

4 octobre 2007 à 21:42:53

decidément ils aiment bien les hors sujet sur ce site ...
merci quand même pour l'astuce mais on peut discuter d'autre chose jusque demain si on veut et ce dans n'importe quelle sujet donc si ce n'est pas pour répondre à ma question à savoir :

Pourquoi l'utilisation d'un char* déclarer précedament comme je l'ai fait provoque t'il un comportement indéfini ?

Alors mieux vaut allez posté dans discution géneral ....

minirop

4 octobre 2007 à 21:48:27

probablement une erreur avec la variable "decalage" (un de plus ou de moins çà fait toute la différence)

Anonyme

4 octobre 2007 à 21:50:25

Je suis pas sûr mais je tente : avec la déclaration d'un pointeur comme ça :

char*p;

Une seule "case" de mémoire est allouée.
Si tu fais :

char*t = f("ABC"); // f retourne un char* qui est un tableau

C'est bon car les cases ont été allouées.
Mais si tu fais :

char*ft;
ft = f("ABC");

Les cases ne sont pas allouées comme le pointeur n'est pas initialisé comme un tableau.

Je décline de toute responsabilité en cas de faute de ma part.

MatteX

5 octobre 2007 à 21:55:44

Citation : Bull_

decidément ils aiment bien les hors sujet sur ce site ...
merci quand même pour l'astuce mais on peut discuter d'autre chose jusque demain si on veut et ce dans n'importe quelle sujet donc si ce n'est pas pour répondre à ma question à savoir :
Pourquoi l'utilisation d'un char* déclarer précedament comme je l'ai fait provoque t'il un comportement indéfini ?

Alors mieux vaut allez posté dans discution géneral ....

Pas dans discussions générales mais bien sur le forum C... il seront bien plus en mesure de t'aider. Perso, je n'ai jamais utiliser sprintf() et strstr(), c'est archaïque...