后台工作者被调用两次

时间:2015-08-10 12:15:01

标签: c# .net winforms backgroundworker

我遇到了后台工作程序的问题,因此调用了两次,增加了我的长例程的执行时间,我手动创建了后台工作程序,因此没有机会在initializeComponent()方法中初始化DoWork ,任何帮助表示赞赏。

这是我的代码:

// constructor


           public TeacherScheduleForm(Therapist therapist)
            {
                this.therapist = therapist;

                InitializeComponent();

                bw = new BackgroundWorker();
                bw.WorkerSupportsCancellation = true;
                bw.WorkerReportsProgress = true;
                bw.DoWork += bw_DoWork;
                bw.ProgressChanged += bw_ProgressChanged;
                bw.RunWorkerCompleted += bw_RunWorkerCompleted;
                load = new LoadingForm();
    }
 private void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {
            load.AppendProgress(e.ProgressPercentage);
           // load.AppendText(e.ProgressPercentage.ToString() + "%");
            Console.Write("Progress: " + e.ProgressPercentage);
           // MessageBox.Show("Progress : " + e.ProgressPercentage);

        }

        private void bw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            if ((e.Cancelled == true))
            {
                MessageBox.Show("Cancelled");

            }

            else if (!(e.Error == null))
            {
                MessageBox.Show("Error : " + e.Error);

            }

            else
            {
                updateUI();
                load.Close();
                Console.Write( "Done!");
            }
        }


        // do work of background worker
        private void bw_DoWork(object sender, DoWorkEventArgs e)
        {
            BackgroundWorker worker = sender as BackgroundWorker;

            for (int i = 1; (i <= 2); i++)
            {
                if ((worker.CancellationPending == true))
                {
                    e.Cancel = true;
                    break;
                }
                else
                {
                    // Perform a time consuming operation and report progress.
                    Console.Write("Before Doing work");

                    setup(therapist.therapistID + "", schoolYear);// the time consuming operation
                    Console.Write("Doing work");
                    //System.Threading.Thread.Sleep(100);
                    worker.ReportProgress((i*5));
                }
            }
        }

当用户通过以下代码中的组合框选择学年时,将调用后台工作程序:

 private void comboBoxSchoolYear_SelectedIndexChanged(object sender, EventArgs e)
    {
        //load = new LoadingForm();
        schoolYear = int.Parse(comboBoxSchoolYear.SelectedValue + "");
        try{
            if (!bw.IsBusy)
            {
                bw.RunWorkerAsync();
                load.ShowDialog();
            }
            else
            {
                bw.CancelAsync();
            }
        }
        catch(Exception ex)
        {
            Console.Write("Error : " + ex.Message);
        }
    }

1 个答案:

答案 0 :(得分:3)

您在创建事件处理程序后加载表单。这是我能想到的唯一一点。尝试先加载表单,然后创建处理程序。

原因:InitializeComponent();IndexChanged通常会启动,因为此时控件已设置其索引。我一直注意到FormLoad上的这种行为。但是,由于我在这里看不到任何其他问题,所以值得一试。

IF 这并不能解决问题,如果TeacherScheduleForm被调用两次,你也应该注意。

用于调试目的的方便:

MessageBox.Show((new StackTrace().GetFrame(0).GetMethod().Name));

将其粘贴到您的事件/方法或其他任何内容中。它会弹出一个带有方法名称的消息框,该方法名称调用当前方法。在这种情况下(来自评论),它会为两者提出2个messageBoxes TeacherScheduleForm

我已将此保存到我的代码片段中。