MvvmLight：从 RelayCommand 执行中设置属性时视图未更新

Question

由于在开发人员的 GitHub 上没有任何回应，我将在这里重复我的问题。 我希望有人能以某种方式帮助我。

这是我第一次使用 MvvmLight，所以我希望我没有忽略一些明显的东西。

在我的 WPF ViewModel 我有类似的东西：

  private ICommand readFileCommand;
  public ICommand ReadFileCommand => readFileCommand ?? (readFileCommand = new RelayCommand(ReadFile));

  private void ReadFile()
  {
        FileMessage = "Message.";
  }

  private string fileMessage;
  public string FileMessage
  {
       get { return fileMessage; }
       set
       {
           //Set(ref fileMessage, value);

           fileMessage = value;
           RaisePropertyChanged();
       }
   }

我有几个问题。

• 主要问题是在 ReadFile() 等方法中设置 FileMessage 等属性不会导致在 ReadFile 完成之前更新视图。
• 使用当时成功的 RaisePropertyChanged() 和使用什么都不做的 Set() 是有区别的。尽管后者确实在这种方法之外起作用。
• 问题延伸到其他元素，例如 DataView 上的 DataGrid。

想知道被调用的方法是否应该是异步的，但这似乎不合逻辑。我还没有尝试过，因为这并不真正符合我想要实现的目标。

那么发生了什么？我忽略了什么吗？这是框架的限制吗？或者这是一个错误？

谢谢！

Answer 1

主要问题是在 ReadFile() 等方法中设置 FileMessage 等属性不会导致在 ReadFile 完成之前更新视图。

这是有道理的，因为您不能同时在同一线程上执行 ReadFile 方法和更新 UI。这与 MvvmLight 或命令无关。

如果您在运行任何可能长时间运行的代码之前设置该属性，无论是在后台线程上异步还是同步，它都应该按预期工作。

试试这个例子：

private async void ReadFile()
{
    FileMessage = "Message.";
    await Task.Delay(5000); //simulate...
    FileMessage = "Done!";
}

或者这个：

private async void ReadFile()
{
    FileMessage = "Message.";
    await Task.Run(() => Thread.Sleep(5000));
    FileMessage = "Done!";
}

Answer 2

我通过以下方式“解决”了这个问题，这对我来说是我最终想要的，但仍然给我留下了疑问。

代码是一个完整的例子来说明我的观点。 一句话。使用带中断的调试器可能会在行为上产生很大的误导。

观察

• “运行”消息起作用。
• 但是，不会显示所有其他消息。这让我目瞪口呆。

ReadFile 方法是同步的，并且直接在 UI 线程上（我检查了名称），并且对任务之外的线程没有任何作用。 那么为什么它不能显示任何东西呢？我什至多余地使用了 Invoke，这没有帮助。

所以这仍然让我怀疑 RelayCommand 是否正常工作。

我什至考虑过改用 Windows 社区工具包，但这似乎太大了一步。

无论如何，它可能会就此结束，因为 MVVVM Light 已经被 Windows Template Studio 放弃，并且开发在 2018 年 12 月停止！

如果我忽略了任何内容，仍然感谢您的澄清。

using GalaSoft.MvvmLight;
using Application.Contracts.ViewModels;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Threading;

namespace ViewModels
{
    public class ViewModel : ViewModelBase, INavigationAware
    {
        public ViewModel()
        {
            uiDispatcher = Dispatcher.CurrentDispatcher;
        }

        private Dispatcher uiDispatcher;

        public async void OnNavigatedTo(object parameter)
        {
            uiDispatcher.Thread.Name = "OnNavigatedTo";
        }

        public void OnNavigatedFrom()
        { }

        private string fileMessage = "No file yet";
        public string FileMessage
        {
            get { return fileMessage; }
            set
            {
                // Using this simple way instead of Set, which did not work.
                fileMessage = value;
                RaisePropertyChanged();
            }
        }

        private void ReadFile()
        {
            FileMessage = "ReadFile 1.";

            Thread.Sleep(1000);

            uiDispatcher.Invoke(() => FileMessage = "ReadFile Invoke 1.", DispatcherPriority.Send);

            Thread.Sleep(1000);

            // Use Run on a non UI-thread and Dispatcher to enable intermediate updates back on the UI-thread.
            Task.Run(() =>
            {
                uiDispatcher.Invoke(() => FileMessage = "Run 1.", DispatcherPriority.Send);

                Thread.Sleep(1000);

                uiDispatcher.Invoke(() => FileMessage = "Run 2.", DispatcherPriority.Send);

                Thread.Sleep(1000);
            });

            Thread.Sleep(1000);

            FileMessage = "ReadFile 2.";

            Thread.Sleep(1000);

            uiDispatcher.Invoke(() => FileMessage = "ReadFile Invoke 2.", DispatcherPriority.Send);
        }
    }
}

Answer 3

尝试稍微修改的实现：

   private async void ReadFile()
    {
        FileMessage = "ReadFile 1.";

        await Task.Delay(1000);

        await uiDispatcher.BeginInvoke(() => FileMessage = "ReadFile Invoke 1.", DispatcherPriority.Send);

        await Task.Delay(1000);

        // Use Run on a non UI-thread and Dispatcher to enable intermediate updates back on the UI-thread.
        await Task.Run(async () =>
        {
            uiDispatcher.Invoke(() => FileMessage = "Run 1.", DispatcherPriority.Send);

            await Task.Delay(1000);

            uiDispatcher.Invoke(() => FileMessage = "Run 2.", DispatcherPriority.Send);

            await Task.Delay(1000);
        });

        await Task.Delay(1000);

        FileMessage = "ReadFile 2.";

        await Task.Delay(1000);

        await uiDispatcher.BeginInvoke(() => FileMessage = "ReadFile Invoke 2.", DispatcherPriority.Send);
    }
}

总的来说，这段代码很荒谬。
但我没有做任何大的改变来保持连续性。

P.S.代码是在帖子编辑器中编写的。
抱歉 - 可能会出现小错误。