Abbruch foreach Schleife

oneill5000

Wie kommst du darauf, dass ich den Root Knoten lösche?
Und ich hab auch nicht geschrieben, dass ich den Root Knoten löschen will.

Ich hole mir eine Liste aller Knoten aus xeRootNode. Dann gehe ich diese Liste durch und wenn dort ein leerer Knoten bei ist, gehe ich zum übergeordneten Knoten (xeRootNode) und lösche diesen leeren Knoten (xnNode).
Das funktioniert auch, aber nur einmal, da dann die foreach Schleife frühzeitig abbricht.

mogel

oneill5000 schrieb:

Mein Problem ist, dass die foreach Schleife sofort abbricht, wenn das erste mal
xnNode.ParentNode.RemoveChild(xnNode);
aufgerufen wird, obwohl noch mehrere Knoten vorhanden sind.

es bricht nichts willkürlich ab ... Du verschluckst irgendwo eine Exception - etwa so

try
{
  // foo()
} catch(exception ex) { }

ansonsten kannst Du nicht einfach die Enumeration ändern wärend Du durchläufst - und genau diese Exception wird da geworfen ... sammle einfach die zu entfernen Elemente in einer anderen Liste und schmeiß sie anschließend raus

hand, mogel

oneill5000

Das alles beifindet sich im original in einem try catch Block. Es wird keine exception geworfen.

solick

sorry, hattezu flüchtig drüber gelesen.

Benutz mal ein Delegate um die aus der Liste zu löschen wg. der Enumeration.

oneill5000

Ich habe irgendwie noch keine Ansatz. Wie soll ich die Knoten denn sammeln?

Zur Info, die xml Datei könnte so aussehen

<?xml version="1.0" encoding="utf-8"?>
<entries>
    <entry>
    </entry>
    <entry>
    </entry>
    <entry>
        <header>test</header>
        <content>test</content>
        <entrydate>20100205</entrydate>
        <deletedate>20100206</deletedate>
        <user>Max</user>
    </entry>
</entries>

Dort sollen jetzt

<entry>
    </entry>
    <entry>
    </entry>

gelöscht werden.

solick

Du hast ja eine Node Liste. Wenn sie Ilist eingebunden hat, sollte folgendes funktionieren:

list.RemoveAll(delegate(string s) { return s == "irgendweineBedingung"; });

Müsstest Du dann noch auf Dein Problem anpassen, also vermutlich in der Art:

list.RemoveAll(delegate(XmlNode n) { return n == (xnNode.ChildNodes.Count == 0) ; });

Hoffe das funktioniert so...

oneill5000

Hmm. Nein, sowas gibt es nicht für eine XmlNodeList.

theta

Holzhammermethode:

XmlDocument document = new XmlDocument();
document.Load(@"..\..\test.xml");
XmlNode entriesNode = document.SelectSingleNode("entries");
XmlNodeList entryNodes = entriesNode.SelectNodes("entry");
List<XmlNode> entryNodesToDelete = new List<XmlNode>();
foreach (XmlNode entryNode in entryNodes)
{
	XmlElement entryElement = entryNode as XmlElement;
	if (entryElement != null)
	{
		if (entryElement.ChildNodes.Count == 0)
		{
			entryNodesToDelete.Add(entryElement);
		}
	}
}
foreach (XmlNode entryNode in entryNodesToDelete)
{
	entriesNode.RemoveChild(entryNode);
}
document.Save(@"..\..\test_new.xml");

Firefighter

Du kannst auch über LINQ To XML rangehen und löschen.

oneill5000

@theta

Danke, so funktioniert es.