Внешняя переменная ловушка
Что именно является ловушкой внешней переменной? Пояснения и примеры в C# приветствуются.
РЕДАКТИРОВАТЬ: Включение Diktat Джон Скит:)
5 ответов
"Ловушка внешней переменной" возникает, когда разработчик ожидает, что значение переменной будет захвачено лямбда-выражением или анонимным делегатом, когда фактически переменная захвачена сама.
Пример:
var actions = new List<Action>();
for (var i = 0; i < 10; i++)
{
actions.Add(() => Console.Write("{0} ", i));
}
foreach (var action in actions)
{
action();
}
Возможный вывод № 1:
0 1 2 3 4 5 6 7 8 9
Возможный вывод № 2:
10 10 10 10 10 10 10 10 10 10
Если вы ожидали выхода #1, вы попали в ловушку внешней переменной. Вы получаете вывод № 2.
Fix:
Объявите "Внутреннюю переменную", которая будет захвачена несколько раз вместо "Внешней переменной", которая захватывается только один раз.
var actions = new List<Action>();
for (var i = 0; i < 10; i++)
{
var j = i;
actions.Add(() => Console.Write("{0} ", j));
}
foreach (var action in actions)
{
action();
}
Для получения дополнительной информации см. Также блог Эрика Липперта.
Что-то вроде
foreach (var s in strings)
var x = results.Where(r => (r.Text).Contains(s));
Не даст ожидаемых результатов, поскольку Contains не выполняется для каждой итерации. Присвоение s временной переменной внутри цикла исправит это.
Стоит отметить, что эта ловушка существовала для foreach
циклы тоже, но был изменен с C# 5.0, то есть внутри foreach
замыкания циклов теперь закрываются каждый раз над свежей копией переменной цикла. Итак, код ниже:
var values = new List<int>() { 100, 110, 120 };
var funcs = new List<Func<int>>();
foreach (var v in values)
funcs.Add(() => v);
foreach (var f in funcs)
Console.WriteLine(f());
Печать 120 120 120
100 110 120
>= C# 5.0
тем не мение for
петли по-прежнему ведут себя так же.
@dtb является правильным (большое +1), но важно отметить, что это применимо, только если область действия замыкания выходит за пределы цикла. Например:
var objects = new []
{
new { Name = "Bill", Id = 1 },
new { Name = "Bob", Id = 5 },
new { Name = "David", Id = 9 }
};
for (var i = 0; i < 10; i++)
{
var match = objects.SingleOrDefault(x => x.Id == i);
if (match != null)
{
Console.WriteLine("i: {0} match: {1}", i, match.Name);
}
}
Это напечатает:
я: 1 матч: Билл я: 5 матч: Боб я: 9 матч: Дэвид
ReSharper предупредит о "Доступе к измененному закрытию", который в этом случае можно смело игнорировать.
Эта статья, объясняющая концепцию замыканий, полезна:
http://en.wikipedia.org/wiki/Closure_(computer_science)
Кроме того, эта статья действительно хороша из более конкретной реализации C#:
http://blogs.msdn.com/b/abhinaba/archive/2005/08/08/448939.aspx
В любом случае, tl;lr заключается в том, что область видимости переменной так же важна в выражениях анонимных делегатов или лямбда-выражений, как и где-либо еще в вашем коде - поведение просто не так очевидно.