C＃取消后台工作

Question

C＃2008

我使用下面的代码登录到一个软电话的DoWork的。但是，登录过程是一个漫长的过程，因为有很多事情必须进行初始化和检查，我只在这里放了一些，因为它会使代码长时间发布。

在下面的代码中，我检查CancellationPending是否在取消按钮单击事件中调用了CancelAsync，然后再执行每次检查。它是否正确？此外，如果检查失败，我还会调用CancelAsync并将e.Cancel设置为true。

我想知道我的方法，我在这里使用的是最好的方法来使用。

非常感谢任何建议，

private void bgwProcessLogin_DoWork(object sender, DoWorkEventArgs e) 
    { 
     /* 
     * Perform at test to see if the background worker has been 
     * cancelled by the user before attemping to continue to login. 
     * 
     * Cancel background worker on any failed attemp to login 
     */ 

     // Start with cancel being false as to reset this if cancel has been set to true 
     // in the cancel button. 
     e.Cancel = false; 

     NetworkingTest connection_test = new NetworkingTest(); 
     if (!this.bgwProcessLogin.CancellationPending) 
     { 
      // Check local LAN or Wireless connection    
      if (!connection_test.IsNetworkConnected()) 
      { 
       // Update label 
       if (this.lblRegistering.InvokeRequired) 
       { 
        this.lblRegistering.Invoke(new UpdateRegisterLabelDelegate(UpdateRegisterLabel), "No network connection"); 
       } 
       else 
       { 
        this.lblRegistering.Text = "No network connection"; 
       } 
       // Failed attemp 
       this.bgwProcessLogin.CancelAsync(); 
       e.Cancel = true; 
       return; 
      } 
      // Report current progress 
      this.bgwProcessLogin.ReportProgress(0, "Network connected"); 
     } 
     else 
     { 
      // User cancelled 
      e.Cancel = true; 
      return; 
     } 

     // Test if access to Server is available 
     if (!this.bgwProcessLogin.CancellationPending) 
     { 
      if (!connection_test.IsSIPServerAvailable()) 
      { 
       // Update label 
       if (this.lblRegistering.InvokeRequired) 
       { 
        this.lblRegistering.Invoke(new UpdateRegisterLabelDelegate(UpdateRegisterLabel), "Server unavailable"); 
       } 
       else 
       { 
        this.lblRegistering.Text = "Server unavailable"; 
       } 
       // Failed attemp 
       this.bgwProcessLogin.CancelAsync(); 
       e.Cancel = true; 
       return; 
      } 
      // Report current progress 
      this.bgwProcessLogin.ReportProgress(1, "Server available"); 
     } 
     else 
     { 
      // User cancelled 
      e.Cancel = true; 
      return; 
     } 
     . 
     . 
     . 
} 


private void bgwProcessLogin_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e) 
    { 
     // Check for any errors 
     if (e.Error == null) 
     { 
      if (e.Cancelled) 
      { 
       // User cancelled login or login failed     
      } 
      else 
      { 
       // Login completed successfully     
      } 
     } 
     else 
     { 
      // Something failed display error 
      this.statusDisplay1.CallStatus = e.Error.Message; 
     } 
    } 


private void bgwProcessLogin_ProgressChanged(object sender, ProgressChangedEventArgs e) 
    { 
     this.lblRegistering.Text = e.UserState.ToString(); 
    } 

private void btnCancel_Click(object sender, EventArgs e) 
    { 
     // Cancel the logging in process 
     this.bgwProcessLogin.CancelAsync(); 
     this.lblRegistering.Text = "Logged out"; 
} 

Answer 1

可能只有一个问题：如果DoWork事件处理程序中的某个操作会持续很长时间。在这种情况下，您可以在操作完成后中止待处理操作。如果DoWork事件中的所有操作都不能持续很长时间（例如，不超过5秒），那么它的一切正常，但如果其中一项操作可以持续很长时间（例如5分钟），则在这种情况下，用户有等到这个操作完成。

如果DoWork包含持久的操作，您可以使用AbortableBackgroundWorker之类的东西。事情是这样的：

public class AbortableBackgroundWorker : BackgroundWorker 
{ 
    private Thread workerThread; 

    protected override void OnDoWork(DoWorkEventArgs e) 
    { 
     workerThread = Thread.CurrentThread; 
     try 
     { 
      base.OnDoWork(e); 
     } 
     catch (ThreadAbortException) 
     { 
      e.Cancel = true; //We must set Cancel property to true! 
      Thread.ResetAbort(); //Prevents ThreadAbortException propagation 
     } 
    } 


    public void Abort() 
    { 
     if (workerThread != null) 
     { 
      workerThread.Abort(); 
      workerThread = null; 
     } 
    } 
} 

在这种情况下，你可以真正中止等待的操作，但也有一些限制（有关详细信息，关于中止托管线程和一些限制看到Plumbing the Depths of the ThreadAbortException Using Rotor）。

P.S.我同意Oliver的观点，你应该以更有用的形式包装InvokeRequired。

Answer 2

你正在做正确的方式，我相信。你会发现允许你终止或中止一个线程的线程成员，但你不想用它来做这样的事情。在你的代码中拥有所有的“取消”检查可能看起来有点奇怪，但是这允许你准确地控制你何时退出你的线程。如果你“粗暴地”放弃工作者线程，那么线程无法控制它何时退出，并且可能会损坏状态。

Answer 3

有一件事我不需要调用this.bgwProcessLogin.CancelAsync（）;因为你可以设置这个e.Cancel = true;

Answer 4

在您的DoWork（）函数中，您写了...。根据显示的两个结构的相同结构的多少任务，您可以将此结构重构为一个自己的方法，将不断变化的部分作为参数。

此外，InvokeRequired if-else分支已将输出字符串加倍。在stackoverflow或网页上的一个小搜索应该会显示出一个模式来完成这个加倍。

Evernything其他看起来相当不错。