MVVM c#如何将异步数据加载到属性中？

Question

我想知道是否有更好的方法将异步数据加载到属性中。现在我创建一个异步函数并在属性的 Get 部分引发一个任务，如下所示：

private ObservableCollection<CProyecto> prope;

public ObservableCollection<CProyecto> Prope
{
    get 
    {
        if (prope == null)
        {
            Task.Run(()=> LoadData()).Wait();
        }

        return proyectos;
    }
    set 
    { 
        prope = value; 
        RaisePropertyChanged(); 
    }
}

async private Task LoadData() 
{
    Prope = await clsStaticClassDataLoader.GetDataFromWebService();
}

这种方法有效，但我不喜欢使用 .Wait，因为如果服务没有快速响应，它会冻结屏幕。

你能在这件事上指导我吗？

提前致谢

Answer 1

我建议您阅读我在 async MVVM data-binding 上的 MSDN 文章。我有a library (github link)，它提供了一个NotifyTask<T> type，可以这样使用：

public class MyClass : INotifyPropertyChanged
{
  public NotifyTask<ObservableCollection<CProyecto>> Prope { get; private set; }

  public MyClass()
  {
    // Synchronously *start* the operation.
    Prope = NotifyTask.Create(LoadDataAsync());
  }

  async private Task<ObservableCollection<CProyecto>> LoadDataAsync()
  {
    return await clsStaticClassDataLoader.GetDataFromWebService();
  }
}

然后您的数据绑定将在Prope.Result 上运行。

这种方法的优点是您还可以使用数据绑定来隐藏/显示繁忙指示器 (Prope.IsNotCompleted)、在数据可用时显示控件 (Prope.IsSuccessfullyCompleted) 和错误通知 (Prope.IsFaulted / @987654334 @)。

此外，如果您愿意，您可以指定一个非null 默认值：

Prope = NotifyTask.Create(LoadDataAsync(), new ObservableCollection<CProyecto>());

Answer 2

我处理这个的方式是在构造对象时开始加载属性的过程，但我没有等待结果。由于该属性会在填充时发出通知，因此绑定工作得很好。本质上它是这样工作的：

public class MyClass : INotifyPropertyChanged
{
    private ObservableCollection<CProyecto> prope;

    public ObservableCollection<CProyecto> Prope
    {
        get { return prope; }
        set { prope = value; RaisePropertyChanged(nameof(Prope)); }
    }

    public MyClass()
    {
        // Don't wait or await.  When it's ready
        // the UI will get notified.
        LoadData();
    }

    async private Task LoadData() 
    {
        Prope = await clsStaticClassDataLoader.GetDataFromWebService();
    }
}

这很好用，不会导致 UI 出现任何延迟或卡顿。如果您希望集合永远不会是 null（IMO 的良好做法），您可以使用空集合预初始化 prope 字段。

Answer 3

您当前对 Prope 属性的实现没有多大意义。在后台线程上执行LoadData 方法是没有意义的，因为当您调用Wait() 时无论如何都会阻塞主线程。你不妨直接在LoadData()方法返回的任务上调用Wait()：

//BAD IMPLEMENTATION!
private ObservableCollection<CProyecto> prope;
public ObservableCollection<CProyecto> Prope
{
    get
    {
        if (prope == null)
            LoadData().Wait();
        return proyectos;
    }
    set { prope = value; RaisePropertyChanged(); }
}

上述实现仍然是一个糟糕的实现。属性的 getter 不应该执行异步操作。你应该阅读@Stephen Cleary 关于这个主题的博文：https://blog.stephencleary.com/2013/01/async-oop-3-properties.html

...在他的AsyncEx 库中查看他的NotifyTaskCompletion 类型：https://github.com/StephenCleary/AsyncEx