Getting object reference not set to an instance in PLINQ forall loop

Asked Feb 02 '17 at 14:50

Active Feb 02 '17 at 14:51

Viewed 18 times

I have written a code like following:

 newFilteredList.AsParallel().WithDegreeOfParallelism(60).ForAll(qqmethod =>
{
    var xdocic = new XmlDocument();
    xdocic.LoadXml(SendRequest(null, null, qqmethod.ItemID, true, TotalDaysSinceLastUpdate.ToString(), null));
    int TotalPages = 0;
    if (xdocic.GetElementsByTagName("TotalNumberOfPages").Item(0) != null)
    {
        TotalPages = Convert.ToInt32(xdocic.GetElementsByTagName("TotalNumberOfPages").Item(0).InnerText);
    }
    if (TotalPages > 1)
    {
        Parallel.For(1, TotalPages + 1, i =>
        {

            Products.Add(//Add something here);

        });
    }
    else
    {
        Products.Add(//add omething here);
    }
});

The line where I get error is the following:

 var xdocic = new XmlDocument();
 xdocic.LoadXml(SendRequest(null, null, qqmethod.ItemID, true, TotalDaysSinceLastUpdate.ToString(), null));

And it says that some of these variables is either null or I'm missing something here.

On a side note, TotalDaysSinceLastUpdate variable is declared just as a regular integer variable, is that why I'm getting this error?

What could it be guys?

edited Feb 02 '17 at 14:51

DavidG

95,392
10
185
181

asked Feb 02 '17 at 14:50

User987

3,273
10
38
85

What variable is null? What does your debugger say? – rory.ap Feb 02 '17 at 14:51
@rory.ap I couldn't tell by looking at it because the exception occured while the app was running as standalone exe... – User987 Feb 02 '17 at 14:52
@rory.ap do you want me to post the stacktrace? – User987 Feb 02 '17 at 14:52
Set your debugger to break on managed exceptions... – rory.ap Feb 02 '17 at 14:52
No, integer.ToString will never raise this exception, no value type can be a null-reference. But `qqmethod` can be null, so the item in the list. – Tim Schmelter Feb 02 '17 at 14:55
@TimSchmelter should i add check if(qqmethod != null) { // do some stuff here ?} – User987 Feb 02 '17 at 15:02
1

@User987: that would be a good idea. Or don't let them into the list in the first place – Tim Schmelter Feb 02 '17 at 15:11
@TimSchmelter you're the man ,thanks =) – User987 Feb 02 '17 at 15:13

Getting object reference not set to an instance in PLINQ forall loop

0 Answers0