Discussion :

[Maliono]

Bonjour,
je travaille sous c++ Builder et suis en train de faire un programme qui fait des recherches dans des fichiers xml. Pour cela j'utilise TinyXml.
Je suis arrivé à terme de mon programme après test en débug.
Une fois que j'essaie de le mettre en release mon programme fait une violation d'accès (vers un pointeur non nul) lors de la recherche.

Cela se fait uniquement lorsque j'essaie de lire une valeur dans le fichier XML, (j'ai pu le vérifié à l'aide de MessageBox avant et après l'instruction).
Apres recherche on m'a dit que cela venait d'une variable mal initialisée, mais ça ne vient pas de la (je ne pense pas être passé à coté de l'une d'elle).

Avez-vous des pistes pour pouvoir m'expliquer ce qui ne va pas en release s'il vous plait ?
Je vous joins le code provoquant mon erreur.

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
TiXmlDocument monFichier((path + fileName).c_str());
	if(monFichier.LoadFile()){
		TiXmlHandle monHandle(&monFichier);
		monHandle = monHandle.FirstChildElement();

		if (monHandle.ToElement() != NULL) {
			maNode = monHandle.ToNode();
			ParcourirNoeud(maNode);
		} 


void __fastcall ParserXml::ParcourirNoeud(TiXmlNode* NodeAParcourir, bool estDansLeParent){
	bool dansLeNoeudParent = false;
	while(NodeAParcourir){
	//test
		AnsiString nom, valeur;
		ShowMessage("ok");
		nom = NodeAParcourir->Value();
		ShowMessage("et la ça bug");
	//
....
}

Merci, bonne journée

[Pyramidev]

Bonjour,

Est-ce que ça plante tout de suite après le premier "ok" ou bien après plusieurs "ok" ?
Dans le premier cas, je n'ai pas de piste.
Dans le deuxième cas, il faudrait qu'on voit le reste de la boucle while qui modifie sûrement NodeAParcourir (autrement, la condition du while n'aurait pas de sens).

[Maliono]

Merci pour ta réponse
et oui malheureusement ça plante dès la 1ère itération ... Pourtant je fais des test sur le document, le handle et la node, à ne rien comprendre ...

Pour te répondre ma boucle while a du sens. Je fais NextSibling(), si il n'y en a plus il devient NULL donc on sort de la boucle.
Je mets l'essentiel de ma fonction (en enlevant les divers test que je fais sur la node qui ne la modifie pas pour autant).

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
void __fastcall ParserXml::ParcourirNoeud(TiXmlNode* NodeAParcourir, bool estDansLeParent){
	bool dansLeNoeudParent = false;
	while(NodeAParcourir){
	//test
		AnsiString nom, valeur;
		ShowMessage("ok");
		nom = NodeAParcourir->Value();
		ShowMessage("ok");
		if(NodeAParcourir->ToElement()->FirstAttribute())
		valeur = NodeAParcourir->ToElement()->FirstAttribute()->Value();
	//
		if(!NodeAParcourir->NoChildren()){
			ParcourirNoeud(NodeAParcourir->FirstChild(), estDansLeParent);
		}
 
		NodeAParcourir = NodeAParcourir->NextSibling();
	}
}

[Pyramidev]

Je ne sais pas ce qui a fait crasher le programme.
Cependant, je vois que tu oublies de tester que ToElement() retourne un pointeur non nul.

Voici une autre version du code de ta fonction ParcourirNoeud :

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
25
void __fastcall ParserXml::ParcourirNoeud(const TiXmlNode& PremierNoeud, bool estDansLeParent)
{
	bool dansLeNoeudParent = false;
	for(const TiXmlNode* NodeAParcourir = &PremierNoeud;
	    NodeAParcourir != NULL;
	    NodeAParcourir = NodeAParcourir->NextSibling())
	{
	//test
		ShowMessage("ok");
		AnsiString nom = NodeAParcourir->Value();
		ShowMessage("ok");
		AnsiString valeur;
		const TiXmlElement* element = NodeAParcourir->ToElement();
		if(element) {
			const TiXmlAttribute* attribut = element->FirstAttribute();
			if(attribut)
				valeur = attribut->Value();
		}
	//
 
		const TiXmlNode* premierEnfant = NodeAParcourir->FirstChild();
		if(premierEnfant != NULL)
			ParcourirNoeud(*premierEnfant, estDansLeParent);
	}
}

[Maliono]

J'ai testé ta solution mais cela a empiré mon cas. Maintenant ça plante même en débug lorsque l'on passe par :

Code :

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

1
2
3
4
5
6
 
		const TiXmlElement* element = NodeAParcourir->ToElement();
		if(element) {
			const TiXmlAttribute* attribut = element->FirstAttribute();
			if(attribut)
				valeur = attribut->Value();

"valeur" contient des valeurs étonnantes, à la 1ère itération une flèche, à la seconde un carré puis à la 3eme plante...

[Bousk]

Tu check que toElement fonctionne, puis le transformes en node avec toNode, puis retourne en element. C'est au mieux bancal