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

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

另外，我知道我可以做这样的事情来改变用户界面：

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在拥有该控件的线程上调用委托 - 只要所有控件都由同一个线程拥有，那么就没有区别。

this.Invoke（...） - 这是一种不好的做法吗？

1 个答案: