当前位置:  开发笔记 > 运维 > 正文

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

如何解决《应该将"行动"代表重构为新方法吗?》经验,为你挑选了1个好方法。

我有以下方法

    private void SetProcessDocumentStatus(string status)
    {
        var setStatusWith = new Action(
            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"委托.



1> RossFabrican..:

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

我可能会把第一行写成:

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

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

推荐阅读
大大炮
这个屌丝很懒,什么也没留下!
DevBox开发工具箱 | 专业的在线开发工具网站    京公网安备 11010802040832号  |  京ICP备19059560号-6
Copyright © 1998 - 2020 DevBox.CN. All Rights Reserved devBox.cn 开发工具箱 版权所有