Есть ли причина повторного использования C # переменной в foreach?

Компилятор объявляет переменную таким образом, что делает ее очень подверженной ошибкам, которые часто трудно найти и отладить, при этом не получая никаких ощутимых преимуществ.

Ваша критика полностью оправдана.

Здесь я подробно обсуждаю эту проблему:

Закрытие по переменной цикла считается вредным

Есть ли что-то, что вы можете делать с циклами foreach таким образом, что вы не могли бы, если бы они были скомпилированы с переменной внутреннего охвата? или это просто произвольный выбор, который был сделан до того, как анонимные методы и lambda-выражения были доступны или распространены и которые не были пересмотрены с тех пор?

Последний. Спецификация C # 1.0 фактически не указала, была ли переменная цикла внутри или снаружи тела цикла, поскольку она не делала заметной разницы. Когда семантика закрытия была введена в C # 2.0, был сделан выбор, чтобы поместить переменную цикла вне цикла, в соответствии с циклом «для».

Я считаю справедливым сказать, что все сожалеют об этом решении. Это один из худших «gotchas» на C #, и мы будем исправлять его, чтобы исправить это. В C # 5 переменная цикла foreach будет логически находиться внутри тела цикла, и поэтому замыкания будут получать новую копию каждый раз.

Цикл for не будет изменен, и изменение не будет «обратно портировано» на предыдущие версии C #. Поэтому вы должны быть осторожны при использовании этой идиомы.

То, что вы просите, полностью покрывается Эриком Липпертом в его блоге. Закрытие по переменной цикла считается вредной и ее продолжением.

Для меня наиболее убедительным аргументом является то, что наличие новой переменной в каждой итерации будет противоречить циклу стиля for(;;) . Вы ожидали бы иметь новый int i на каждой итерации for (int i = 0; i < 10; i++) ?

Наиболее распространенной проблемой с этим поведением является закрытие переменной итерации, и она имеет простой способ:

 foreach (var s in strings) { var s_for_closure = s; query = query.Where(i => i.Prop == s_for_closure); // access to modified closure 

Мое сообщение в блоге об этой проблеме: Закрытие переменной foreach в C # .

Будучи укушенным этим, у меня есть привычка включать локально определенные переменные в самый внутренний объем, который я использую для передачи на любое закрытие. В вашем примере:

 foreach (var s in strings) { query = query.Where(i => i.Prop == s); // access to modified closure 

Я делаю:

 foreach (var s in strings) { string search = s; query = query.Where(i => i.Prop == search); // New definition ensures unique per iteration. 

Как только у вас есть эта привычка, вы можете избежать этого в очень редком случае, когда вы намеревались привязываться к внешним областям. Честно говоря, я не думаю, что когда-либо делал это.

В C # 5.0 эта проблема исправлена, и вы можете закрыть переменные цикла и получить ожидаемые результаты.

Спецификация языка гласит:

8.8.4. Заявка foreach

(…)

Утверждение foreach формы

 foreach (V v in x) embedded-statement 

затем расширяется до:

 { E e = ((C)(x)).GetEnumerator(); try { while (e.MoveNext()) { V v = (V)(T)e.Current; embedded-statement } } finally { … // Dispose e } } 

(…)

Размещение v внутри цикла while важно для того, как он захватывается какой-либо анонимной функцией, встречающейся во встроенной инструкции. Например:

 int[] values = { 7, 9, 13 }; Action f = null; foreach (var value in values) { if (f == null) f = () => Console.WriteLine("First value: " + value); } f(); 

Если v был объявлен вне цикла while, он будет разделяться между всеми итерациями, а его значение после цикла for будет конечным значением 13 , что и будет вызывать вызов f . Вместо этого, поскольку каждая итерация имеет свою собственную переменную v , первая, захваченная f в первой итерации, будет продолжать удерживать значение 7 , которое будет напечатано. ( Примечание: более ранние версии C #, объявленные v за пределами цикла while ).

По-моему, это странный вопрос. Хорошо знать, как работает компилятор, но это только «полезно знать».

Если вы пишете код, который зависит от алгоритма компилятора, это плохая практика. И лучше переписать код, чтобы исключить эту зависимость.

Это хороший вопрос для собеседования. Но в реальной жизни я не сталкивался с какими-либо проблемами, которые я решил на собеседовании.

90% foreach использует для обработки каждого элемента коллекции (не для выбора или вычисления некоторых значений). иногда вам нужно вычислить некоторые значения внутри цикла, но не рекомендуется создавать цикл BIG.

Для вычисления значений лучше использовать выражения LINQ. Потому что, когда вы вычисляете много вещей внутри цикла, через 2-3 месяца, когда вы (или кто-либо еще) прочитаете этот код, человек не поймет, что это такое и как он должен работать.