WPF 异步命令阻止 UI

Question

我是新人，使用 Task.Run() 以及 async 和 await 以使 UI 响应更快，所以我可能没有正确实现某些东西。

我已经阅读了 Stephen Cleary 关于使用 AsyncCommands 的精彩文章，并使用了他来自 Patterns for Asynchronous MVVM Applications: Commands 的代码作为拥有响应式 UI 的基础，但是当我运行代码时，它似乎仍然冻结（我无法移动窗口或与其他按钮交互，直到功能完全完成。

我正在尝试执行通常需要 5-10 秒才能返回的搜索。下面是创建AsyncCommand 的代码以及函数的作用。

代码：

    public ICommand SearchCommand
    {
        get
        {
            if (_SearchCommand == null)
            {
                _SearchCommand = AsyncCommand.Create(() => Search());
            }
            return _SearchCommand;
        }
    }
    private async Task Search()
    {
        IEnumerable<PIPoint> points = await SearchAsync(_CurrentPIServer, NameSearch, PointSourceSearch).ConfigureAwait(false);
        SearchResults.Clear();
        foreach (PIPoint point in points)
        {
            SearchResults.Add(point.Name);
        }
    }
    private async Task<IEnumerable<PIPoint>> SearchAsync(string Server, string NameSearch, string PointSourceSearch)
    {
        {
            PIServers KnownServers = new PIServers();
            PIServer server = KnownServers[Server];
            server.Connect();
            return await Task.Run<IEnumerable<PIPoint>>(()=>PIPoint.FindPIPoints(server, NameSearch, PointSourceSearch)).ConfigureAwait(false);
        }
    }

我认为问题在于我如何将长时间运行的函数推送到线程上，并且它没有脱离 UI 线程，或者我对 Tasks 和 async/await 如何完全关闭的理解。

编辑 1： 按照斯蒂芬的回答，我更新了功能，但我没有看到 UI 响应能力有任何变化。我创建了第二个命令来执行相同的操作，并且在任何一种情况下我都会从 UI 中得到相同的响应。现在的代码如下所示

代码：

    public ICommand SearchCommand
    {
        get
        {
            if (_SearchCommand == null)
            {
                _SearchCommand = AsyncCommand.Create(async () =>
                {
                    var results = await Task.Run(()=>Search(_CurrentPIServer, NameSearch, PointSourceSearch));
                    SearchResults = new ObservableCollection<string>(results.Select(x => x.Name));
                });
            }
            return _SearchCommand;
        }
    }
    public ICommand SearchCommand2
    {
        get
        {
            if (_SearchCommand2 == null)
            {
                _SearchCommand2 = new RelayCommand(() =>
                {
                    var results = Search(_CurrentPIServer, NameSearch, PointSourceSearch);
                    SearchResults = new ObservableCollection<string>(results.Select(x => x.Name));
                }
                ,()=> true);
            }
            return _SearchCommand2;
        }
    }
    private IEnumerable<PIPoint> Search(string Server, string NameSearch, string PointSourceSearch)
    {
        PIServers KnownServers = new PIServers();
        PIServer server = KnownServers[Server];
        server.Connect();
        return PIPoint.FindPIPoints(server, NameSearch, PointSourceSearch);
    }

我一定错过了什么，但我现在不确定是什么。

编辑 2： 在对花费了这么长时间的原因进行了更多调查之后，发现结果后对列表的迭代是挂起该过程的原因。通过简单地更改 Search 函数返回的内容并让它已经在对象列表上迭代，可以让 UI 保持响应。我将斯蒂芬的回答标记为正确，因为它处理了我的主要问题，即正确地将工作移出 UI 线程，我只是没有移走实际耗时的工作。

Answer 1

我的第一个猜测是排队到Task.Run的工作相当快，延迟是由其他代码引起的（例如PIServer.Connect）。

另一件值得注意的事情是，您在Search 中使用了ConfigureAwait(false)，它更新了SearchResults——我怀疑这是错误的。如果 SearchResults 绑定到 UI，那么更新时应该在 UI 上下文中，因此不应使用 ConfigureAwait(false)。

也就是说，有一个Task.Run 原则需要牢记：将Task.Run 尽可能推到调用堆栈上。 I explain this in more detail on my blog。总体思路是Task.Run应该用于invoke同步方法；它不应该在异步方法的实现中使用（至少，不应该被重用）。

最后一点，async 本质上是功能性的。因此，返回结果比作为副作用更新集合更自然。

结合这些建议，生成的代码如下所示：

private IEnumerable<PIPoint> Search(string Server, string NameSearch, string PointSourceSearch)
{
  PIServers KnownServers = new PIServers();
  PIServer server = KnownServers[Server];
  // TODO: If "Connect" or "FindPIPoints" are naturally asynchronous,
  //  then this method should be converted back to an asynchronous method.
  server.Connect();
  return PIPoint.FindPIPoints(server, NameSearch, PointSourceSearch);
}

public ICommand SearchCommand
{
  get
  {
    if (_SearchCommand == null)
    {
      _SearchCommand = AsyncCommand.Create(async () =>
      {
        var results = await Task.Run(() =>
            Search(_CurrentPIServer, NameSearch, PointSourceSearch));
        SearchResults = new ObservableCollection<string>(
            results.Select(x => x.Name));
      });
    }
    return _SearchCommand;
  }
}