Winform并行执行双任务时多线程更新Label控件不生效问题

根本原因 #

Winform 控件的所有操作都必须在创建它的UI线程中执行，你在子线程直接修改Label.Text会触发跨线程操作异常，由于代码中写了空的catch块吞掉了所有异常，因此看不到报错，只会表现为Label不更新。

解决方案 #

1. 修改mma方法，通过Invoke跨线程更新UI #

private void mma(List<int> parameter_list, int parameter, Label label)
{
    // 先判断是否需要跨线程调用
    if (label.InvokeRequired)
    {
        label.Invoke(new Action(() => mma(parameter_list, parameter, label)));
        return;
    }
    
    // 后续逻辑保持不变
    if (parameter_list.Count == 9) 
    { 
        parameter_list.Add(parameter); 
        parameter_list.RemoveAt(0); 
    }
    else 
    { 
        parameter_list.Add(parameter); 
    }
    var minpara = parameter_list.Min();
    var maxpara = parameter_list.Max();
    var avrgpara = parameter_list.Sum() / parameter_list.Count;

    label.Text = $"Min = {minpara}, Avg = {avrgpara}, Max = {maxpara}";
}

2. 优化原有业务代码的隐患 #

原有代码存在多个会影响稳定性和性能的问题，建议同步修改：

• 避免重复读取传感器值：你现在ReadRPM()等方法在主线程读一次赋值给文本框，子线程里又读一次，两次读取的数值可能不一致，应该先把读取到的值存到本地变量再复用
• 不要每次都新建Thread：Thread的创建销毁开销很大，25个控件每次循环都开新线程会浪费大量系统资源，改用轻量的Task.Run更合适
• 不要空吞异常：空catch块会掩盖所有运行时错误，不利于问题排查，至少要保留异常日志输出
• 保证线程安全：List<int>不是线程安全类型，如果多个线程操作同一个列表需要加锁处理

优化后的循环代码示例：

while (start)
{
    try
    {
        // 先读取值存到本地变量，避免重复读取
        int rpmVal = ReadRPM();
        RPM_TEXT.Text = rpmVal.ToString() + " rpm";
        // 用Task.Run代替手动新建Thread
        _ = Task.Run(() => mma(rpm_list, rpmVal, RPM_mma));
    }
    catch (Exception ex)
    {
        // 此处可添加日志输出异常信息，不要空吞异常
    }

    try 
    { 
        int elVal = ReadEngineLoad();
        EL_TEXT.Text = elVal.ToString() + " %";
        _ = Task.Run(() => mma(el_list, elVal, EL_mma));
    }
    catch (Exception ex)
    {
        // 日志输出异常
    }
    
    try 
    { 
        int ectVal = ReadCoolantTemp();
        ECT_TEXT.Text = ectVal.ToString() + " °C";
        _ = Task.Run(() => mma(ect_list, ectVal, ECT_mma));
    }
    catch (Exception ex)
    {
        // 日志输出异常
    }
}

额外优化建议 #

如果25个传感器的读取操作本身耗时较高，可以把读取逻辑也全部放到后台线程执行，所有数值读取完成后再批量更新UI，避免UI线程被IO操作卡住，可以进一步降低整体填充延迟。

内容的提问来源于stack exchange，提问作者Adnan abid