c# - 修复 "A method was called at an unexpected time"错误的最佳方法是什么

Question

首先，我已经看到了this question ，并且我(有点)理解为什么会出现此异常，我想知道修复它的最佳方法是什么。我的代码看起来有点像这样(这是一个 WinRT 应用程序):

//Here is my App constructor:
public App()
{
    this.InitializeComponent();
    this.Suspending += this.OnSuspending;

    //Initializing the model
    _model = new Model();
    _model.LoadData();
}

//the LoadData method looks like this:
public async void LoadData()
{
    StorageFolder folder = Windows.ApplicationModel.Package.Current.InstalledLocation;
    StorageFile file = await folder.GetFileAsync(@"Assets\Data.json");
    string data = await FileIO.ReadTextAsync(file);

    var dataList = JsonConvert.DeserializeObject<List<MyDataClass>>(data);
    // From time to time (pretty rarely, honestly) this line causes the
    // "A method was called at an unexpected time" thing:
    var dispatcher = CoreApplication.MainView.CoreWindow.Dispatcher;
    foreach (var item in dataList)
    {
        //do some stuff
        //<...>
        await dispatcher.RunAsync(Windows.UI.Core.CoreDispatcherPriority.Normal,
        () => {
            //do some stuff in the UI thread
        });                
    }
}

显然，使用 LoadData 方法 async void 并不是最好的解决方案。但是，如您所见，我必须在其中执行一些异步函数调用(以从文件中读取数据)。现在我能想到两种可能的解决方案:

1. 将 LoadData 更改为 public async Task LoadData() 并将其在应用程序构造函数中的调用更改为 _model.LoadData().GetAwaiter().GetResult( ); 为了同步运行它;
2. 将 LoadData 更改为 public void LoadData()，并将其中的所有 await 调用更改为使用等待程序，例如StorageFile file = folder.GetFileAsync(@"Assets\Data.json").GetAwaiter().GetResult()。

其中哪一个是更好的解决方案，或者更好的是，是否有任何其他正确的方法在应用程序启动时运行异步代码？另外，为什么调度程序行会出现“在意外时间调用方法”错误？

Answer 1

Which of these is a better solution

这两种预期解决方案都阻塞 - 具体来说，它们阻塞 I/O。让我们考虑一下阻塞，但这次不是应用程序的“我需要这些数据才能显示我想要的内容”的角度，而是从运行时的角度考虑。

运行时绝对绝对不应该阻塞 UI。阻止 UI 会阻止用户，这简直是一种 Not Acceptable 用户体验。这就是为什么整个移动世界都是异步优先的；这让许多桌面开发人员感到震惊。桌面应用应该是异步优先的，但移动应用必须是异步优先的。

因此，从运行时的角度来看，当它启动应用程序时，应用程序必须显示一些内容。立即地。现在。 I/O 不是一个选项。它不一定是完美的应用程序主屏幕，但应用程序必须同步显示某些内容。

这就是阻止不起作用的原因。自动应用商店代码分析应该拒绝这两种阻止方法。

is there any other proper way to run async code on the application startup?

是的，但应用程序必须妥协。在向用户显示漂亮、完整的第一个屏幕之前，它不能在执行 I/O 时阻塞 UI。一些桌面应用程序可以逃脱这一点(即使它们也不应该)，但这不是移动应用程序的选择。

相反，应用程序应该(同步)加载并显示“正在加载...”状态。这可以是初始屏幕，或者只是带有“正在加载...”消息或代替数据的微调器的实际主屏幕。这满足了运行时的要求。当然，在返回之前，应用还应该启动异步操作来检索数据，然后更新 UI 以显示它真正想要显示的内容(常规 View 完成数据)。

这可以像使用异步事件一样简单(正如@marcinax 评论的那样):

protected override async void OnLaunched(...)
{
  ... // Initialize UI to "Loading" state.

  _model = new Model();
  await _model.LoadData();

  ... // Update UI with data in _model.
}

就个人而言，我更喜欢将所有 UI 代码保留在 UI 层(不是 Model 的一部分)，因此如果您有任何显式(非数据绑定(bind))UI 初始化与数据，它应该放在 //Update UI 之后。

但是，如果您发现自己在应用程序中多次执行此操作(例如，每个窗口/页面都需要加载数据)，那么您可能更喜欢 asynchronous data-binding solution。就像我在 MSDN 文章中描述的那样。

Also, why is the "A method was called at an unexpected time" error happening on the dispatcher line?

我不完全确定，但我怀疑调度程序尚未运行。

将 UI 代码保留在模型/VM 层之外的一个非常好的副作用是不再需要显式 CoreDispatcher 代码。总有更好的解决方案。