Question

I have a class called PurchaseOrderItem

public class PurchaseOrderItem
{

    public Int64 PONumber { get; set; }
    public string Description { get; set; }
    public string UM { get; set; }
    public int QTY { get; set; }
    public double Cost { get; set; }
}

I’m trying to to read data from XML file by iterating sections from XML and populate data into PurchaseOrderItems which is List. But when i try the code below i’m getting an error message of “Object reference not set to an instance of an object.”
I’m pretty sure i’m missing something because my poor OOP knowledge. Would someone kindly explain to me what causes this problem?

 PurchaseOrderItems =
    (from purchaseOrderItem in PO.Descendants("PurchaseOrder").Elements("ProductLineItem")
    select new PurchaseOrderItem
    {
        PONumber = PONumber,
        Description = purchaseOrderItem.Element("PurchaseOrder").Element("ProductLineItem").Element("comments").Value.Trim(),
        QTY = Convert.ToInt16(purchaseOrderItem.Element("PurchaseOrder").Element("ProductLineItem").Element("OrderQuantity").Element("requestedQuantity").Element("ProductQuantity").Value.Trim()),
        UM = purchaseOrderItem.Element("PurchaseOrder").Element("ProductLineItem").Element("GlobalProductUnitOfMeasureCode").Value.Trim(),
        Cost = Convert.ToDouble(purchaseOrderItem.Element("PurchaseOrder").Element("ProductLineItem").Element("requestedUnitPrice").Element("FinancialAmount").Element("MonetaryAmount").Value.Trim()),
    }).ToList<PurchaseOrderItem>();

Answer 1

purchaseOrderItem is already a ProductLineItem element by the time you reach your select clause – so currently you’re trying to find a PurchaseOrder element within ProductLineItem, which will return null if it isn’t found. I suspect you just want:

PurchaseOrderItems =
    (from purchaseOrderItem in PO.Descendants("PurchaseOrder").Elements("ProductLineItem")
    select new PurchaseOrderItem
    {
        PONumber = PONumber,
        Description = purchaseOrderItem.Element("comments").Value.Trim(),
        QTY = (short) purchaseOrderItem.Element("OrderQuantity")
                                        .Element("requestedQuantity")
                                        .Element("ProductQuantity"),
        UM = purchaseOrderItem.Element("GlobalProductUnitOfMeasureCode")
                              .Value.Trim(),
        Cost = (double) purchaseOrderItem.Element("requestedUnitPrice")
                                         .Element("FinancialAmount")
                                         .Element("MonetaryAmount")
    }).ToList();

Notes:

• Using the explicit conversions from XElement to short, double etc is a more appropriate way of performing conversions for XML data
• It looks like you’re using double for currency values: don’t do that – use decimal instead

Also, you may want to consider creating a static method in PurchaseOrderItem which knows how to deserialize one from an XElement; I often find that a pattern of:

class Foo
{
    static Foo FromXElement(XElement element) { ... }
    XElement ToXElement() { ... }
}

works well.