this.Invoke(...) - 这是一种不好的做法吗？

Question

我有一个名为ExecuteCommand的函数,它根据用户的输入执行操作.这些事情的范围可以从简单地做一个Console.Writeline(),检查我的表单上的复选框,或模拟键盘到另一个进程,完全独立于我自己.该函数在单独的线程上运行,因此更改UI将需要一些调用.我有两种方法可以做到这一点......其中一种我不确定是一种好方法,但它很容易.

下面的代码,第3行是我有一个问题:

private void ExecuteCommand()
{
   this.Invoke((MethodInvoker)delegate()
   {
      if (current_line_index < command_que.Count)
      {
         current_line = command_que[current_line_index];

         if (current_line.StartsWith(">>Auto Enter"))
         {
            chkAutoEnter.Checked = false;
         }
         else if (current_line.StartsWith("+WinWait("))
         {
            string title_to_wait_for = current_line;

            title_to_wait_for = title_to_wait_for.Remove(0, "+WinWait(\"".Length);          
            title_to_wait_for = title_to_wait_for.Remove(title_to_wait_for.Length - 2, 2);

            t_WinWait = new Thread(() => WinWait(title_to_wait_for));
            t_WinWait.Name = "WinWait";
            t_WinWait.Start();
         }
      }
   });
}

代码完美无缺......但我不确定这是不是很好的做法.

或者,我知道我可以做这样的事情来改变UI:

private delegate void CheckCheckBoxHandler(bool checked);

private void CheckCheckBox(bool checked)
{
   if (this.chkAutoEnter.InvokeRequired)
   {
       this.chkAutoEnter.Invoke(new CheckCheckBoxHandler(this.CheckCheckBox), checked);
   }
   else
   {
       chkAutoEnter.Checked = checked;
   }
}

但是因为我在我的表单上有多个控件将从另一个线程更改,所以我必须添加一些函数来执行此操作,而不是第一个示例中的简单方法.

反正的第一种方式是不是很糟糕？有没有涉及的风险我还没有遇到过？真的好似乎......

谢谢!

Answer 1

不,不错.你调用哪个控件并不重要,Invoke因为它们都具有相同的效果. Invoke在拥有该控件的线程上调用委托 - 只要所有控件都由同一个线程拥有,那么就没有区别.