C#多线程通信：子线程向主线程的事件无法持续触发问题求助

嘿，我帮你捋清楚问题根源，再给你修正后的代码～你的进度条一直没动静直到最后跳100%，其实是踩了两个小坑：

1. 跨线程操作UI控件的线程安全问题 #

WinForms的控件都是“认线程”的——只能在创建它们的主线程（也就是UI线程）里修改状态。你的子线程通过事件回调直接更新pb.Value，本质还是在子线程里操作UI，这不仅会让更新不实时，搞不好还会偷偷抛出InvalidOperationException（只是你可能没察觉到）。

解决办法很简单：在更新控件前，先检查是否需要跨线程调用，用Control.Invoke把更新操作切回主线程执行。

2. 整数除法导致进度计算一直为0 #

你代码里的(ef/cd)*100是整数除法：因为ef和cd都是int类型，只要ef还没等于cd，ef/cd的结果就永远是0，乘100还是0，所以进度条一直显示0，直到最后一步才跳到100%。

这个坑也好填：把其中一个数转成浮点数再计算，比如(double)ef / cd * 100，这样就能得到0到100之间的连续进度值了。

修正后的完整代码 #

using System;
using System.Drawing;
using System.Threading;
using System.Windows.Forms;

public delegate void MyHandler<MyEventArgs>(object sender, MyEventArgs e);

public class thr : Form
{
    Button b1;
    ProgressBar pb;

    [STAThread]
    static void Main()
    {
        Application.EnableVisualStyles();
        Application.Run(new thr());
    }

    public thr()
    {
        b1 = new Button();
        b1.Text = "Open";
        pb = new ProgressBar();
        pb.Location = new Point(pb.Location.X, b1.Location.Y + b1.Size.Height + 5);
        pb.Step = 1;
        pb.Style = ProgressBarStyle.Continuous;
        pb.Maximum = 100; // 显式设置最大值为100，对应百分比进度
        this.Controls.Add(b1);
        this.Controls.Add(pb);
        b1.Click += (s, e) => { this.M_work(); };
    }

    private void M_work()
    {
        dhr obb = new dhr();
        // 修改事件回调，加入线程安全的UI更新逻辑
        obb.Ev1 += (s, e) => 
        {
            // 检查是否需要跨线程调用
            if (pb.InvokeRequired)
            {
                pb.Invoke(new Action(() => pb.Value = e.PbVal));
            }
            else
            {
                pb.Value = e.PbVal;
            }
        };
        ThreadStart tst = new ThreadStart(obb.prform);
        Thread thh = new Thread(tst);
        thh.Start();
        
        // 注意哦：这里直接拿obb.Txtt会拿到初始值，因为子线程还没跑完
        // 如果要在子线程结束后更新窗口标题，最好加个完成事件通知主线程，别直接阻塞UI
    }
}

class dhr
{
    public string Txtt;
    public event MyHandler<MyEventArgs> Ev1;

    public void prform()
    {
        int ab = 1;
        int cd = Int32.MaxValue - 3;
        int ef = 1;
        while (ef != cd)
        {
            if (ab % 2 == 0)
            {
                ab = 0;
                ef++;
            }
            ab++;
            // 修正进度计算：转成浮点数避免整数除法陷阱
            int progress = (int)Math.Round((double)ef / cd * 100);
            // 空值判断简化写法，替代原来的if(Ev1 != null)
            this.Ev1?.Invoke(this, new MyEventArgs(progress));
            
            // 可选：加个1毫秒延迟，避免UI更新太频繁导致卡顿
            // Thread.Sleep(1);
        }
        this.Txtt = ef.ToString();
        // 可以在这里加个完成事件，通知主线程更新窗口标题
        // 比如：public event Action OnCompleted; 然后调用 OnCompleted?.Invoke();
    }
}

public class MyEventArgs : EventArgs
{
    public int PbVal;
    public MyEventArgs(int pbval)
    {
        this.PbVal = pbval;
    }
}

额外小建议：用BackgroundWorker更省心 #

你提到了解BackgroundWorker，确实它已经封装了线程安全的进度更新逻辑，用起来更简洁，不用自己处理线程切换：

private void M_work()
{
    var worker = new BackgroundWorker();
    worker.WorkerReportsProgress = true; // 必须开启这个才能报告进度
    worker.DoWork += (s, e) =>
    {
        int ab = 1;
        int cd = Int32.MaxValue - 3;
        int ef = 1;
        while (ef != cd)
        {
            if (ab % 2 == 0)
            {
                ab = 0;
                ef++;
            }
            ab++;
            int progress = (int)Math.Round((double)ef / cd * 100);
            worker.ReportProgress(progress); // 报告进度，自动在UI线程触发ProgressChanged
        }
        e.Result = ef.ToString(); // 把结果传给完成事件
    };
    worker.ProgressChanged += (s, e) => pb.Value = e.ProgressPercentage; // 直接更新UI，线程安全
    worker.RunWorkerCompleted += (s, e) => this.Text = e.Result.ToString(); // 线程结束后更新标题
    worker.RunWorkerAsync(); // 启动后台线程
}

这样连事件和委托都不用自己定义了，BackgroundWorker帮你搞定了所有线程安全的细节，非常适合这种后台任务+UI进度更新的场景。

内容的提问来源于stack exchange，提问作者Ashish Sinha