Question

我有一个带有发布按钮，取消按钮，标签和进度条的表单。单击该按钮时，将复制到复制到其他目录所需的所有文件。这部分工作正常。我现在正在使用进度条让用户知道在任何给定时间发布的文件。这样，如果出现问题或文件非常大，用户就会知道导致问题的文件（通过标签）。文件通过字符串列表填充。最初，我只是使用Thread.Sleep使进程比复制文件的瞬间更长。但是，仅使用它可以防止与UI的交互（例如，如果用户想要取消）。所以我正在使用BackgroundWorker来解决我的问题。

点击“发布”

，我期望发生什么

标签告诉我“开始发表......”。在每个文件之后，进度条值递增，标签指示正在发布的文件，复制结束时，标签显示“发布完成”，进度条值最大。

什么都不会发生

标签不会更新文件。它只是说“开始发布”然后“发布完成”。

问题

如何正确“报告进度”，以便在复制每个文件后更新我的标签？

代码

   private void btnPublish_Click(object sender, EventArgs e)
        {
            try
            {                
                ProgressBar();
            }
            catch (Exception ex)
            {

                Data.Common.ErrorHandling.DisplayError(ex);
            }
        }



private void ProgressBar()
        {
            lblPublish.Text = "Starting publication...";
            //DoWork
            if (!backgroundWorker1.IsBusy)
            {
                backgroundWorker1.RunWorkerAsync();
            }
        }




 private void btnCancel_Click(object sender, EventArgs e)
    {
        if (backgroundWorker1.IsBusy)
        {
            backgroundWorker1.CancelAsync();
        }
    }


  private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            List<string> allAcros = NumOfTotalAcros();
        foreach (var acro in allAcros)
        {
            backgroundWorker1.ReportProgress(allAcros.IndexOf(acro));
            if (backgroundWorker1.CancellationPending)
            {
                e.Cancel = true;
                backgroundWorker1.ReportProgress(0);
                return;
            }
            Thread.Sleep(500);
        }

    }

    private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        List<string> acros = new List<string>();
        cdsPublishBar.Value = e.ProgressPercentage;
        for (int i = 0; i < acros.Count; i++)
        {
            lblPublish.Text = "Processing " + acros[i].ToString() + "...";
        }
    }

    private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Cancelled)
        {
            lblPublish.Text = "Processing cancelled";
        }
        else if (e.Error != null)
        {
            lblPublish.Text = e.Error.Message;
        }
        else
        {
            cdsPublishBar.Value = 100;
           lblPublish.Text = "Publish Complete";

        }
    }

Answer 1

您正在创建一个新的空集合，该集合与DoWork事件中的集合无关，然后迭代它...它将永远不会打印任何内容。

// empty list
List<string> acros = new List<string>();

// no elements to iterate over
for (int i = 0; i < acros.Count; i++)
{
    lblPublish.Text = "Processing " + acros[i].ToString() + "...";
}

相反，通过ReportProgress的第二个参数传递您需要的值：

backgroundWorker1.ReportProgress(0, acro);

然后在ProgressChanged方法中使用它：

cdsPublishBar.Value = e.ProgressPercentage;
lblPublish.Text = string.Format("Processing {0}...", e.UserState);

Winform利用进度条和文件复制

1 个答案: