16

我在一个可以工作的类中创建了一个后台工作人员，但是如果我打电话并等到最后运行，第二次调用它，它将执行两次相同的过程

我认为bw.DoWork +=有问题

private void button1_Click(object sender, EventArgs e)
{
    nptest.test.start("null", "null");    
}


namespace nptest
{
    class test
    {
        public static void start(string str, string strb)
        {
            if (bw.IsBusy != true)
            {
                bw.WorkerSupportsCancellation = true;
                bw.DoWork += (obj, e) => bw_DoWork(str, strb);
                bw.RunWorkerCompleted += new RunWorkerCompletedEventHandler(bw_RunWorkerCompleted);
                bw.RunWorkerAsync();
            }
        }
        private static BackgroundWorker bw = new BackgroundWorker();
        private static void bw_DoWork(string str, string strb)
        {
            System.Windows.Forms.MessageBox.Show("initializing BackgroundWorker");
        }
        private static void bw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            if ((e.Cancelled == true))
            {
                Console.WriteLine("Canceled");
            }
            else if (!(e.Error == null))
            {
                Console.WriteLine("Error: " + e.Error.Message);
            }
            bw.Dispose();

        }
    }
}

问题解决了

  class test
    {
        private static List<object> arguments = new List<object>();

        // initializing with program startup
        public static void bwinitializing()
        {
            bw.WorkerSupportsCancellation = true;
            bw.DoWork += new DoWorkEventHandler(bw_DoWork);
            bw.RunWorkerCompleted += new RunWorkerCompletedEventHandler(bw_RunWorkerCompleted);
        }

        public static void start(string str, string strb)
        {
            if (bw.IsBusy != true)
            {
                arguments.Clear();
                arguments.Add(str);
                arguments.Add(strb);
                bw.RunWorkerAsync(arguments);
            }
        }
        private static BackgroundWorker bw = new BackgroundWorker();
        private static void bw_DoWork(object sender, DoWorkEventArgs e)
        {
            List<object> genericlist = e.Argument as List<object>;
            System.Windows.Forms.MessageBox.Show("BackgroundWorker " + genericlist[0]);

        }
4

7 回答 7

16

我怀疑无意中添加了多个 DoWork事件。

也就是说，每次调用该start方法时，它都会注册一个新 DoWork的事件处理程序。这会添加而不是替换现有的处理程序处理DoWork程序。所以会有多个DoWork处理程序被称为后续时间.. 1、2、3等。

// creates a NEW delegate and adds a NEW handler
bw.DoWork += (obj, e) => bw_DoWork(str, strb);

我建议不要在这里使用闭包，而只是使用方法组（隐式转换为委托），然后将数据传递给RunWorkerAsync调用（有一个表单接受数据的参数）。

该RunWorkerCompleted +=行不存在此问题，因为它传递了来自方法组的委托（保证始终评估为相同的委托对象1）。因此，对该行的重复+=调用将替换处理程序。

例子：

class MyData {
   public string StrA { get; set; }
}

// These only need to be setup once (and should be for clarity).
// However it will be "ok" now if they are called multiple times
// as, since the delegates are the same, the += will
// act as a replacement (as it replaces the previous delegate with itself).
bw.WorkerSupportsCancellation = true;
bw.DoWork += bw_DoWork;
bw.RunWorkerCompleted += bw_RunWorkerCompleted;

// Pass data via argument
bw.RunWorkerAsync(new MyData {
    StrA = str,
});

void bw_DoWork (object sender, DoWorkEventArgs e) {
    var data = (MyData)e.Argument;
    var str = data.StrA;
    // stuff
}

1我不确定它是否保证引用等于相等，但使用这种方法可以稳定地调用方法组+=的-=委托，即使是通过new DelegateType(MethodGroup).

写。我在主帖中的评论：如果 UI 元素是从未创建它们的线程访问的，那么将会有有趣的“跨线程操作异常”。我相信消息框的这种用法是“好的”（当不是用另一个线程的所有者创建时），但是在 BackgroundWorker 的 DoWork 中访问 UI 的做法通常是可疑的。

另外，不要在这里打电话bw.Dispose()；将其与拥有的容器或上下文一起处理。在这种情况下，它似乎是好的和良性的，但只有在该 BGW 实例将不再被使用时才这样做。从事件处理程序调用它也是可疑的，因为 BGW 仍然是“活动的”。

于 2012-09-01T21:09:16.260 回答
15

我遇到了与上述评论者“Power-Mosfet”相同的问题

最后，添加一个new BackgroundWorker()然后分配给全局 bw 值将解决我的问题。

代码是，从改变：

private BackgroundWorker gBgwDownload;

private void yourFunction_bw(xxx)
{
    // Create a background thread
    gBgwDownload.DoWork += bgwDownload_DoWork;
    gBgwDownload.RunWorkerCompleted += bgwDownload_RunWorkerCompleted;
    //omited some code
    gBgwDownload.RunWorkerAsync(paraObj);
}

至：

private BackgroundWorker gBgwDownload;

private void yourFunction_bw(xxx)
{
    // Create a background thread
    gBgwDownload = new BackgroundWorker(); /* added this line will fix problem */
    gBgwDownload.DoWork += bgwDownload_DoWork;
    gBgwDownload.RunWorkerCompleted += bgwDownload_RunWorkerCompleted;
    //omited some code
    gBgwDownload.RunWorkerAsync(paraObj);

}
于 2013-03-22T08:15:19.713 回答
5

还有另一个原因。DoWorkEventHandler在其生成的代码中查找InitializeComponent()是否通过组件 UI 属性生成它并自己注册。

因为如果您再次注册它，它不会覆盖前一个，而是会添加另一个事件并调用两次。

于 2014-01-10T03:11:37.827 回答
3

就我而言，BackgroundWorker 运行了两次，因为在我的表单的构造函数类中，我声明了 DoWork、ProgressChanged 和 RunWorkerCompleted 事件处理程序，但 Visual Studio 2013 已经在此表单类的设计器部分声明了它。

所以，我刚刚删除了我的声明，它工作得很好。

于 2015-05-27T15:15:22.913 回答
0

我今天遇到了这个问题，当我注意到每次我显示表单时，后台工作人员 RunWorkerCompleted 事件被多次调用时，我将一个后台工作人员放在一个正在执行长时间运行任务的弹出表单上。

我的问题是我在关闭表单后没有处理它，这意味着每次我显示表单时，它每次都会添加另一个处理程序。

完成后处理表格解决了我的问题。只是想在这里提一下，因为我在为我的情况寻找解决方案时遇到了这个页面。

于 2013-06-22T15:44:41.253 回答
0

谢谢……这段代码运行良好……为 backroundworker 创建新实例是个好主意……现在我们可以在 for/while 循环中调用这个函数，并且可以运行多个 backgroundworker 进程。

当按钮单击完成时，我这样编码.. 不分散主线程流......多个进程将在后面运行......我只是使用消息框弹出......但我们可以做计时过程来运行“ bgwDownload_DoWork" 函数...并且将创建多个进程...并且我们不需要检查BackgroundWorker是否忙...

private void button1_Click(object sender, EventArgs e)
{
   for (int i = 0; i < 3; i++)
     yourFunction_bw(i);

 }
private BackgroundWorker gBgwDownload;

private void yourFunction_bw(int i)
{
    // Create a background thread
    gBgwDownload = new BackgroundWorker(); // added this line will fix problem 
    gBgwDownload.DoWork += bgwDownload_DoWork;
    gBgwDownload.RunWorkerAsync(i);

}

private void bgwDownload_DoWork(object sender, DoWorkEventArgs e)
{
  int stre = (int)e.Argument;
  MessageBox.Show(stre.ToString ()); // time taken process can be added here
}
于 2013-04-18T06:43:07.473 回答
0

我从设计器中删除了控件并在代码中实例化了一个新的 WorkerProcess：

例如： var bwProcess = new BackgroundWorker();

bwProcess.DoWork += 新 DoWorkEventHandler(bwProcess_DoWork);

bwProcess.RunWorkerCompleted += bwProcess_RunWorkerCompleted;

于 2018-10-31T08:48:58.247 回答