后台工作人员多次运行 doWork答案

【问题标题】：Background worker runs doWork multiple times后台工作人员多次运行 doWork
【发布时间】：2019-01-27 23:24:59
【问题描述】：

private void button_uploadToPI_Click(object sender, RoutedEventArgs e)
    {


        this.newFilePath = this.textbox_input_filePath.Text;
        this.label_status.Content = "";
        if (bgWorker.IsBusy != true)
        {


            bgWorker.RunWorkerAsync();
        }
    }

这是我的点击事件。

public MainWindow()
    {

        InitializeComponent();

        this.progressBar.Minimum = 0;
        this.progressBar.Maximum = 100;

        this.bgWorker = new BackgroundWorker();

        this.bgWorker.WorkerReportsProgress = true;
        this.bgWorker.WorkerSupportsCancellation = true;
        reader.IsOpen = false;

        this.bgWorker.DoWork += bgWorker_DoWork;
        this.bgWorker.ProgressChanged += bgWorker_ProgressChanged;
        this.bgWorker.RunWorkerCompleted += bgWorker_RunWorkerCompleted;
    }

主窗口方法。

public partial class MainWindow : Window
{

    private BackgroundWorker bgWorker;

bgWorker 在这里初始化。

void bgWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {

        this.progressBar.Value = 0;
        this.label_progressPercentage.Content = "0%";
        reportTextBox.ScrollToEnd();
    }

RunWork完整代码

主要问题是，如果我运行该应用程序一次，它会按预期工作。如果我然后再次单击，它将执行该方法两次，如果我再次单击它，它将执行该方法 3 次。每次新点击都会增加。

谁能看到这里的主要问题？

【问题讨论】：

我没有看到发布的代码有任何问题。您有机会发布 BackgroundWorker 的事件处理程序吗？
我为我的 runworkcomplete 方法添加了代码。
“我再次单击它”是指单击button_uploadToPI 还是再次显示表单本身？
听起来你多次订阅按钮点击事件而不取消订阅。
在每次点击时创建一个新的 worker 实例并将其放入 bgWorker 字段中。当工作人员完成时，使该字段无效。检查该字段是否为空，而不是检查 IsBusy 属性。 ...并且根本不要使用 BackgroundWorker。

标签： c# backgroundworker

【解决方案1】：

我根据@galenus 的建议制作的样本。我与后台工作人员有相同的情况，单击执行工作按钮多次触发工作，直到我使用该按钮初始化工作人员。代码块：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

using System.ComponentModel;
using System.Data;

namespace MouseClick
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {



        public MainWindow()
        {
            InitializeComponent();
        }

        public BackgroundWorker worker = new BackgroundWorker();

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            //e.Handled = true; // Din't do anything 
            worker = new BackgroundWorker(); //stopped the dowork being executed multiple times when button pressed again in the same session
            worker.WorkerReportsProgress = true;
            worker.WorkerSupportsCancellation = true;
            worker.DoWork += worker_DoWork;
            worker.ProgressChanged += worker_ProgressChanged;
            worker.RunWorkerCompleted += worker_RunWorkerCompleted;
            worker.RunWorkerAsync();

        }

        void worker_DoWork(object sender, DoWorkEventArgs e)
        {


            int tot = 1;

            MessageBox.Show(tot.ToString()); //if this message box appears multiple times, it means the dowork is executing multiple times

            DataTable dt = new DataTable();

            dt.Columns.Add("Running Number");

            for (int i = 0; i <= 100; i++)
            {
                if (worker.CancellationPending == true)
                {
                    //http://stackoverflow.com/questions/8300799/cancel-background-worker-exception-in-e-result
                    //  e.Cancel = true; //This does the trick
                    e.Result = 100;
                    return;
                }
                worker.ReportProgress(i);
                System.Threading.Thread.Sleep(1);
                dt.Rows.Add(i);
            }
            e.Result = dt;
        }

        #region "worker_ProgressChanged"
        void worker_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {

        }
        #endregion

        #region "worker_RunWorkerCompleted"
        void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            myGrid.ItemsSource = ((DataTable)e.Result).DefaultView;
            worker.Dispose();

        }
        #endregion

    }
}

【讨论】：

这会导致内存泄漏吗？

【解决方案2】：

虽然这可能有效，但您可以在 MainWindow 的构造函数中声明并初始化 worker，之后您还可以附加事件。这样，在您的 Button_Click 方法中，您只需要运行工作程序即可。无需在每次单击按钮时创建新工作人员并附加这些事件，您可以节省一些时间；）

【讨论】：