c# - MVVMLight CanExecute 在单击窗口之前无法正常工作

Question

快速记录，这样我就不会浪费任何人的时间。从 nuget 安装 MVVMLight 时，我最终收到错误 null : The term 'null' is not recognized as the name of a cmdlet, function, script file, or operable program。尽管如此，MVVMLight 似乎工作正常，但下面将描述的问题除外，但我想提一下以防万一。

---

问题

我遇到命令执行完成后按钮未重新启用的问题。如果执行速度非常快，它们似乎有时会起作用，但任何需要一段时间的操作似乎都不起作用。这是竞争条件的尖叫声。

我正在使用 Task 来执行冗长的操作，以便 UI 可以更新。 Task 的第一步也是最后一步是适本地翻转 IsBusy(其中每个 CanExecute 方法 返回 !IsBusy;)

我已经构建了一个简单的示例，它将使用 Thread.Sleep 来模拟缓慢的操作，它很好地展示了这个问题。 WaitOneSecondCommand 似乎间歇性地工作。 WaitTenSecondsCommand 和 WaitThirtySecondsCommand 从不工作。

不起作用 我的意思是按钮保持禁用状态，直到我点击表单上的某处。

---

我尝试过的事情

我已经进行了大量研究，到目前为止我尝试过的解决方案都没有改变这种行为。我最终导致强制执行所有不同的“修复”以防我误解。

我尝试的一件事是扩展 RelayCommands 以引发属性更改。例如:

来自:

public RelayCommand WaitOneSecondCommand {
    get; set;
}

收件人:

public RelayCommand WaitTenSecondsCommand {
    get {
        return _waitTenSecondsCommand;
    }

    set {
        _waitTenSecondsCommand = value;
        RaisePropertyChanged();
    }
}

我没想到它会起作用，但我想尝试一下。我还尝试添加 WaitTenSecondsCommand.RaiseCanExecuteChanged();

我还尝试将 WaitTenSecondsCommand.RaiseCanExecuteChanged() 添加到 CommandExecute 方法，但这也没有改变任何东西。

private void WaitTenSecondsCommandExecute() {
    Task.Run(() => {
        IsBusy = true;
        Thread.Sleep(10000);
        IsBusy = false;
        WaitTenSecondsCommand.RaiseCanExecuteChanged();
    });
}

我还阅读了有关 CommandManager 的内容，因此我也添加了 CommandManager.InvalidateRequerySuggested()。我将它添加到 IsBusy 中，认为这将是非常垃圾的，但我认为它会消除任何疑问

这又是竞争条件的味道，我在这里使用 Task，但这些任务不能同时运行，并且由于使用了 IsBusy 标志而不会相互冲突。

---

完整代码

这是一个基本的 WPF 应用程序，使用 .Net 4.6.1 和从 Nuget 安装的 MVVMLight 5.2.0。

主窗口.xaml

<Window x:Class="StackOverflowExample.View.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        mc:Ignorable="d"
        Title="MainWindow" MinHeight="100" Width="150" ResizeMode="NoResize" SizeToContent="Height"
        DataContext="{Binding Source={StaticResource Locator}, Path=Main}">
    <StackPanel>
    <Button Height="70" Margin="5" Command="{Binding WaitOneSecondCommand}">Wait 1 Second</Button>
    <Button Height="70" Margin="5" Command="{Binding WaitTenSecondsCommand}">Wait 10 Seconds</Button>
    <Button Height="70" Margin="5" Command="{Binding WaitThirtySecondsCommand}">Wait 30 Seconds</Button>
    </StackPanel>
    <Grid>
        <Label HorizontalAlignment="Left" Width="84">IsBusy:</Label>
        <TextBox IsReadOnly="True" HorizontalAlignment="Right" Width="45" Text="{Binding IsBusy}" Margin="4,5,5,5" />
    </Grid>
</Window>

我已经添加了 IsBusy 的绑定(bind)以清楚地表明它正在正确更新。这也是了解 10 秒和 30 秒命令何时完成的唯一可靠方法。我不希望任何 MessageBoxes 强制您单击“确定”导致 CanExecute 更新。这是创可贴修复。

MainViewModel.cs

public class MainViewModel : ViewModelBase {
    private bool _isBusy;
    private RelayCommand _waitTenSecondsCommand;

    public MainViewModel() {
        WaitOneSecondCommand = new RelayCommand(WaitOneSecondCommandExecute, WaitOneSecondCommandCanExecute);
        WaitTenSecondsCommand = new RelayCommand(WaitTenSecondsCommandExecute, WaitTenSecondsCommandCanExecute);
        WaitThirtySecondsCommand = new RelayCommand(WaitThirtySecondsCommandExecute, WaitThirtySecondsCommandCanExecute);
    }

    public RelayCommand WaitOneSecondCommand {
        get; set;
    }

    public RelayCommand WaitTenSecondsCommand {
        get {
            return _waitTenSecondsCommand;
        }

        set {
            _waitTenSecondsCommand = value;
            RaisePropertyChanged();
            WaitTenSecondsCommand.RaiseCanExecuteChanged();
        }
    }

    public RelayCommand WaitThirtySecondsCommand {
        get; set;
    }

    public bool IsBusy {
        get {
            return _isBusy;
        }

        set {
            _isBusy = value;
            RaisePropertyChanged();
            CommandManager.InvalidateRequerySuggested();
        }
    }

    private void WaitOneSecondCommandExecute() {
        Task.Run(() => {
            IsBusy = true;
            Thread.Sleep(1000);
            IsBusy = false;
        });
    }

    private void WaitTenSecondsCommandExecute() {
        Task.Run(() => {
            IsBusy = true;
            Thread.Sleep(10000);
            IsBusy = false;
            WaitTenSecondsCommand.RaiseCanExecuteChanged();
        });
    }

    private void WaitThirtySecondsCommandExecute() {
        Task.Run(() => {
            IsBusy = true;
            Thread.Sleep(30000);
            IsBusy = false;
        });
    }

    private bool WaitOneSecondCommandCanExecute() {
        return !IsBusy;
    }

    private bool WaitTenSecondsCommandCanExecute() {
        return !IsBusy;
    }

    private bool WaitThirtySecondsCommandCanExecute() {
        return !IsBusy;
    }
}

请注意，在 View 模型中，我只在 WaitTenSeconds 上放置了一个支持字段以表明它不会改变行为。

Answer 1

这个问题完全归功于 Viv:https://stackoverflow.com/a/18385949/865868

我面临的问题是我在单独的线程中更新 IsBusy，而主 UI 线程显然依赖于它来更新按钮。

2020 年更新!

我想重新审视这个答案，因为我的理解是有缺陷的，尽管我觉得还有更好的方法。目标应该是不修改 IsBusy 的设置，或者根本不修改任何支持属性。此外，应避免使用 async void。

相应的 CommandExecute 方法现在只使用异步等待

private async Task WaitOneSecondCommandExecute() {
    CanWaitOneSecond = true;
    await Task.Delay(1000);
    CanWaitOneSecond = false;
}

我对“CanWaitOneSecond”进行了此更改

public bool CanWaitOneSecond {
    get => _canWaitOneSecond ;
    set {
        _canWaitOneSecond  = value;
        Set(() => CanWaitOneSecond , ref _canWaitOneSecond , value);
        RaiseAllCanExecuteChanged();
     }
}

public void RaiseAllCanExecuteChanged() {
    foreach (var command in Commands) {
        command.RaiseCanExecuteChanged();
    }
}

public List<RelayCommand> Commands {
    get;
}

我还对设置 RelayCommand 以支持 async Task 而不是 async void 的构造函数进行了细微更改

Commands = new List<RelayCommand>();
Commands.Add(WaitOneSecondCommand = new RelayCommand(async () => await WaitOneSecondCommandExecute(), WaitOneSecondCommandCanExecute));

我希望这可以节省人们大量的研究时间，如果此实现有任何问题，请告诉我。我还是希望把它清理干净