Backgroundworker progresschanged事件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;
    }

我想在userstate中添加listbox2，同时想在另一种形式创建的弹出窗口中显示。我已经评论了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语句的块中......这样你以后可以在逻辑中添加内容而不会陷入这种微妙的bug。