两种foreach遍历写法是否有差异？第二种是否更不优化？

嘿，这个问题问得挺到位的，我来给你掰扯清楚两种写法的区别和性能情况～

核心结论先放前面 #

大部分日常开发场景下，这两种写法的性能差异可以完全忽略，但如果Collection属性的getter实现比较特殊，写法二可能会带来逻辑问题或不必要的损耗。

先搞懂C# foreach的底层逻辑 #

很多人误以为写法二会在每次循环时都调用someClass.Collection的getter，但其实C#编译器会自动帮你做优化！当你写：

foreach (var elem in someClass.Collection) { /* 逻辑 */ }

编译器会把它翻译成类似这样的代码：

// 编译器自动生成的临时变量，只调用一次getter
var tempCollection = someClass.Collection;
using (var enumerator = tempCollection.GetEnumerator())
{
    while (enumerator.MoveNext())
    {
        var elem = enumerator.Current;
        // 你的循环逻辑
    }
}

看到没？不管你是手动把集合存到变量里（写法一），还是直接写在foreach里（写法二），最终都会只调用一次Collection的getter方法，性能上完全一致。

那什么时候写法二会踩坑？ #

只有当Collection的getter不是简单返回一个字段的时候，才会出现差异：

• 场景1：getter每次返回新的集合实例
  比如你的属性实现是这样的：

  public class SomeClass
  {
      private List<int> _internalList = new List<int> {1,2,3};
      public List<int> Collection
      {
          get { return new List<int>(_internalList); } // 每次调用都返回新列表
      }
  }
  

  这时候两种写法的逻辑结果其实还是一致的（因为foreach只取一次集合），但如果你的业务逻辑依赖于“循环过程中集合实例不变”，写法一的可读性会更强，能明确告诉其他开发者：我就是要遍历这个固定的集合。
• 场景2：getter包含昂贵操作或副作用
  比如getter里有数据库查询、复杂计算：

  public List<User> Collection
  {
      get { return DbContext.Users.Where(u => u.IsActive).ToList(); }
  }
  

  放心，不管哪种写法，都只会执行一次数据库查询（因为foreach只调用一次getter）。但如果是你不小心写成了for循环的错误写法：

  // 错误示例！每次循环都调用getter，性能爆炸
  for (int i = 0; i < someClass.Collection.Count; i++)
  {
      var elem = someClass.Collection[i];
  }
  

  这时候才会每次循环都触发getter，那性能就真的拉胯了——但这和foreach的写法无关，是循环方式的问题。

写法一的真正价值 #

写法一的优势其实不在性能，而在可读性和维护性：

• 如果someClass.Collection是很长的属性路径（比如AppContext.Current.Module.DataCollection），把它赋值给一个有意义的变量名（比如activeUsers），代码会清晰很多。
• 能明确避免后续代码修改时，不小心把foreach改成了每次都访问属性的错误写法（比如上面的for循环坑）。

内容的提问来源于stack exchange，提问作者sohi