Question

I have one XML file and I have created on function for retrieving value from XML nodes. Now the problem is if my node have no inner node, still flow goes inside the recursive sentence !

Have a look in my function

/*
XmlDocument x = new XmlDocument(); x.Load(filename);
*/
   public string getValue(string nodename)
    {

        if (File.Exists(filename))
        {


            foreach (XmlNode xx in x.SelectNodes("//"+nodename))
            {
                if (xx.HasChildNodes)
                    getValue(nodename + "//"+xx.ChildNodes[0].Name);
                else
                    return xx.InnerText;
            }   

        }
        return null;
    }

My xmlfile is

<oodle_response stat="ok">
    <current>
        <region>
            <id>barrie</id>
            <name>Barrie</name>
        </region>
        <start>1</start>
        <num>10</num>
    </current>
</oodle_response>

and I am calling my function through

protected void btnLets_Click(object sender, EventArgs e)
{
   string filename = Server.MapPath("~/xmls/newXmlFile.xml";
   oodleXmlParser ox = new oodleXmlParser(filename);
   Response.Write(ox.getValue("oodle_response"));
}

Answer 1

Now the problem is if my node have no inner node, still flow goes inside the recursive sentence !

It’s not quite right, e.g. in

<start>1</start>

‘1’ will be a XmlNode of type XmlNodeType.Text, so you need to check node type of child node instead(or maybe with) xx.HasChildNodes.
I think your method should look like this

public static string getValue(string nodename)
    {

        if (File.Exists(filename))
        {

            int i = 0;
            foreach (XmlNode xx in x.SelectNodes("//" + nodename))
            {                   
                if (xx.HasChildNodes && xx.ChildNodes[i].NodeType == XmlNodeType.Element)
                    return getValue(nodename + "//" + xx.ChildNodes[i].Name);//i'm sure it should return value
                else
                    return xx.InnerText;
                i++;
            }

        }
        return null;
    }