CODEWITHSUNDEEP
c#linqlambdaclosures
one.beat.consumer
one.beat.consumer

Reputation: 9504

Access to Modified Closure in Lambda Expression

foreach(var category in categories) {
    a.AddRange(_db.Articles.Where(c => c.Categories.Contains(category)));
}

The code runs fine, yet I get a warning about "access to modified closure" in reference to category used in the lambda expression.

Question: Is the warning of any consequence in this circumstance?

Upvotes: 4

Views: 2091

Answers (2)

Alexey Raga
Alexey Raga

Reputation: 7525

It tells you that the "category" variable lives in closure and can be modified outside your LINQ expression.

Look at the question here for some explanation.

Upvotes: 0

JaredPar
JaredPar

Reputation: 754545

The warning here is because you are accessing the variable category inside the closure for the Where lambda. The value category changes with every iteration and Where is delay executed hence it will see the current value of category vs. the value at the time the lambda was created.

In this case you are likely fine. Even though Where is delay evaluated the AddRange method is prompt and will force the evaluation of Where to completion. Hence the Where method will see the value of category it expects.

If you'd like to remove the warning though simply declare a local copy of the iteration variable and capture that instead. 

foreach(var category in categories) {
  var localCategory = category;
  a.AddRange(_db.Articles.Where(c => c.Categories.Contains(localCategory)));
}

Upvotes: 6

Related Questions