应该将"行动"代表重构为新方法吗？

Question

我有以下方法

    private void SetProcessDocumentStatus(string status)
    {
        var setStatusWith = new Action<string>(
            statusValue => processDocumentStatusLabel.Text = statusValue);
        if (processDocumentStatusLabel.InvokeRequired)
            processDocumentStatusLabel.Invoke(
                (MethodInvoker)(() => setStatusWith(status)));
        else
            setStatusWith(status);
    }

从上面的代码中,我将动作封装到setStatusWith.该行为是否应重构为另一种方法如下？

    private void SetProcessDocumentStatusWith(string status)
    {
        processDocumentStatusLabel.Text = status;
    }

    private void SetProcessDocumentStatus(string status)
    {
        if (processDocumentStatusLabel.InvokeRequired)
            processDocumentStatusLabel.Invoke(
                (MethodInvoker)(() => SetProcessDocumentStatusWith(status)));
        else
            SetProcessDocumentStatusWith(status);
    }

我想知道是否应该在代码中谨慎使用"Action"委托.

Answer 1

你所拥有的东西对我来说似乎非常清楚.将lambda移动到单独的函数会增加代码行但不清晰.

我可能会把第一行写成:

  Action<string> setStatusWith = 
    statusValue => processDocumentStatusLabel.Text = statusValue;

但我不知道哪种方式通常是首选.