处理BackGroundWorker的正确方法

Question

这是处理BackGroundWorker的正确方法吗？我不确定在调用.Dispose（）之前是否需要删除事件。另外调用RunWorkerCompleted委托内的.Dispose（）可以吗？

public void RunProcessAsync(DateTime dumpDate) 
{ 
    BackgroundWorker worker = new BackgroundWorker(); 
    worker.RunWorkerCompleted += new RunWorkerCompletedEventHandler(worker_RunWorkerCompleted); 
    worker.DoWork += new DoWorkEventHandler(worker_DoWork); 
    worker.RunWorkerAsync(dumpDate); 
} 

void worker_DoWork(object sender, DoWorkEventArgs e) 
{ 
    // Do Work here 
} 

void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e) 
{ 
    BackgroundWorker worker = sender as BackgroundWorker; 
    worker.RunWorkerCompleted -= new RunWorkerCompletedEventHandler(worker_RunWorkerCompleted); 
    worker.DoWork -= new DoWorkEventHandler(worker_DoWork); 
    worker.Dispose(); 
} 

Answer 1

BackgroundWorker派生自Component。组件实现了IDisposable接口。这反过来使BackgroundWorker继承Dispose（）方法。

派生自Component对于Windows Forms程序员来说很方便，他们可以将BGW从工具箱拖放到表单上。一般来说，组件可能有些东西需要处理。 Windows窗体设计器会自动处理此问题，请在Designer.cs文件中查找“组件”字段的窗体。其自动生成的Dispose（）方法为所有组件调用Dispose（）方法。

但是，BackgroundWorker实际上并没有任何需要处理的成员。它不重写Dispose（）。其基本实现Component.Dispose（）仅确保将组件从“组件”集合中删除。并提出Disposed事件。但没有其他处置任何东西。

长话短说：如果您在表单上放置了BGW，那么所有内容都会自动处理，您不必提供帮助。如果您没有将它放在表单上，​​那么它不是组件集合中的元素，也不需要做任何事情。

您不必调用Dispose（）。

Answer 2

是的，这似乎是正确的。当然，一次性物品最好用using块处理，但您在这里没有该选项。

我通常用表单生命周期创建我的背景handers，重用它们，并让设计器代码在窗体关闭时处理处理。少想一想。

Answer 3

如果它是一个“的WinForms”形式让容器照顾它（查看生成的代码Dispose在Form.Designer.xyz文件）

在实践中我发现，你可能需要创建一个如果有人知道一个更正式的方式来做到这一点，那么容器的实例并添加工人（或其他companent）到它！

PK :-)

public partial class Form1 : Form 
{ 
    public Form1() 
    { 
     InitializeComponent(); 

     // watch the disposed event.... 
     backgroundWorker1.Disposed += new EventHandler(backgroundWorker1_Disposed); 

     // try with and without the following lines 
     components = new Container(); 
     components.Add(backgroundWorker1); 
    } 

    void backgroundWorker1_Disposed(object sender, EventArgs e) 
    { 
     Debug.WriteLine("backgroundWorker1_Disposed"); 
    } 

//... from the Designer.xyz file ... 

    /// <summary> 
    /// Clean up any resources being used. 
    /// </summary> 
    /// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param> 
    protected override void Dispose(bool disposing) 
    { 
     if (disposing && (components != null)) 
     { 
      components.Dispose(); 
     } 
     base.Dispose(disposing); 
    } 

} 

Answer 4

晚了比赛，但我只是碰到与你的问题，我想我会分享一个场景跑去。如果您在课堂级别创建工作人员并在不关闭应用程序的情况下在连续的操作中重复使用它，如果您未在完成后移除事件，则它们将在每次后续执行时递增并运行多次。

worker.RunWorkerCompleted -= new RunWorkerCompletedEventHandler(worker_RunWorkerCompleted); 
worker.DoWork -= new DoWorkEventHandler(worker_DoWork); 

没有上面我DoWork的闪光一次，第一次，两次，第二次，等等。这可能是一个没有脑子的大多数，但我花了一点的数字出来，所以希望这将帮助别人。

Answer 5

worker.Dispose()不是必需的，因为Dispose()被自动调用。但在处理对象之前，您需要删除所有事件处理程序。

这个article通知我们这件事。

worker.RunWorkerCompleted -= new RunWorkerCompletedEventHandle(worker_RunWorkerCompleted); 
worker.DoWork -= new DoWorkEventHandler(worker_DoWork);