Тематические события - это «чистый» способ?

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. Не нужно ссылаться на каждый элемент управления, просто вызывать в любом месте потока пользовательского интерфейса.

Проверка InvokeRequired не обязательна, так как вы должны знать о том, в какой теме событие поднято.

В любом случае дублирующая логика, такая как "Current state: " + e.Step, является действительно плохой идеей, и я бы отказался от этого в обзоре кода независимо от того, что.

Наличие Application.DoEvents - очень плохой знак. Наверное, недоразумение, потому что имеет смысл называть его в потоке пользовательского интерфейса, но почему Invoke, когда уже на потоке пользовательского интерфейса ?! Похоже на противоречие.

lstStatus.Refresh(); также является недоразумением, вероятно, суеверным. Элементы управления обновляются автоматически (если вы разрешаете обработку событий).

Answer 2

Когда вы используете invoke, оператор добавляется в очередь, подлежащую обработке потоком пользовательского интерфейса.

Используйте это простое решение:

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