从后台线程更新 DataContext

Question

我在这样的后台线程中为 wpf 窗口获取数据 [framework 4.0 with async/await]：

async void refresh()
{
    // returns object of type Instances
    DataContext = await Task.Factory.StartNew(() => serviceagent.GetInstances());
    var instances = DataContext as Instances;
    await Task.Factory.StartNew(() => serviceagent.GetGroups(instances));
    // * problem here * instances.Groups is filled but UI not updated
}

当我在 GetInstances 中包含 GetGroups 的操作时，UI 会显示组。
当我在单独的操作中更新时，DataContext 包括正确的组，但 UI 不显示它们。

在GetGroups() 方法中，我将NotifyCollectionChangedAction.Reset 包含在ObservableCollection 组中，但这没有帮助。
更奇怪的是，我在列表中只调用了一次NotifyCollectionChangedAction.Reset，却被执行了3次，而列表有十项？！

我可以通过写来解决这个问题：

DataContext = await Task.Factory.StartNew(() => serviceagent.GetGroups(instances));

但这是通过后台进程更新 DataContxt 和 UI 的常规方式吗？
其实我只想更新现有的DataContext而不重新设置？

编辑：serviceagent.GetGroups(instances) 更详细：

public void GetGroups(Instances instances)
{
    // web call
    instances.Admin = service.GetAdmin();

    // set groups for binding in UI
    instances.Groups = new ViewModelCollection<Groep>(instances.Admin.Groups);

    // this code has no effect
    instances.Groups.RaiseCollectionChanged();
}

这里ViewModelCollection<T>继承自ObservableCollection<T>，我添加了方法：

public void RaiseCollectionChanged()
{
    var handler = CollectionChanged;
    if (handler != null)
    {
        Trace.WriteLine("collection changed");
        var e = new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset);
        handler(this, e);
    }
}

Answer 1

您的代码的 async 部分有几点很突出：

• 我在我的 MSDN 文章中解释了 why we should avoid async void。总之，void 是 async 方法的不自然返回类型，因此它有一些怪癖，尤其是在异常处理方面。
• We should prefer TaskEx.Run over StartNew for asynchronous tasks，正如我在博客中解释的那样。
• 虽然不完全要求，但最好遵循Task-based Asynchronous Pattern 中的指南；遵循这些命名约定（等）将有助于其他开发人员维护代码。

基于这些，我也推荐我的intro to async博文。

关于实际问题...

从后台线程更新数据绑定代码总是很棘手。我建议您将 ViewModel 数据视为 UI 的一部分（可以说是“逻辑 UI”）。所以可以在后台线程上检索数据，但更新实际 VM 值应该在 UI 线程上完成。

这些更改使您的代码看起来更像这样：

async Task RefreshAsync()
{
  var instances = await TaskEx.Run(() => serviceagent.GetInstances());
  DataContext = instances;
  var groupResults = await TaskEx.Run(() => serviceagent.GetGroups(instances));
  instances.Admin = groupResults.Admin;
  instances.Groups = new ObservableCollection<Group>(groupResults.Groups);
}

public GroupsResult GetGroups(Instances instances)
{
  return new GroupsResult
  {
    Admin = service.GetAdmin(),
    Groups = Admin.Groups.ToArray(),
  };
}

接下来需要检查的是Instances 是否实现了INotifyPropertyChanged。设置Groups 时不需要引发Reset 集合更改事件；因为Groups 是Instances 上的一个属性，所以Instances 有责任提升INotifyPropertyChanged.PropertyChanged。

或者，您可以只设置DataContext last：

async Task RefreshAsync()
{
  var instances = await TaskEx.Run(() => serviceagent.GetInstances());
  var groupResults = await TaskEx.Run(() => serviceagent.GetGroups(instances));
  instances.Admin = groupResults.Admin;
  instances.Groups = new ObservableCollection<Group>(groupResults.Admin.Groups);
  DataContext = instances;
}

Answer 2

似乎对什么是 DataContext 有点困惑。 DataContext 不是您必须更新的特殊对象。它是对要绑定到 UI 的对象的引用。每当您对这些对象进行更改时，UI 都会收到通知（如果您实现了正确的接口）。

因此，除非您显式更改 DataContext，否则您的 UI 无法猜测您现在想要显示一组不同的对象。

实际上，在您的代码中，没有理由将 DataContext 设置两次。只需将其设置为您要显示的最终对象集。事实上，由于您处理相同的数据，没有理由使用两个任务：

async Task refresh()
{
    // returns object of type Instances
    DataContext=await Task.Factory.StartNew(() => {
             var instances = serviceagent.GetInstances();
             return serviceagent.GetGroups(instances);
    });
}

注意：

您不应该使用async void 签名。它仅用于即发即弃的事件处理程序，您不关心它们是成功还是失败。原因是无法等待async void 方法，因此没有人知道它是否成功。

Answer 3

我发现RaiseCollectionChanged 对DataContext 绑定的属性Groups 没有影响。我只需要通知：instances.RaisePropertyChanged("Groups");。