foreach,表现明智.我们应该在循环之前或里面声明变量吗？

Question

对于性能明智而言更好的是在foreach语句之外声明变量,并且每次都将它重新分配到它(foreach)或在foreach中创建一个新变量,例如

private List<ListItem> GetItems()
        {
            var items = new List<ListItem>();
            var collection = new List<int> { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
            ListItem item;
            foreach (var i in collection)
            {
                item = new ListItem { Text = i.ToString() };
                items.Add(item);
            }

            return items;
        }

还是这个？

private List<ListItem> GetItems()
        {
            var items = new List<ListItem>();
            var collection = new List<int> { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
            foreach (var i in collection)
            {
                ListItem item = new ListItem { Text = i.ToString() };
                items.Add(item);
            }

            return items;
        }

肯定在这里我说的是物品对象.谢谢你们.

Answer 1

这听起来像是一个不成熟的优化.

首先,您有理由相信这里存在性能问题吗？

其次,在发布版本中,编译器的优化器可能会为这两种情况生成相同的代码 - 因此它可能无关紧要.在调试版本中,这可能并非总是如此,但是您不希望进行优化,因为调试版本的目的是允许您准确地逐步执行代码.

Answer 2

还有就是在那里这重要的一个边缘情况; 如果你将变量"捕获"到一个匿名方法/ lambda中.否则它是不成熟的,没有任何区别.完全没有.

它何时起作用的一个例子:

// prints all items in no particular order
foreach (var i in collection)
{
    string s = i.ToString();
    ThreadPool.QueueUserWorkItem(delegate { Console.WriteLine(s); });
}

VS

// may print the same item each time, or any combination of items; very bad
string s;
foreach (var i in collection)
{
    s = i.ToString();
    ThreadPool.QueueUserWorkItem(delegate { Console.WriteLine(s); });
}