Discussion :

[rollbich]

bonsoir,

Je bute sur le problème suivant.
Avec libxml, je lis un fichier xml sans aucun problème et souhaiterais initialiser une structure avec le résultat. Le programme plante à l'éxécution.

La ligne avec printf affiche bien la donnée que je veux, elle est donc bien récupérée depuis le fichier xml.
Si je remplace la ligne strcpy par

Code :

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

strcpy(waypoints[i].wp_nom, "coco");

le code s'exécute sans 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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
static int nb_wp = 0;
 
typedef struct
{
    char wp_nom[5];
    int wp_x;
    int wp_y;
} s_waypoint;
 
s_waypoint *waypoints;
 
void import_wp()
{
    xmlDoc *doc = xmlReadFile("basic.xml", NULL, 0);
    if (doc == NULL)
    {
       g_print("Le fichier xml n'existe pas.\n");
    } else
    {
        xmlNode *xmlroot = xmlDocGetRootElement(doc)->children;
        xmlNode *current = NULL;
        int i = 0;
 
        for (current=xmlroot; current != NULL; current=current->next)
        {
            if (strcmp((char*)current->name, "targetpoint") == 0)
            {
                nb_wp++;
            }
        }
        waypoints = malloc(nb_wp*sizeof(s_waypoint));
       for (current=xmlroot; current != NULL; current=current->next)
        {
            printf("%s\n", (const char*) xmlGetProp(current, (const xmlChar*)"label"));
            strcpy(waypoints[i].wp_nom, (const char*) xmlGetProp(current, (const xmlChar*)"label"));
            i++;
        }
 
        xmlFreeDoc(doc);
        xmlCleanupParser();
    }
}

Le fichier xml :

Code :

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

1
2
3
4
5
<?xml version="1.0"?>
<Waypoints>
  <targetpoint x="398" y="470" label="MTL"/>
  <targetpoint x="473" y="288" label="LTP"/>
</Waypoints>

Je ne sais plus quoi faire. Un petit peu d'aide serait la bienvenue.
Merci

[Médinoc]

J'ai du mal à voir, mais déjà ça c'est suspect: strcmp((char*)current->name, "targetpoint")Pourquoi un cast est-il nécessaire? Quel est le type de name?

[rollbich]

J'ai repris le code d'un exemple sur internet pour lire un fichier xml.
current->name est le nom de la balise xml courante (soit ici targetpoint).
Je ne sais pas quel est le type retourné par xmlGetProp.
Il faudrait peut-être voir dans la doc de libxml pour savoir pourquoi il faut un cast mais comme je le disais, l'erreur ne vient pas de cette ligne puisque ça compile et s'exécute en modifiant la ligne avec strcpy.

Si tu veux, je peux poster le code en entier ?

[kwariz]

Bonjour,

quand on ne sait plus quoi faire : on utilise un debugger

Met un breakpoint sur ton printf qui plante et examine tes variables. Ton cuurent sera de type XML_TEXT_NODE, or toi ce que tu veux c'est un XML_ELEMENT_NODE pour pouvoir en récupérer les propriétés éventuelles ... et là tu te dis ... ah ok ... autant j'ai vérifié être sur un XML_ELEMENT_NODE de name "targetpoint" (ou presque ) dans la boucle précédente autant sur la seconde boucle je ne le fais pas. Tu ne vérifie pas non plus l'éventualité où il n'y a aucune propriété ...

En utilisant DDD (un débugger graphique on obtient la visualisation suivante de tes données) ce sera peut-être plus clair :

Le current et son next


Où se trouvent les propriétés et leurs valeurs


EDIT: la doc est ... pas forcément facilement lisible/navigable mais elle a le mérite d'exister et d'être complète. C'est vrai qu'en général les tutos laissent à désirer aussi (comme beaucoup de projets gnome).
Mais http://www.xmlsoft.org/html/libxml-tree.html#xmlGetProp précise que la valeur de retour peut être NULL -> c'est donc à vérifier.
Je n'ai pas souvent utilisé l'interface DOM, faut s'y faire et souvent avec des programmes de test avec pleins d'erreurs La plupart du temps j'ai utilisé l'interface SAX2.

[rollbich]

bonjour,

Un debugger !! Ca ne m'était même pas venu à l'idée !!! Il est temps de savoir utiliser celui de CodeBlocks.

autant j'ai vérifié être sur un XML_ELEMENT_NODE de name "targetpoint" (ou presque ) dans la boucle précédente autant sur la seconde boucle je ne le fais pas

Tu as raison et c'était ça le problème, le debugger m'indiquait 'segmentation fault strcat'. Avec la vérification dans la 2è boucle, y'a plus de pb.

Merci beaucoup, maintenant je ferais des vérifs systématiques.

Y'a toujours un autre bug par contre c'est quand je fais :

Code :

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

strcpy(waypoints[i].wp_x, (const char*) xmlGetProp(current, (const xmlChar*)"x"));

car waypoints[i].wp_x est déclaré de type int donc j'en déduis que je peux pas utiliser strcpy cette fois.

Comment je fais alors pour initialiser waypoints[i].wp_x ? Y a-t-il une fonction identique strcpy mais pour les int ??

[rollbich]

J'ai compris je peux faire directement

Code :

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

waypoints[i].wp_x = (int)xmlGetProp(current, (const xmlChar*)"x");

pour des int.

Par contre au lieu de 398, waypoints[0].wp_x contient 27854768 ????
une idée?