无论何时选择ComboBox的值，如何触发按钮命令

Question

我有一个组合框和一个按钮。我希望每当我从中选择一项时，该按钮都不会被禁用

XAML：

<Border Style="{StaticResource borderMain}"
                Grid.Row="7"
                Grid.Column="0">
            <ComboBox   ItemsSource="{Binding Source={StaticResource portNames}}"
                        SelectedItem="{Binding SelectedPort, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"
                        x:Name="Port_Selector" Grid.Column="0"
                        Text="Port Selector" Background="White"/>
        </Border>

        <Border Style="{StaticResource borderMain}"
                Grid.Column="1"
                Grid.Row="7">
            <Button Content="Connect"
                    Command="{Binding OpenPortCommand}"
                    CommandParameter="{Binding SelectedPort}"
                    Style="{StaticResource buttonMain}"
                    Margin="5"/>
        </Border>

命令：

public class OpenPortCommand : ICommand
    {
        public OpenPortVM OpenPortVM{ get; set; }

        public OpenPortCommand(OpenPortVM OpenPortVM)
        {
            this.OpenPortVM = OpenPortVM;
        }

        public event EventHandler? CanExecuteChanged
        {
            add { CommandManager.RequerySuggested += value; }
            remove { CommandManager.RequerySuggested -= value; }
        }

        public bool CanExecute(object? parameter)
        {
            string? portCom = parameter as string;

            if (!string.IsNullOrEmpty(portCom))
                return true;
            return false;
        }

        public void Execute(object? parameter)
        {
            OpenPortVM.ConnectPort();
        }
    }

我已经对其进行了调试，并检查了我用于绑定SelectedPort 的变量的值，它上面有一个值，但不知何故，我的按钮的CommandParameter 未检测到，因此CanExecute 方法运行不正常。我错过了什么吗？

更新

预期：

结果：

更新 2

Setter 上的断点：

CanExecute 上的断点

Answer 1

您的代码有两个问题：

1. 实际上阻止您的代码正确执行的最关键的一个是您引发INotifyPropertyChanged.PropertyChanged 事件的方式。您当前不是传入属性名称，而是传入支持字段的名称。

代替

OnPropertyChanged(nameof(portSelected));

应该是

OnPropertyChanged(nameof(SelectedPort));

为简化事件调用，您的调用者应使用CallerMemberName 属性：

protected virtual void OnPropertyChanged([CallerMemberName] string propertyName = "")
  => this.PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName);

然后不带任何参数调用方法OnPropertyChanged，因为现在会自动检测到属性名称：

OnPropertyChanged();

1. 方法RaiseExecuteChangedCommand 是多余的。您对CanExecuteChanged 事件的ICommand 实现将事件处理程序委托给CommandManager.RequerySuggested 事件（这是正确的）。 CommandManager.RequerySuggested 事件由 WPF 框架自动引发，并且由于事件委托，框架还将引发 ICommand.CanExecuteChanged（因为您使用 CommandManager.RequerySuggested += value 将所有 ICommand.CanExecuteChanged 处理程序附加到 CommandManager.RequerySuggested 事件）。如果您不希望框架为您引发 CanExecuteChanged 事件，则必须删除 CommandManager.RequerySuggested += value 部分，即不要将客户端处理程序附加到 CommandManager.RequerySuggested 事件。因为现在您将从RaiseExecuteChangedCommand 方法中明确提出它。所以你不能两者兼得。要么明确提出 ICommand.CanExecuteChanged，要么让 CommandManager 为你做这件事。

Answer 2

您的OpenPortCommand 缺少实际引发CanExecuteChanged 事件的可能性。我建议如下：

public class OpenPortCommand : ICommand
{
    public OpenPortVM OpenPortVM{ get; set; }

    public OpenPortCommand(OpenPortVM OpenPortVM)
    {
        this.OpenPortVM = OpenPortVM;
    }

    // Leave this event as is, don't do an explicit implementation        
    public event EventHandler? CanExecuteChanged;

    public void RaiseCanExecuteChange() => CanExecuteChanged?.Invoke(this, EventArgs.Empty);
    
    public bool CanExecute(object? parameter) => !string.IsNullOrEmpty(parameter as string);

    // rest ommitted
}

在 ViewModel 的 SelectedPort 属性中：

public string SelectedPort
{
    get => _selectedPort;
    set
    {
        _selectedPort = value;
        // call your OnPropertyChanged( ... );
        OpenPortCommand?.RaiseCanExecuteChanged();
    }
}

在 XAML 中，我简化了 ComboBox.SelectedItem 属性的绑定，如下所示：

<ComboBox ... SelectedItem="{Binding SelectedPort}"/>