线程安全事件 - 这是一种“干净”的方式吗？

Question

我偶然发现了专业库中的一些代码，并且不确定这是否是处理跨线程事件调用的干净方式。

以下代码位于表单应用程序中。线程调用是从一个类本身开始一个新的线程和接收消息：

private void Library_StatusChanged(object sender, AbstractTestCase.StatusChangedEventArgs e) 
{ 
    if (this.InvokeRequired) 
    { 
     this.lblProgress.Invoke((MethodInvoker)delegate() 
     { 
      lblProgress.Text = "Current state: " + e.Step; 
      lblProgress.Refresh(); 
     } 
     ); 

     this.pbProgess.Invoke((MethodInvoker)delegate() 
     { 
      pbProgess.Value = e.Percentage; 
      pbProgess.Refresh(); 
     }); 

     this.lstStatus.Invoke((MethodInvoker)delegate() 
     { 
      lstStatus.Items.Add(" " + e.Step); 
      lstStatus.Refresh(); 

     }); 

     this.Invoke((MethodInvoker)delegate() 
     { 
      this.Refresh(); 
     }); 
    } 
    else 
    { 
     lblProgress.Text = "Current state:" + e.Step; 
     lblProgress.Refresh(); 

     pbProgess.Value = e.Percentage; 
     pbProgess.Refresh(); 

     lstStatus.Items.Add(" " + e.Step); 
     lstStatus.Refresh(); 

     this.Refresh(); 
    } 

    Application.DoEvents(); 
} 

这是“最先进的”？在我看来这有点凌乱？！

Answer 1

技术现状是使用await。如果这在这里是不可能的，至少简化代码到一个单一的Invoke调用。无需在每个控件上调用，只需调用UI线程上的任何位置即可。

该InvokeRequired检查不应该被要求，因为你应该知道什么线程的事件引发。

在任何情况下，重复逻辑如"Current state: " + e.Step真的是一个坏主意，无论如何我都会在代码审查中失败。

Application.DoEvents的存在是一个非常糟糕的迹象。可能是一个误解，因为它只在UI线程上调用它是有道理的，但为什么Invoke已经在UI线程上？！看起来像是一个矛盾。

lstStatus.Refresh();也是一种误解，可能是迷信。控件自动刷新（如果您允许进行事件处理）。

Answer 2

当您使用invoke时，将在队列中添加一条语句以供UI线程处理。

使用这个简单的解决方案：

private void Library_StatusChanged(object sender, AbstractTestCase.StatusChangedEventArgs e) 
{ 
    this.lblProgress.Invoke((MethodInvoker)delegate() 
    { 
     lblProgress.Text = "Current state: " + e.Step; 
    }); 

    this.pbProgess.Invoke((MethodInvoker)delegate() 
    { 
     pbProgess.Value = e.Percentage; 
    }); 

    this.lstStatus.Invoke((MethodInvoker)delegate() 
    { 
     lstStatus.Items.Add(" " + e.Step); 
    }); 
}