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

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

Для меня самым убедительным аргументом является то, что наличие новой переменной в каждой итерации будет противоречить циклу стиля for(;;). Ожидаете ли вы, что на каждой итерации for (int i = 0; i < 10; i++) есть новый int 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 в С#.

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

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.

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

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

В спецификации языка указано:

8.8.4 Оператор foreach

(...)

A для любой формулировки формы

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, что и будет напечатано. (Примечание: более ранние версии С# объявлен v вне цикла while.)

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

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

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

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

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