访问修改后的闭包：ReSharper

Question

我创建了一个处理数据库访问的库。我最近添加了事务处理；但是，我遇到了一个小问题。为了概述这一点，我编写了这个示例用于演示目的：

class Program
{
    static void Main(string[] args)
    {
        String data = null;
        DoAction(ref data, () =>
        {
            Console.WriteLine(data);
        });
        Console.ReadLine();
    }

    private static void DoAction(ref String data, Action action)
    {
        if (data == null)
            data = "Initialized Data";
        action();
    }
}

我在“数据”变量的以下代码行中得到“访问修改后的闭包”下划线：

Console.WriteLine(data);

我了解修改 ref 数据变量可能会导致问题（例如，在运行 foreach 循环时）。但是，在以下情况下，我认为不会发生这种情况。

这是另一个版本，其中有一个循环进一步更改变量 - 输出符合预期：

class Program
{
    static void Main(string[] args)
    {
        String data = null;
        for (var i = 0; i < 10; i++)
            DoAction(ref data, () =>
            {
                Console.WriteLine(data);
            });
        Console.ReadLine();
    }

    private static void DoAction(ref String data, Action action)
    {
        if (data == null)
            data = "Initialized Data";
        else
            data += "|";

        action();
    }
}

ReSharper 让我可以创建一个局部变量，但我明确希望使用从 DoAction() 方法创建的字符串。如果我接受 ReSharpers 方法，它实际上会破坏代码。有没有其他方法可以解决这个问题？我想使用这种 Action 方法，但我也不希望 ReSharper 抱怨它（并且可能不会禁用 ReSharpers 检查）。

有什么建议吗？

Answer 1

我建议首先避免为此使用ref 参数——这对我来说似乎是不必要的复杂。我会将DoAction 重写为：

static string DoAction(string data, Action<string> action)
{
    data = data == null ? "Initialized Data" : data + "|";
    action(data);
    return data;
}

那么你可以：

data = DoAction(data, Console.WriteLine);

或者如果你想使用 lambda 表达式：

data = DoAction(data, txt => Console.WriteLine(txt));

如果您之后实际上不需要结果，您可以将 DoAction 设为 void 方法。 （不清楚为什么您需要返回结果和在DoAction 中执行的委托，但大概这在您更广泛的上下文中更有意义。）

Answer 2

如果您确定该警告不合适，InstantHandleAttribute 记录为：

告诉代码分析引擎参数是否被完全处理 当调用的方法在堆栈上时。如果参数是委托， 表示在执行方法时执行委托。 如果参数是可枚举的，则表示它是可枚举的 方法执行时。

我想这正是你想要的。

您可以从 JetBrains.Annotations 包中获取属性，或者从 ReSharper 选项中复制粘贴。