LinqToSql странное поведение

У меня есть следующий код:


var tagToPosts = (from t2p in dataContext.TagToPosts
                                    join t in dataContext.Tags on t2p.TagId equals t.Id
                                    select new { t2p.Id, t.Name });
//IQueryable tag2postsToDelete;
foreach (Tag tag in tags)
{
    Debug.WriteLine(tag);
    tagToPosts = tagToPosts.Where(t => t.Name != tag.Name);
}
IQueryable tagToPostsToDelete = (from t2p in dataContext.TagToPosts
                                            join del in tagToPosts on t2p.Id equals del.Id
                                            select t2p);
dataContext.TagToPosts.DeleteAllOnSubmit(tagToPostsToDelete);

где tags это List<Tag> - список тегов, созданных конструктором, поэтому у них нет идентификатора. Я запускаю код, который ставит точку торможения на линию с Debug, и жду нескольких циклов. Затем я помещаю tagToPosts.ToList() в окно Watch для выполнения запроса. В профилировщике SQL я вижу следующий запрос:

exec sp_executesql N'SELECT [t0].[Id], [t1].[Name]
FROM [dbo].[tblTagToPost] AS [t0]
INNER JOIN [dbo].[tblTags] AS [t1] ON [t0].[TagId] = [t1].[Id]
WHERE ([t1].[Name]  @p0) AND ([t1].[Name]  @p1) AND ([t1].[Name]  @p2)',N'@p0 nvarchar(4),@p1 nvarchar(4),@p2 nvarchar(4)',@p0=N'tag3',@p1=N'tag3',@p2=N'tag3'

Мы можем видеть, что каждый параметр параметра имеет значение последнего tag.Name в цикле. Есть ли у вас какие-либо идеи о том, как добиться этого и получить цикл, чтобы добавить Where с новым состоянием каждый тым? Я могу видеть IQueryable хранит только указатель на переменную перед выполнением.

Источник

2 ответа

Решение

Измени свой foreach чтобы:

foreach (Tag tag in tags){
    var x = tag.Name;
    tagToPosts = tagToPosts.Where(t => t.Name != x);
}

Причиной этого является ленивая оценка и захват переменных. В основном то, что вы делаете в foreach утверждение не отфильтровывает результаты, как это может выглядеть. Вы строите дерево выражений, которое зависит от некоторых переменных, которые нужно выполнить. Важно отметить, что фактические переменные записываются в деревья выражений, а не их значения во время захвата. Поскольку переменная tag используется каждый раз (и это сфера является целым foreachтаким образом, он не выйдет из области видимости в конце каждой итерации), он захватывается для каждой части выражения, и последнее значение будет использоваться для всех его вхождений. Решение состоит в том, чтобы использовать временную переменную, имеющую область видимости в foreach, чтобы она выходила из области видимости на каждой итерации, а на следующей итерации она будет считаться новой переменной.

Источник

Да, ответ Мердада правильный. Когда замыкание захватывает переменную в C# (что и происходит, когда ваша лямбда "Где" ссылается на переменную "тег"), компилятор применяет довольно точное правило о том, как сделать захват. Если захваченная переменная находится в той же области видимости, которая определена в окружающих скобках {и}, то значение этой переменной будет зафиксировано как есть. Если он находится за пределами области действия, то будет захвачена только ссылка на эту переменную. В вашем исходном сообщении переменная "tag" уже перебрала весь цикл до последнего значения. Но если вы сделаете модификацию, предложенную Мехрдадом, то вы захватите переменную в той же области видимости, поэтому индивидуальное значение переменной будет встроено в ваше замыкание, давая вам желаемые результаты.

Кстати, вы можете сказать: "Да, но переменная" tag "находится в той же области". Но это не совсем так, потому что под капотом компилятор превращает for-each в нечто вроде этого (это ОЧЕНЬ грубо, просто чтобы показать, что происходит с скобками):

{ 
   var iterator = GetTheIterator();
   { 
      while(iterator.MoveNext())
      {
          // Your loop code here
      }
   }
}

Дело в том, что ваш итератор для for-each всегда будет находиться в области видимости, отличной от той, в которой находится код вашего цикла.

Источник
Другие вопросы по тегам