Problem with Deleting QDomNode

[metdos]

Here my source code:

QDomElement rootOfTag=fieldState.firstChild().firstChildElement(parent);
QDomNodeList sameTags=rootOfTag.elementsByTagName(node.firstChild().toElement().tagName());
for(uint i=0;i<sameTags.length();i++)
{
	QDomNode nodeToReplace=sameTags.item(i);
	if(nodeToReplace.toElement().attribute("id")==node.firstChild().toElement().attribute("id"))
        {
		rootOfTag.removeChild(nodeToReplace);
		return true;
	}
}

QDomElement rootOfTag=fieldState.firstChild().firstChildElement(parent);
QDomNodeList sameTags=rootOfTag.elementsByTagName(node.firstChild().toElement().tagName());
for(uint i=0;i<sameTags.length();i++)
{
	QDomNode nodeToReplace=sameTags.item(i);
	if(nodeToReplace.toElement().attribute("id")==node.firstChild().toElement().attribute("id"))
        {
		rootOfTag.removeChild(nodeToReplace);
		return true;
	}
}

To copy to clipboard, switch view to plain text mode 

Although it enters into if part, it can't remove "nodeToReplace" . What can be my problem?

Edit: nodeToReplace.nodeType() returns QDomNode::ElementNode, can problem related with that?

[Lykurg]

Are you sure that nodeToReplace is a direct child of rootOfTag? How does a sample XML file look like?

[metdos]

fieldState is QDomDocument,
parent="Blocks"
node.firstChild().toElement().tagName() returns "Block"

<root>
<Blocks>
   <Block id="1"></Block>
   <Block id="2"></Block>
   <Block id="3"></Block>
</Blocks>
</root>

<root>
<Blocks>
   <Block id="1"></Block>
   <Block id="2"></Block>
   <Block id="3"></Block>
</Blocks>
</root>

To copy to clipboard, switch view to plain text mode 

[Lykurg]

Hm, in the if block you have

return true;

return true;

To copy to clipboard, switch view to plain text mode 

which exits your function, so how does the function definition looks like? Is fieldState a copy or a pointer so that the changes you make are propagated to your "real" QDomDocument? That's the only reason I can think of. May be you want check inside the if block if the node is deleted.

[metdos]

Whole Function:

bool FieldModel::updateNode(QDomNode node,QString parent){
 
	QDomElement rootOfTag=fieldState.firstChild().firstChildElement(parent);
	QDomNodeList sameTags=rootOfTag.elementsByTagName(node.firstChild().toElement().tagName());
	for(uint i=0;i<sameTags.length();i++){
		QDomNode nodeToReplace=sameTags.item(i);
		if(nodeToReplace.toElement().attribute("id")==node.firstChild().toElement().attribute("id")){
			rootOfTag.removeChild(nodeToReplace);
			qDebug()<<"Hello:\n"<<fieldState.toString();
			return true;
		}
	}
	insertNode(node,parent);
	return true;
}

bool FieldModel::updateNode(QDomNode node,QString parent){

	QDomElement rootOfTag=fieldState.firstChild().firstChildElement(parent);
	QDomNodeList sameTags=rootOfTag.elementsByTagName(node.firstChild().toElement().tagName());
	for(uint i=0;i<sameTags.length();i++){
		QDomNode nodeToReplace=sameTags.item(i);
		if(nodeToReplace.toElement().attribute("id")==node.firstChild().toElement().attribute("id")){
			rootOfTag.removeChild(nodeToReplace);
			qDebug()<<"Hello:\n"<<fieldState.toString();
			return true;
		}
	}
	insertNode(node,parent);
	return true;
}

To copy to clipboard, switch view to plain text mode 

Its output from line 9.

Hello:
 "<FieldState>
 <Blocks>
  <Block id="3T">
   <OccupiedState>1</OccupiedState>
   <TanzimState>0</TanzimState>
   <ProtectionState>0</ProtectionState>
   <FailureState>0</FailureState>
  </Block>
  <Block id="001BT">
   <OccupiedState>1</OccupiedState>
   <TanzimState>0</TanzimState>
   <ProtectionState>0</ProtectionState>
   <FailureState>0</FailureState>
  </Block>
 </Blocks>
 <Switches/>
 <Signals/>
 <LevelCrossings/>
 <Routes/>
</FieldState>
"

Hello:
 "<FieldState>
 <Blocks>
  <Block id="3T">
   <OccupiedState>1</OccupiedState>
   <TanzimState>0</TanzimState>
   <ProtectionState>0</ProtectionState>
   <FailureState>0</FailureState>
  </Block>
  <Block id="001BT">
   <OccupiedState>1</OccupiedState>
   <TanzimState>0</TanzimState>
   <ProtectionState>0</ProtectionState>
   <FailureState>0</FailureState>
  </Block>
 </Blocks>
 <Switches/>
 <Signals/>
 <LevelCrossings/>
 <Routes/>
</FieldState>
"

To copy to clipboard, switch view to plain text mode 

fieldState is Class member and it is fine.

[metdos]

http://www.qtcentre.org/threads/1231...de-in-xml-file

This solved my problem.

[Lykurg]

Are you sure that nodeToReplace is a direct child of rootOfTag?

http://www.qtcentre.org/threads/1231...de-in-xml-file

This solved my problem.

You could have had it earlier...