WPF MVVM中ComboBox代码设置SelectedItem后命令参数为旧值

我采用MVVM模式绑定了ComboBox，XAML代码如下：

<ComboBox x:Name="CBRootPathComboBox"
  ItemsSource="{Binding RootPathComboBoxItems, Mode=OneTime}" 
  DisplayMemberPath="DisplayName"
  SelectedItem="{Binding SelectedRootPathComboBoxItem, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}">
  <i:Interaction.Triggers>
    <i:EventTrigger EventName="SelectionChanged">
      <i:InvokeCommandAction 
          Command="{Binding RootPathComboBoxItemSelectionChangedCommand}"
          CommandParameter="{Binding ElementName=CBRootPathComboBox, Path=SelectedItem}" />
    </i:EventTrigger>
  </i:Interaction.Triggers>
</ComboBox>

其中Interaction来自Microsoft.Xaml.Behaviors.Wpf NuGet包，我期望SelectionChanged触发时，对应的ICommand能获取当前选中项作为参数。

但在ViewModel中通过属性将SelectedItem从“Item2”设置为“Item1”时出现异常，调用流程如下：

1. 将属性设置为“Item1”
2. 触发INotifyPropertyChanged.PropertyChanged事件
3. .NET内部调用属性getter，返回“Item1”
4. 但ICommand收到的参数却是“Item2”，预期应为“Item1”

ViewModel中对应的属性代码：

private RootPathItem _selectedRootPathComboBoxItem;
public RootPathItem SelectedRootPathComboBoxItem
{
  get
  {
    Debug.WriteLine($"getting {_selectedRootPathComboBoxItem?.DisplayName ?? "null"}");
    return _selectedRootPathComboBoxItem;
  }
  set
  {
    Debug.WriteLine($"setting {value?.DisplayName ?? "null"}");
    if (_selectedRootPathComboBoxItem != value)
    {
      Debug.WriteLine($"overwriting {_selectedRootPathComboBoxItem?.DisplayName ?? "null"}");
      _selectedRootPathComboBoxItem = value;
      this.OnPropertyChanged();
    }
  }
}

问题原因 #

这是WPF中SelectionChanged事件触发时机与绑定更新顺序冲突导致的：当在ViewModel中设置属性时，绑定会先触发UI状态更新，但SelectionChanged事件捕获的是变更前的选中项——此时UI的SelectedItem还未完成同步，命令参数绑定自然拿不到最新值。同时，你同时使用TwoWay绑定的SelectedItem和SelectionChanged事件命令，属于逻辑冗余，进一步放大了时序问题。

解决方案 #

方案1：移除事件触发器，直接用属性变更处理逻辑

既然已经通过TwoWay绑定将选中项同步到ViewModel，完全不需要监听SelectionChanged事件。可以直接在属性setter中触发命令或处理业务逻辑：

private RootPathItem _selectedRootPathComboBoxItem;
public RootPathItem SelectedRootPathComboBoxItem
{
  get
  {
    Debug.WriteLine($"getting {_selectedRootPathComboBoxItem?.DisplayName ?? "null"}");
    return _selectedRootPathComboBoxItem;
  }
  set
  {
    Debug.WriteLine($"setting {value?.DisplayName ?? "null"}");
    if (_selectedRootPathComboBoxItem != value)
    {
      Debug.WriteLine($"overwriting {_selectedRootPathComboBoxItem?.DisplayName ?? "null"}");
      _selectedRootPathComboBoxItem = value;
      this.OnPropertyChanged();
      // 直接触发命令，参数为最新选中项
      RootPathComboBoxItemSelectionChangedCommand?.Execute(value);
    }
  }
}

同时删除XAML中的<i:Interaction.Triggers>部分，避免重复触发。

方案2：保留事件触发器，修改命令参数绑定源

如果必须使用SelectionChanged事件，不要通过ElementName绑定UI的SelectedItem，直接绑定ViewModel的属性（此时ViewModel属性已经是最新值）：

<i:InvokeCommandAction 
    Command="{Binding RootPathComboBoxItemSelectionChangedCommand}"
    CommandParameter="{Binding SelectedRootPathComboBoxItem}" />

总结 #

MVVM模式下优先依赖属性绑定而非UI事件，既能避免时序问题，也更符合模式设计原则。响应选中变更时，直接在属性变更阶段处理是最可靠的方式。

内容的提问来源于stack exchange，提问作者Dee J. Doena