2

是否有任何理由将参数值分配给方法内的局部变量，以便在不更改它们的情况下使用这些值？即像下面这样：

private void MyMethod(string path)
{
    string myPath = path;
    StreamReader mystream = new StreamReader(myPath);
    ...
}

或者我可以一直这样说吗（上面的代码是多余的，只是不干净）：

private void MyMethod(string path)
{
    StreamReader mystream = new StreamReader(path);
    ...
}

我知道它是双向的，但我想确保在我的理解中没有遗漏任何东西。

4

3 回答 3

4

唯一需要执行此操作（本地分配）的情况是您在 foreach 循环中或使用 Linq。否则，您可能会遇到修改后的闭包问题。

这是来自 MSDN 博客的片段（以下所有内容均来自链接）。

http://blogs.msdn.com/b/ericlippert/archive/2009/11/12/closure-over-the-loop-variable-considered-harmful.aspx

但我正在超越自己。这个片段的输出是什么？

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());

大多数人期望它是 100 / 110 / 120。实际上是 120 / 120 / 120。为什么？

因为 ()=>v 的意思是“返回变量 v 的当前值”，而不是“返回创建委托时返回的值 v”。闭包关闭变量，而不是值。并且当方法运行时，很明显分配给 v 的最后一个值是 120，所以它仍然具有该值。

这非常令人困惑。编写代码的正确方法是：

foreach(var v in values) 
{
  var v2 = v;
  funcs.Add( ()=>v2 );
}

现在会发生什么？每次我们重新启动循环体时，我们都会在逻辑上创建一个全新的变量 v2。每个闭包都在不同的 v2 上关闭，该 v2 只分配一次，因此它始终保持正确的值。

基本上，问题的出现是因为我们指定 foreach 循环是一个语法糖

 {
    IEnumerator<int> e = ((IEnumerable<int>)values).GetEnumerator();
    try
    { 
      int m; // OUTSIDE THE ACTUAL LOOP
      while(e.MoveNext())
      {
        m = (int)(int)e.Current;
        funcs.Add(()=>m);
      }
    }
    finally
    { 
      if (e != null) ((IDisposable)e).Dispose();
    }
  }

如果我们指定扩展是

try
{ 
  while(e.MoveNext())
  {
    int m; // INSIDE
    m = (int)(int)e.Current;
    funcs.Add(()=>m);
  }

那么代码将按预期运行。

于 2012-04-22T17:13:31.310 回答
3

这完全一样，唯一的区别是，在第一种情况下，您制作了引用的副本（当方法超出范围时，它无论如何都会被销毁，这会在执行结束时发生）。

为了更好的可读性，坚持第二种情况。

于 2012-04-22T12:23:33.120 回答
1

我更喜欢第二种选择。使用参数创建新变量是没有意义的。此外，从阅读的角度来看，从路径（您收到的路径）创建流比实例化“myPath”变量更有意义。

于 2012-04-22T12:24:33.380 回答