c# - 简化winforms代码跨线程调用

Question

我正在开发一个跨线程应用程序，其中包含一些方法Invoke：

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

我怎样才能把下面的代码写得更短一点？

Answer 1

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

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

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