WPF中Dispatcher调用仍触发CollectionView跨线程修改错误求助

问题根源 #

1. 跨线程读取集合：你在AddGame里的Games.Any()、Games.Where()这些读取操作，如果是在非UI线程执行，本身就违反了WPF线程规则——ObservableCollection的所有操作（读/写）都必须在创建它的UI线程执行，哪怕后续修改用了Dispatcher，前面的跨线程读取已经打乱了CollectionView的线程状态，导致报错。
2. 重复添加逻辑：AddGame中检测到当前是UI线程时，先直接执行Games.Add，之后又无条件跑一遍Dispatcher.Invoke的Add操作，不仅会重复添加游戏，还可能引发线程状态混乱。
3. 后台调用链混乱：如果HandleLaunchersAndGames是在后台线程拉取数据库数据，然后直接在后台线程调用AddGame，那整个AddGame的执行上下文都是非UI线程，线程检查那一步根本起不到作用。

修复方案 #

1. 把AddGame的所有逻辑移到UI线程执行 #

修改AddGame，让判断和添加操作全在UI线程运行，彻底避免跨线程问题：

private Task AddGame(Game gameModel)
{
    // 将整个逻辑委托到UI线程执行
    return App.Current.Dispatcher.InvokeAsync(() =>
    {
        var lGameModel = gameModel;
        // 统一在UI线程做存在性判断，避免跨线程读取
        bool gameExists = Games.Any(x => 
            x.Name == lGameModel.Name && 
            x.AId == lGameModel.AId && 
            x.LauncherName == lGameModel.LauncherName);

        if (!gameExists)
        {
            Games.Add(lGameModel);
        }
    }).Task;
}

2. 修正FindGameInBackground的线程调度 #

HandleLaunchersAndGames是拉取数据库的耗时操作，不能放到UI线程运行（会导致界面卡顿），应该丢到后台线程执行，再回到UI线程处理排序：

public async Task FindGameInBackground()
{
    // 后台线程执行耗时的数据库拉取操作
    await Task.Run(() => HandleLaunchersAndGames());
        
    // 排序和集合更新必须在UI线程执行
    await App.Current.Dispatcher.InvokeAsync(() =>
    {
        Games = new ObservableCollection<Game>(Games.OrderBy(x => x.Name));
        GamesView = CollectionViewSource.GetDefaultView(Games);
    });
    // ...其他后续逻辑
}

3. 调整HandleLaunchersAndGames的调用逻辑 #

如果HandleLaunchersAndGames里是循环拉取游戏数据，直接调用修改后的AddGame即可，不用额外处理线程——AddGame内部已经搞定UI线程调度：

private void HandleLaunchersAndGames()
{
    // 从数据库拉取游戏数据的逻辑
    var dbGames = FetchGamesFromDatabase();
    
    foreach (var game in dbGames)
    {
        // 直接调用，内部已确保在UI线程操作集合
        _ = AddGame(game);
    }
}

4. 优化加载速度：批量更新集合（可选） #

如果拉取的游戏数量多，单个Add会频繁触发UI更新导致卡顿。可以先在后台线程把数据整理好（去重），再一次性更新到ObservableCollection：

public async Task FindGameInBackground()
{
    // 后台线程拉取+去重数据
    var processedGames = await Task.Run(() =>
    {
        var rawDbGames = FetchGamesFromDatabase();
        // 后台线程去重，避免UI线程做大量计算
        return rawDbGames.DistinctBy(g => new { g.Name, g.AId, g.LauncherName }).ToList();
    });
        
    // UI线程一次性合并、排序、更新集合
    await App.Current.Dispatcher.InvokeAsync(() =>
    {
        var combined = Games.Concat(processedGames)
                            .DistinctBy(g => new { g.Name, g.AId, g.LauncherName })
                            .OrderBy(g => g.Name)
                            .ToList();
        Games = new ObservableCollection<Game>(combined);
        GamesView = CollectionViewSource.GetDefaultView(Games);
    });
}

核心提醒 #

• ObservableCollection不是线程安全的，所有读/写操作必须在UI线程执行，哪怕是简单的Any、Count这些读取操作，跨线程触碰就容易出问题。
• 耗时操作（数据库拉取、数据计算）全丢后台线程，UI线程只负责更新界面和集合。
• 批量更新比单个Add高效得多，能显著提升游戏列表的加载速度，减少界面卡顿。

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