Problems reading XML with QDomDocument

[grantbj74]

Thanks for replies,

I tried both suggestions. Now it only collects the root element. Then after NextSibling is called it breaks out of the while loop. So nothing is collected. I must be missing something.

[kwisp]

I use this code to read xml.

bool CSKHttp::GetDataFromXml(CSKRequest* req)
{
    QDomDocument    domDoc;
    QString         errorString;
    int             errorLine;
    int             errorColumn;
    req->m_buffer.open(QIODevice::ReadOnly);
    QByteArray byteArray(req->m_buffer.readAll());
    req->m_buffer.close();
    if(!(domDoc.setContent(byteArray,false,&errorString,&errorLine,&errorColumn))) {
        mocW("rq")<<"xml document error <MESS:"<<errorString<<"><LINE"
        <<errorLine<<"><COLUMN"<<errorColumn<<">"<<endl;
        mocW("rq")<<"<ANSWER:"<<byteArray<<">"<<endl;
        return false;
    }
    QDomElement domElement = domDoc.documentElement();
    byteArray.replace("\n"," ");
    mocD("rq")<<"<ANSWER:"<<byteArray<<">"<<endl;
    TraverseXmlNode(domElement,req);
    return true;
}
 
void CSKHttp::TraverseXmlNode(const QDomNode& node, CSKRequest* req)
{
    QDomNode domNode = node.firstChild();
    QDomElement domElement;
    while(!(domNode.isNull())) {
        if(domNode.isElement()) {
            domElement = domNode.toElement();
            if(!(domElement.isNull()))
                req->Data(domElement.tagName(),domElement.text());
        }
        TraverseXmlNode(domNode,req);
        domNode = domNode.nextSibling();
    }
}

bool CSKHttp::GetDataFromXml(CSKRequest* req)
{
    QDomDocument    domDoc;
    QString         errorString;
    int             errorLine;
    int             errorColumn;
    req->m_buffer.open(QIODevice::ReadOnly);
    QByteArray byteArray(req->m_buffer.readAll());
    req->m_buffer.close();
    if(!(domDoc.setContent(byteArray,false,&errorString,&errorLine,&errorColumn))) {
        mocW("rq")<<"xml document error <MESS:"<<errorString<<"><LINE"
        <<errorLine<<"><COLUMN"<<errorColumn<<">"<<endl;
        mocW("rq")<<"<ANSWER:"<<byteArray<<">"<<endl;
        return false;
    }
    QDomElement domElement = domDoc.documentElement();
    byteArray.replace("\n"," ");
    mocD("rq")<<"<ANSWER:"<<byteArray<<">"<<endl;
    TraverseXmlNode(domElement,req);
    return true;
}

void CSKHttp::TraverseXmlNode(const QDomNode& node, CSKRequest* req)
{
    QDomNode domNode = node.firstChild();
    QDomElement domElement;
    while(!(domNode.isNull())) {
        if(domNode.isElement()) {
            domElement = domNode.toElement();
            if(!(domElement.isNull()))
                req->Data(domElement.tagName(),domElement.text());
        }
        TraverseXmlNode(domNode,req);
        domNode = domNode.nextSibling();
    }
}

To copy to clipboard, switch view to plain text mode 

may be useful for you.

[spirit]

I wrote this in a browser, but I'm sure that this code works fine.

QDomDocument doc;
QFile file("phones.xml");
if (!file.open(QIODevice::ReadOnly))
	return;
if (!doc.setContent(&file)) {
	file.close();
	return;
}
file.close();
 
QDomElement docElem = doc.documentElement();
 
QDomNode n = docElem.firstChild();
while(!n.isNull()) {
	QDomElement e = n.toElement();
	if(!e.isNull()) {
		if (e.tagName() == "phone") {
			qDebug() << "location" << e.attribute("location");
			qDebug() << "value" << e.attribute("value");
		} else if (e.tagName() == "settings") {
			qDebug() << "AfterHourOffset" << e.attribute("AfterHourOffset");
			qDebug() << "NumPerButton" << e.attribute("NumPerButton");
			qDebug() << "MaxNumToDial" << e.attribute("MaxNumToDial");
		}
	}
	n = n.nextSibling();
}

QDomDocument doc;
QFile file("phones.xml");
if (!file.open(QIODevice::ReadOnly))
	return;
if (!doc.setContent(&file)) {
	file.close();
	return;
}
file.close();

QDomElement docElem = doc.documentElement();

QDomNode n = docElem.firstChild();
while(!n.isNull()) {
	QDomElement e = n.toElement();
	if(!e.isNull()) {
		if (e.tagName() == "phone") {
			qDebug() << "location" << e.attribute("location");
			qDebug() << "value" << e.attribute("value");
		} else if (e.tagName() == "settings") {
			qDebug() << "AfterHourOffset" << e.attribute("AfterHourOffset");
			qDebug() << "NumPerButton" << e.attribute("NumPerButton");
			qDebug() << "MaxNumToDial" << e.attribute("MaxNumToDial");
		}
	}
	n = n.nextSibling();
}

To copy to clipboard, switch view to plain text mode 

[grantbj74]

Thanks poeple.

Spirits version worked for me.

I ended up making mine a combination of Spirit and victor.fernandez ideas.

I'm a bit scared to do recursive programming unless its required.

[kwisp]

Thanks poeple.
....
I'm a bit scared to do recursive programming unless its required.

don`t be afraid it`s realy easy.

[victor.fernandez]

I'm a bit scared to do recursive programming unless its required.

There is some risk of making a mistake and entering an infinite recursion but no more than there is with a plain for(). Being a bit careful, you should not be afraid of it. Also keep in mind that by traversing a QDomDocument you won't have any problem because it's a plain tree without loop backs.

[victor.fernandez]

You get the first child, which would be "phones". Then you try to get its siblings but it doesn't have any. You need to get the children of phones. So you can do:

QDomDocument d;
d.setContent(&file);
 
QDomNode n = d.firstChild();
if(!n.isNull() || !n.hasChildNodes())
    return;
 
n = n.firstChild();
 
ClearPhoneNumbers();    // Clear phone table.
 
while(!n.isNull()) {
...

QDomDocument d;
d.setContent(&file);
 
QDomNode n = d.firstChild();
if(!n.isNull() || !n.hasChildNodes())
    return;
 
n = n.firstChild();

ClearPhoneNumbers();    // Clear phone table.
 
while(!n.isNull()) {
...

To copy to clipboard, switch view to plain text mode