XmlNodeReader.S kip Method not working as expected

[jambalapamba]

I am trying to skip element in xml document where style attribute is VISIBILITY: hidden or DISPLAY: none.

In the following example i want to skip the last two elements but when i skip the element with id=”Div1”
and next reader.Read() reads “ includeme” but i expected it to read element <a id ="Div2" style="visibili ty:hidden "> and skip it instead its writing “ includeme” to the console

sample xml

<a id="HyperLink1" href="jsisland.htm" >NEXT</a>
<a id="HyperLink2" href="framset.htm" class="pop" >gotoframeset</a>
 
<div id="Div1" style="DISPLAY: none">
   <a>first</a>
   <a1>second</a>
</div>
 
<a3 id ="Div2" style="VISIBILITY: hidden "> includeme</a3>
 

My code

             XmlDocument xDoc = new XmlDocument();
            xDoc.LoadXml(xmldoc);
            XmlNodeReader reader = new XmlNodeReader(xDoc);
 
            while (!reader.EOF)
            {
 
                    reader.Read();
                    XmlNodeType nodeType = reader.NodeType;
 
                    if (nodeType == XmlNodeType.Element)
                    {
                        if (reader.HasAttributes)
                        {
 
                            for (int i1 = 0; i1 < reader.AttributeCount; i1++)
                            {
                                reader.MoveToAttribute(i1);
                                 switch (reader.Name)
                                {
                                    case "style":
                                       if (reader.Value.Contains("VISIBILITY: hidden") || (reader.Value.Contains("DISPLAY: none")))
                                        {
                                            reader.Skip();
                                        }
 
                                        break;
                                }
                            }
 
                        }
                    }
                    else if (nodeType == XmlNodeType.Text)
                    {
 
                        Console.WriteLine("other" + reader.Value.ToString()); 
                        Console.ReadLine();
                    }
 
           } //end while
 

Can any one help me with this

Thank You.

[Plater]

Seems to me your code is doing exactly what it should do.
You tell it to skip nodes that have visibility:hidd en or Display:none
So it skips your <div> and <a3> tag.
And arrives at the text tag "includeme" which it is told to print out.
Although I would also expect it to print out "other first" and "other second" too. (According to your console.writeli ne() )

EDIT: On that note, I would think it would print out the whitespace you have in the file too.

[jambalapamba]

Seems to me your code is doing exactly what it should do.
You tell it to skip nodes that have visibility:hidd en or Display:none
So it skips your <div> and <a3> tag.
And arrives at the text tag "includeme" which it is told to print out.
Although I would also expect it to print out "other first" and "other second" too. (According to your console.writeli ne() )

EDIT: On that note, I would think it would print out the whitespace you have in the file too.

Its printing "other first" and "other second" but i want to skip "includeme" as well because its element value of <a3> tag which i want to skip all the child nodes inside <a3> element.

[Plater]

Its printing "other first" and "other second" but i want to skip "includeme" as well because its element value of <a3> tag which i want to skip all the child nodes inside <a3> element.

Well that's not the code you have written, you're not skipping the "first" and "second" nodes either, even though they are children of the <div> tag that should be skipped.
I think you need to go to next sibbling, and not next child. Not sure if "tree logic" is correct here, but it seems like "skip" just moves to the next node, regardless of if it's a child or a sibbling (or even a step back "up")

[jambalapamba]

I am really sorry skip method skips entire element my code is not displaying "first" and "second". The skip method skips entire element and its children as well.

[Plater]

Will it skip correctly if you do this with your existing code?

<a3 id ="Div2" style="VISIBILITY: hidden "><sometag> includeme</sometag></a3>
 

[jambalapamba]

no its not because its not reading <a3> only if it reads a3 then it goes in to the
nodetype.elemen t loop and skips it, but the problem is its not reading <a3> it is going to the value. Everytime i skip an element its not reading next element tag its reading the value.

[Plater]

Are you going to next AND skip?

[jambalapamba]

when ever i find style attribute in a element with the condition hidden i will be skipping that entire element and i will be reading next node

[Plater]

But if you .Skip() (going to next node) and then .Next(), you are moving twice?