Getting null reference Exception while parsing XML through LINQ

Question

Hi Guys I am having following type of XML to parse.

<?xml version="1.0" encoding="utf-8" ?>
<Functions>

<Function>
<Name>local:FunctionName</Name>
<Library>UserDefined</Library>
<Signature type="xs:decimal+">local:FunctionName(#?VALUE#,arg1,arg2,arg3) as xs:decimal+</Signature>
<Description>Some comments</Description>
<Code><![CDATA[some code]]></Code>
</Function>

<Function>
<Name>local:NewFunction</Name>
<Library>UserDefined</Library>
<Signature type="xs:decimal+">local:NewFunction(num1 as xs:string+,num2 as xs:string+,num3 as xs:string+) as xs:decimal+</Signature>
<Parameters>
     <Parameter type="xs:string" occurence="+" name="num1">num1 as xs:string+</Parameter>
     <Parameter type="xs:string" occurence="+" name="num2">num2 as xs:string+</Parameter>
     <Parameter type="xs:string" occurence="+" name="num3">num3 as xs:string+</Parameter>
</Parameters>
<Description>Some comments</Description>
<Code><![CDATA[some code]]></Code>
</Function>


</Functions>

and after Parsing I have to populate following Models

 List<Function> objFunctionsList = new List<Function>();

where Function class is as follows

 public class Function
    {
        public String Name { get;set;}

        public Parameter ReturnType { get; set;}

        public List<Parameter> Parameters { get; set;}

        public String Library{get;set;}    

        public String Signature{get; set;}

        public String Description{get;set;}

        public String Code{get;set;}
    }

   and the Parameter class is as follows

public class Parameter
{
    [DefaultValue("")] 
    public String Name{get;set;}

    [DefaultValue("")]
    public String DataType{get;set;}

    [DefaultValue("")]
    public String OccurenceType {get; set;}
}

The Code that i am trying is below. I am calling function written below to get list populated.

public static List<Function> GetAllFunctions(String pXMLPath)
        {
            List<Function> objFunctionsList = new List<Function>(); 

            try
            {
                XmlDocument xmlDoc = new XmlDocument();
                xmlDoc.Load(pXMLPath);


                XElement fList = XElement.Parse(xmlDoc.InnerXml);

                // In following 10 lines I am getting Null-reference exception
                var Functions = fList
                    .Element("Functions")
                    .Elements("Function")
                    .Select(Function => new
                        {
                            Name = (string)Function.Element("Name"),
                            Library = (string)Function.Element("Library"),
                            Signature = (string)Function.Element("Signature"),
                            ReturnType = (string)Function.Attribute("type"),
                            Description = (string)Function.Element("Description"),
                            Code = (string)Function.Element("Code"),
                            Parameters =  GetParamsList(Function)
                        });

                foreach (var f in Functions)
                {
                    Function objFunction = new Function();
                    objFunction.Name = f.Name;
                    objFunction.Library = f.Library;
                    objFunction.Description = f.Description;
                    objFunction.Code = f.Code;
                    objFunction.Signature = f.Signature;
                    objFunction.State = 0;//Unmodified (Save not required)

                    #region Populating Return Type

                    string returnType = f.ReturnType;
                    string signature = f.Signature;

                    Parameter objReturnType = new Parameter();
                    if (String.IsNullOrEmpty(returnType) || returnType == Constants.XSNOPARAM)
                    {
                        objReturnType.Name = string.Empty;
                        objReturnType.OccurenceType = string.Empty;
                        objReturnType.DataType = Constants.XSNOPARAM;
                    }
                    else
                    {

                        if (returnType.EndsWith(Constants.ASTERIK))
                        {
                            objReturnType.DataType = returnType.Substring(0, returnType.Length - 1);
                            objReturnType.OccurenceType = Constants.OCCURENCES_ASTERISK;
                        }
                        else if (returnType.EndsWith(Constants.PLUS))
                        {
                            objReturnType.DataType = returnType.Substring(0, returnType.Length - 1);
                            objReturnType.OccurenceType = Constants.OCCURENCES_PLUS;
                        }
                        else if (returnType.EndsWith(Constants.QUESTION_MARK))
                        {
                            objReturnType.DataType = returnType.Substring(0, returnType.Length - 1);
                            objReturnType.OccurenceType = Constants.OCCURENCES_QUESTION;
                        }
                        else if (returnType.Length > 0)
                        {
                            objReturnType.DataType = returnType;
                        }

                    } 
                    #endregion

                    objFunction.ReturnType = objReturnType;

                    objFunction.Parameters = new List<Parameter>();
                    foreach (var param in f.Parameters.ToList())
                    {
                        Parameter objParam = new Parameter();

                        objParam.Name = param.Name;
                        objParam.DataType = param.DataType;
                        objParam.OccurenceType = param.OccurenceType;

                        objFunction.Parameters.Add(objParam);
                    }

                    objFunctionsList.Add(objFunction);

                }

            }
            catch (Exception ex)
            {

            }
            return objFunctionsList;
        }

private static List<Parameter> GetParamsList(XElement pParametersElement)
        {
            if (pParametersElement != null)
            {
                   var Parameters = pParametersElement
                    .Elements("Parameter")
                   .Select(Parameter => new
                   {
                       Name = (string)Parameter.Attribute("name"),
                       Occurence = (string)Parameter.Attribute("occurence"),
                       Type = (string)Parameter.Attribute("type")
                   });

                List<Parameter> objParamsList = new List<Parameter>();

                foreach (var param in Parameters)
                {
                    Parameter objParam = new Parameter();
                    objParam.Name = param.Name;
                    objParam.OccurenceType = param.Occurence;
                    objParam.DataType = param.Type;

                    objParamsList.Add(objParam);
                }
                return objParamsList;
            }
            else
                return null;
        }

Mainly in following code i m getting Exception

 var Functions = fList
                    .Element("Functions")
                    .Elements("Function")
                    .Select(Function => new
                        {
                            Name = (string)Function.Element("Name"),
                            Library = (string)Function.Element("Library"),
                            Signature = (string)Function.Element("Signature"),
                            ReturnType = (string)Function.Attribute("type"),
                            Description = (string)Function.Element("Description"),
                            Code = (string)Function.Element("Code"),
                            Parameters =  GetParamsList(Function)
                        });

enter image description here

Stack Trace

Type : System.NullReferenceException, mscorlib, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089
Message : Object reference not set to an instance of an object.
Source : MyApp.XEG.Utility
Help link : 
Data : System.Collections.ListDictionaryInternal
TargetSite : System.Collections.Generic.List`1[MyApp.XEG.Model.Function] GetAllFunctions(System.String)
Stack Trace :    at MyApp.XEG.Utility.CFLFile.GetAllFunctions(String pXMLPath) in C:\MyApp\XEG.Utility\CFLFile.cs:line 189

here the line 189 is the highlighted line in yellow

Can you post the exception stack trace; or at least indicate whether the exception is originating inside GetParamsList? — Daniel Renshaw
– Daniel Renshaw, Commented Dec 9, 2011 at 7:30
why did you use a screen shot of the xml? that makes it really hard for us to try to repro to help you... — Marc Gravell
– Marc Gravell, Commented Dec 9, 2011 at 7:31

Henk Holterman · Accepted Answer · 2011-12-09 09:59:44Z

2

The basic reason for the null ref:

 var Functions = fList
 //  .Element("Functions")
   .Elements("Function")
   ....

flist already is the <Functions> element.

edited Dec 9, 2011 at 9:59

answered Dec 9, 2011 at 7:44

Henk Holterman

276k33 gold badges353 silver badges540 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Abhi Over a year ago

This is also a right answer , but i can mark only one as right , anyways thanks a lot.

Marc Gravell · Accepted Answer · 2011-12-09 07:42:09Z

1

Errors:

swalling the exception!
you need var Functions = fList.Elements("Function"). etc - no "Functions" first (at the top)
you need Parameters = GetParamsList(Function.Element("Parameters")) (just below that)
you need to null-check the parameters: if (f.Parameters != null) foreach(var param in f.Parameters) {...} (near the bottom of GetAllFunctions)

with those changes, it works. There is also no need for XmlDocument in here; XElement.Load() would be preferable. Personally I'd use XmlSerializer instead, but ... meh.

answered Dec 9, 2011 at 7:42

Marc Gravell

1.1m273 gold badges2.6k silver badges3k bronze badges

1 Comment

Abhi Over a year ago

Thanks a lot Marc for detailed description

Daniel Renshaw · Accepted Answer · 2011-12-09 07:25:52Z

0

Try using xmlDoc.OuterXml in your parse. You're only parsing the content inside the document element but then trying to reference the document element in the first .Element() call.

More explicitly, try changing

XElement fList = XElement.Parse(xmlDoc.InnerXml);

to

XElement fList = XElement.Parse(xmlDoc.OuterXml);

answered Dec 9, 2011 at 7:25

Daniel Renshaw

34.3k8 gold badges78 silver badges95 bronze badges

4 Comments

Abhi Over a year ago

Then also same problem , actually using Outer fList is gettign same value including this line added <?xml version="1.0" encoding="utf-8" ?> , which is present in XML

Abhi Over a year ago

i checked in debugging mode fList is gettign complete XML content as value

Henk Holterman Over a year ago

I was going to say: Don't use InnerXml (OuterXml) or Parse() at all. Just Load an XDocument from the filename.

Abhi Over a year ago

XmlDocument xmlDoc = new XmlDocument(); xmlDoc.Load(pXMLPath); XElement fList = XElement.Parse(xmlDoc.InnerXml);

Collectives™ on Stack Overflow

Getting null reference Exception while parsing XML through LINQ

3 Answers 3

1 Comment

1 Comment

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related