Prism DelegateCommand 中的 CanExecute 不起作用

Question

当我在短时间内多次单击按钮时，每次都会调用该方法并且我的应用程序崩溃（当代码尝试导航到另一个页面时）。此问题仅在 Xamarin.Android 中出现（iOS 处理双击）

public bool IsBusy { get; set; }

private DelegateCommand<string> _eventDetailsCommand;
public DelegateCommand<string> EventDetailsCommand => _eventDetailsCommand ?? (_eventDetailsCommand = new DelegateCommand<string>(EventDetails, (x) => !IsBusy));

private void EventDetails(string obj)
{
    IsBusy = true;
    await _navigationService.NavigateAsync("AnotherPage");
    IsBusy = false;
}

Xamarin.Android

棱镜：7.1.0.172（预）

PropertyChanged.Fody (2.2.6)：

Answer 1

最重要的一点是DelegateCommand.RaiseCanExecuteChanged

你应该先将CanNavigate设置为false，按钮将被禁用，在按钮命令方法代码完成后再设置为true

这是我的例子，它工作正常：

    private bool _canNavigate = true;
    public bool CanNavigate
    {
        get { return _canNavigate; }
        set { 
            SetProperty(ref _canNavigate, value);
            LoginCommand.RaiseCanExecuteChanged();
        }
    }

    private DelegateCommand _loginCommand;
    public DelegateCommand LoginCommand => _loginCommand ?? (_loginCommand = new DelegateCommand(Login, () => CanNavigate));
    public async void Login()
    {
        CanNavigate = false;
        
        ...do something here

        CanNavigate = true;
    }

Answer 2

仅设置 IsBusy 不起作用，因为当 IsBusy 更改时命令不会收到通知，UI 也不会。您必须使用 RaiseCanExecuteChanged 通知所有人。更好的方法是在创建 DelegateCommand 之后使用 ObservesCanExecute（Fluent API，但请注意只能观察到一个属性）。它会为您处理这些，并会自动调用 RaiseCanExecuteChanged。

这是我通常如何处理的示例：

public MyViewModel(): ViewModelBase
{
    private readonly INavigationService _navigationService;
    public bool IsBusy { get; set; }

    public ICommand ShowEventDetailsCommand { get; private set; }

    public  MyViewModel(INavigationService navService)
    {
        _navigationService = navService;
        ShowEventDetailsCommand = new DelegateCommand<string>(async(obj) => await ExecuteShowEventDetailsCommand(obj)).ObservesCanExecute(() => !IsBusy);
    }

    public async Task ExecuteShowEventDetailsCommand(obj)
    {
        IsBusy = true; // Note this is not thread safe, just for demonstration
        try
        {
            await _navigationService.NavigateAsync(...);
        }
        finally
        {
            IsBusy = false;
        }
    }
}

这通常是我处理这个问题的方式。但请注意，对 IsBusy 的访问不是线程安全的，因此您应该使用它。我有类似 LockActivityHandler 的东西，带有 .TryLock、.Unlock 和 .IsLocked。

if(LockActivityHandler.TryLock())
{
    try
    {
        //DoStuff
    }
    finally
    {
        LockActivityHandler.Unlock();
    }
}

IsLocked 可以绑定到 UI 元素的 Enabled 属性以禁用它们。即使那些没有被禁用并且另一个动作正在运行，由于 TryLock() => false

，新的动作也不会被执行

PS：这在文档中也有更多的例子，所以你可以看看这里：https://prismlibrary.github.io/docs/commanding.html

Answer 3

请尝试

_deviceService.BeginInvokeOnMainThread(() =>
    {
        IsBusy = true;
    });
    //await ... long running process.

在上面的代码 sn-p 中，_deviceService 是通过构造函数注入的 IDeviceService 类型，如下所示：

private IDeviceService _deviceService;

/// <summary>
/// Class constructor 
/// </summary>
/// <param name="deviceService"></param>
public MyPageViewModel(IDeviceService deviceService) {
    _deviceService = deviceService;
}

Answer 4

private void EventDetails(字符串 obj) { 忙=真； …… 忙=假； }

这会在事件处理程序执行的整个过程中阻塞 UI 线程，因此IsBusy = true; 和 IsBusy = false; 对 UI 没有任何可观察到的影响。

这是async 的教科书示例。

你应该这样写：

private async void EventDetails(string obj)
{
    IsBusy = true;

    await ....;

    IsBusy = false;
}

如果 .... 碰巧不是可等待的，请将其包装在 Task.Run 中：

private async void EventDetails(string obj)
{
    IsBusy = true;

    await Task.Run( () => .... );

    IsBusy = false;
}