5

Possible Duplicate:
Lambda Expression: == vs. .Equals()

Hi,

I use a lot the keyword Equals to compare variables and other stuff.

but

wines = wines.Where(d => d.Region.Equals(paramRegion)).ToList();

return an error at runtime when in the data Region is NULL

I had to use the code 

wines = wines.Where(d => d.Region == paramRegion).ToList();

to get rid of the error.

Any ideas why this raises an error?

Thanks.

Community
  • 1
  • 1
Filip
  • 1,098
  • 5
  • 17
  • 34
  • 1
    `Equals` in this context isn't a keyword - it is just a method. There **is** a LINQ contextual `equals` keyword, but only when doing a `join`. – Marc Gravell Jan 29 '11 at 14:53
  • This question has been answered before: http://stackoverflow.com/questions/2273600/lambda-expression-vs-equals – Victor Jan 29 '11 at 14:53

3 Answers3

5

You cannot call instance methods with null object reference. You should check that the Region is not null before calling its instance methods.

wines = wines.Where(d => d.Region != null && d.Region.Equals(paramRegion)).ToList();

The d.Region == paramRegion is (most likely) expanded to object.Equals(d.Region, paramRegion) and that static method does check whether the parameters are null or not before calling the Equals() method.

You can also write the condition in different order if you know that the paramRegion cannot be null.

Debug.Assert(paramRegion != null);
wines = wines.Where(d => paramRegion.Equals(d.Region)).ToList();
mgronber
  • 3,399
  • 16
  • 20
3

Basically if

d.Region == null

then any method call, here it's Equals(...) on that will raise an exception since it's not initialized.

laika
  • 1,319
  • 2
  • 10
  • 16
0

Use can use:

paramRegion.Equals(d.Region)
abatishchev
  • 98,240
  • 88
  • 296
  • 433