WPF的MVVM模式下如何正确停止BackgroundWorker后台任务

核心问题原因 #

你现有代码存在3个致命错误导致取消逻辑无法生效：

• 事件注册顺序错误：先调用RunWorkerAsync()启动任务，后绑定DoWork事件，任务启动时未挂载执行逻辑
• 取消命令逻辑错误：点击取消按钮时没有直接调用CancelAsync()，反而额外注册了无用的DoWork事件，没有执行任何实际取消操作
• 取消检查时机错误：原代码在StartEngineeringOperation执行完毕后才判断取消状态，等于任务全部跑完才处理取消请求，毫无意义；后续修改的代码还访问了不存在的局部变量e，本身就存在编译问题

注意：BackgroundWorker的取消是协作式的，系统不会强制终止运行中的后台线程，必须你在业务逻辑的各个耗时节点主动检查取消状态，满足条件时提前返回才能终止任务。

修正后完整代码 #

public class EngineeringViewModel
{
    // 声明后台任务字段
    private BackgroundWorker worker;
    public ICommand StartEngineering { get; }
    public ICommand Cancel { get; }

    public EngineeringViewModel()
    {
        StartEngineering = new DelegateCommand(o =>
        {
            // 避免重复启动任务
            if (worker != null && worker.IsBusy) return;

            worker = new BackgroundWorker
            {
                WorkerReportsProgress = true,
                WorkerSupportsCancellation = true
            };
            // 先注册所有事件
            worker.ProgressChanged += Worker_ProgressChanged;
            worker.RunWorkerCompleted += worker_RunWorkerCompleted;
            worker.DoWork += (s, e) =>
            {
                // 将DoWork参数传入业务方法，用于标记取消状态
                StartEngineeringOperation(e);
            };
            // 所有配置完成后再启动任务
            worker.RunWorkerAsync();
        }, k => true);

        Cancel = new DelegateCommand(CancelEngineeringOperation);
    }

    private void StartEngineeringOperation(DoWorkEventArgs e)
    {
        startAlarmService();
        // 每个耗时步骤结束后检查取消状态
        if (worker.CancellationPending)
        {
            e.Cancel = true;
            return;
        }
        startTrendQualityCheck();
        // 若后续有其他业务步骤，每步结束后都需要加上述取消判断
    }

    private void CancelEngineeringOperation(object param)
    {
        // 直接调用CancelAsync标记取消状态即可
        if (worker != null && worker.IsBusy)
        {
            worker.CancelAsync();
        }
    }

    #region 原有逻辑保留
    private void Worker_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        // 你的原有进度更新逻辑
    }

    private void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Cancelled)
        {
            // 任务取消后的后续处理逻辑，比如弹提示、重置状态
        }
        else if (e.Error != null)
        {
            // 任务异常处理逻辑
        }
        else
        {
            // 任务正常完成逻辑
        }
    }

    private void startAlarmService()
    {
        // 原有业务逻辑，如果该方法本身耗时很长，需要在内部也分段加取消判断
    }

    private void startTrendQualityCheck()
    {
        // 原有业务逻辑
    }
    #endregion
}

补充优化建议 #

如果你的项目允许升级异步实现方式，更推荐使用Task+CancellationToken的组合替代BackgroundWorker，取消逻辑更统一，支持异步等待，代码可维护性更高。

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