Reading XML files with irrXML

[sudi]

maybe its just me but i can't find a good explanation about how to write
an xml file.

Ok this is the programm code to read the file:

Code: Select all

void CXML::LoadConfig(char* File = "Data/config.xml")
{
          IrrXMLReader* xml = createIrrXMLReader(File);

     while(xml && xml->read())
	{
		switch(xml->getNodeType())
		{
		case EXN_ELEMENT:
			
				if (!strcmp("screenx", xml->getNodeName()))
                config.x = xml->getAttributeValueAsInt("x");
                else
                if (!strcmp("screeny", xml->getNodeName()))
                config.y = xml->getAttributeValueAsInt("y");
                break;
		}
	}

	// delete the xml parser after usage
	delete xml;
	
}

and this is the xml file.

Code: Select all

<?xml version="1.0"?>
<config>

<screenx x="1024" />
<screeny y="768" />

</config>

I'm really lost with this and would like some help...

[sudi]

does really nobody knows what i'm doing wrong or how i could do it?

[hybrid]

So you have code and data, where's your problem. Would be good to poin out your problems, errors received etc.

[sudi]

Thats the point their is no compile error it compieles but it wont read the data.

[hybrid]

I don't see where you defined the config variable. Might be a problem with its scope. Try to output the config variable before returning from your reading function in order to check for correct values.

[sudi]

what?
sry but i dint catch that....what should i do?
Isn't there someone out here who already did this and could help with a modification in my code?

[hybrid]

You have a struct (probably object of some class) named config in your code. You refer to config.x and config.y in the lines you read the attributes. This object has to be created somewhere. It apparently is because otherwise your app would crash, but it could be destroyed before you can access the values. This depends on the way you create this object. So try to output the values of config.x and config.y at the end of your posted method. That way you can check if the values are correctly read from the file. If so there is something wrong with the config object. If not you have to fix the way of reading attributes.

[sudi]

Ok i already did that so there must be a problem with reading the attributes
but i don't know where the problem is.

[hybrid]

Does it enter the if statement, i.e. is at some point the strcmp successful such that Attribute method is called? Does getNodeName return char* or wchar_t* ?

[sudi]

should return char* cause i use the extern irrxml parser.
but i don't know if the atribute methode is called. it doesn't return code errors it only doesn't do anything

[hybrid]

Just add a cout to both if statements (and don't forget curly braces) to check if they are reached. And add -Wall (or equivalent) to your compiler options to avoid too simple errors.

[sudi]

its not working.......there is no output at all.

[hybrid]

So it's not with the attribute reading, but the XML parser does not reach the correct node. Check the API and examples if you have to add some default handling for the other nodes of your file. Maybe add output to the main loop to check which nodes are visited and which are not.

[sudi]

Thanks hybrid but u aren't helping...lol....why isn't this working??????AHHHHHHH

[pfo]

Did you close the xml tag? i.e. last line should be "</xml>" otherwise I don't think the parser can grab anything out of the xml