NullReferenceException when referencing an XML Element that doesn't exist

Question

I am receiving the following error when trying to access an XML element that doesnt exist using linq:

A first chance exception of type 'System.NullReferenceException' occurred in Test.exe System.NullReferenceException: Object reference not set to an instance of an object.

bool exists = website.Parent
            .Element("Exclusions")
            .Elements("Exclusion")
            .Where(x => thisKeyword.ToUpper().Contains((string)x.Value.ToUpper() ?? "~"))
            .Any();

I can remedy the issue by simply using .Where(x => thisKeyword.Contains((string)x))

But I need to convert both thisKeyword and the XML element value (if it exists) to upper case to get a case-insensitive comparison.

I was under the impression that ?? "~" meant that "if whats the left is null use what is to the right. So in this case x should become tilde, and therefore not validate against thisKeyword (as it wont ever contain tilde). A crude method I agree...but it doesn't even work.

How can I protect against the NullReferenceException?

Possible duplicate of What is a NullReferenceException, and how do I fix it? — Tony Hopkinson
– Tony Hopkinson, Commented Mar 12, 2017 at 23:09
Look at what is null, thisKeyword, x, or x.Value will kill you. Have to do a null check first. — Tony Hopkinson
– Tony Hopkinson, Commented Mar 12, 2017 at 23:11
I think that you can use the following code thisKeyword?.ToUpper().Contains((string)x?.Value.ToUpper() ?? "~") for checking that both aren't null — ganchito55
– ganchito55, Commented Mar 12, 2017 at 23:12

Alistair Green · Accepted Answer · 2017-03-12 23:33:16Z

1

Assuming that x.Value is null and thisKeyword is known to be non-null:

The crash is happening because the ?? operator is trying to check whether the value returned by ToUpper() is null. However, it can't call ToUpper() because x.Value does not exist.

In C# version 6 or later you can use the null propagation operator, ?., which accesses a property if and only if the object is non-null:

.Where(x => thisKeyword.ToUpper().Contains(
            (string)x.Value?.ToUpper() ?? "~")

Alternatively, just use a separate Where clause to check that x.Value is non-null:

.Where(x => x.Value != null)
.Where(x => thisKeyword.ToUpper().Contains(x.Value.ToUpper())

As an aside, you could put the condition inside the Any() clause instead:

.Elements("Exclusion")
.Any(x => x.Value != null && thisKeyword.ToUpper().Contains(x.Value.ToUpper())

And you don't necessarily need to convert to upper case either:

.Any(x => x.Value != null &&
          thisKeyword.IndexOf(x.Value,
             StringComparison.InvariantCultureIgnoreCase) > 0)

answered Mar 12, 2017 at 23:33

Alistair Green

4723 silver badges11 bronze badges

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

2 Comments

slugster Over a year ago

You are on the right track, but instead of messing around with the Elvis operator you can instead use something far simpler - the string.Equals(string, string, comparisonType) static method.It is tolerant of nulls, and you can specify a case insensitive check (which is the most common reason why people do a ToUpper() or ToLower() before checking).

James Over a year ago

Thanks for this. It looks like I cannot use the Value?. property so I must be on an older version of C#. My next issue is how I should deal with NullReferenceException if .Elements("Exclusion") and .Element("Exclusions") do not exist in the XML?

Collectives™ on Stack Overflow

NullReferenceException when referencing an XML Element that doesn't exist

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related