后台工作人员进度更改事件C＃

Question

我在winforms中使用后台工作者。 在backgroundworker_progresschanged事件中， e.userstate是要添加到listbox 。 同时，我想在弹出窗口中显示e.userstate 。

这是我的代码：

在backgroundworker_progresschanged事件中， setlable()是另一个类的方法。

 public void SetLable(string pbValue)
    {
        try
        {
            label1.Text = pbValue;

        }
        catch (Exception ex)
        { }
        label1.ForeColor = Color.Red;
    }

我想在listbox2添加userstate ，同时要在以其他形式创建的弹出窗口中显示该状态。 我已将listbox.items.add注释掉，因为两者不能同时工作。

private void backgroundWorker2_DoWork(object sender, DoWorkEventArgs e)
    {
        List<string> result1 = new List<string>();

        var found = obj.getFiles();

        foreach (var item in found)
        {
            if (item.Contains("ERROR"))
            {
                result1.Add(item);

                (sender as BackgroundWorker).ReportProgress(0, item);

            }
            else
                (sender as BackgroundWorker).ReportProgress(0);
            System.Threading.Thread.Sleep(500);

        }
        e.Result = result1;
    }

    private void backgroundWorker2_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        if (e.UserState != null)
            pop.SetLable(e.UserState.ToString());
       // listBox2.Items.Add(e.UserState);

    }

我希望两个人同时工作。

   pop.SetLable(e.UserState.ToString());
// listBox2.Items.Add(e.UserState);

这可能吗？

Answer 1

if语句仅执行紧随其后的语句。 如果您打算发生多件事，请使用一个块：

private void backgroundWorker2_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
    if (e.UserState != null)
    {
        pop.SetLable(e.UserState.ToString());
        listBox2.Items.Add(e.UserState.ToString());
    }
}

养成在创建每个if语句时立即在两个方括号中输入的习惯（其他部分都一样！）。 即使单个语句也应该包含在if / else语句的块中...这样，您以后就可以在逻辑中添加内容，而不会陷入这种细微的错误中。