Question

我正在开发一个跨线程的应用程序，它在Invoke：

中有一些方法

if (this.InvokeRequired)
{
    this.Invoke((Action)(() =>
    {
        pbTotalProgress.Value = progress;
        labe1.Text="SomeText";
    }));
}
else
{
    pbTotalProgress.Value = progress;
    labe1.Text="SomeText";
}

如何将代码编写得更短？

Answer 1

您至少应该只定义一次操作（在else分支中没有再次使用代码，只需在没有调用的情况下运行该操作。
您还可以编写一个控件的扩展方法，如果需要，可以将Action作为调用运行，或者只运行它。这将消除事件处理程序的复杂性，最终您将在应用程序中为许多事件使用相同的模式。

static class ControlExtensions
{
  public static void InvokeOrExecute(this Control control, Action action)
  {
     if (control.InvokeRequired)
     {
        control.Invoke(action);
     }
     else
     {
        action();
     }
  }
}

然后在每个可能是跨线程的事件中：

Action setProgress = delegate()
     {
        pbTotalProgress.Value = progress;
        labe1.Text = "SomeText";
     };
this.InvokeOrExecute(setProgress);

Answer 2

由于它是相同的代码块，您可以在Action变量中设置它并Invoke /执行它。尝试类似：

Action setProgress = delegate() {
    pbTotalProgress.Value = progress;
    labe1.Text = "SomeText";
};
if (this.InvokeRequired) {
    this.Invoke(setProgress);
} else {
    setProgress();
}

您的确切代码并不需要优化。唯一发生的事情是您正在对布尔标志this.InvokeRequired执行条件检查。如果有的话，我的回答只提供了一次编写Action代码而不是复制它的能力。

简化winforms代码跨线程调用

2 个答案: