WPF MVVM模式下为DataGrid中ComboBox新增元素的问题

看起来你遇到了WPF里Editable ComboBox结合DataGrid时的两个典型坑，我来帮你一步步梳理原因并给出解决方案：

问题1：SelectionChanged命令乱触发 + 输入新值不触发 #

为什么会这样？ #

• 滚动DataGrid时触发：DataGrid默认开启了虚拟化，当行滚动进入视图时，ComboBox会被重新实例化并绑定数据，这时候SelectedItem的赋值会触发SelectionChanged事件，导致你的命令被意外执行——这不是用户的实际操作，是UI虚拟化带来的“假变更”。
• 输入新值不触发：当你输入一个不在ItemsSource里的值时，SelectedItem根本不会变化（因为找不到匹配的项），所以SelectionChanged事件自然不会触发，命令也就跑不起来。

怎么修复？ #

最优雅的方式是换掉SelectionChanged事件，用更合适的事件组合来覆盖两种场景：选择已有项、输入新值：

1. 用DropDownClosed处理用户从下拉列表选择项的场景——这个事件只会在用户关闭下拉框时触发，不会被虚拟化的绑定操作触发。
2. 用LostFocus处理用户输入新值的场景——当用户离开ComboBox时触发，此时输入的内容已经确定。

修改你的XAML触发器部分：

<i:Interaction.Triggers>
    <!-- 处理选择已有项的情况 -->
    <i:EventTrigger EventName="DropDownClosed">
        <i:InvokeCommandAction Command="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type DataGrid}}, Path=DataContext.InterestOrDividendActionCommand}" 
                              CommandParameter="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type ComboBox}}, Path=Text}" />
    </i:EventTrigger>
    <!-- 处理输入新值后离开控件的情况 -->
    <i:EventTrigger EventName="LostFocus">
        <i:InvokeCommandAction Command="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type DataGrid}}, Path=DataContext.InterestOrDividendActionCommand}" 
                              CommandParameter="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type ComboBox}}, Path=Text}" />
    </i:EventTrigger>
</i:Interaction.Triggers>

如果实在不想换事件，也可以在命令里加判断过滤掉虚拟化带来的假触发：

// 命令执行方法里的判断逻辑
if (parameter is ComboBox comboBox)
{
    // 只有当控件获得焦点且下拉框没打开时，才认为是用户的有效操作
    if (comboBox.IsKeyboardFocusWithin && !comboBox.IsDropDownOpen)
    {
        var inputText = comboBox.Text;
        // 你的业务逻辑
    }
}

不过这个方案有点hack，还是推荐换事件。

问题2：事件触发时Text属性还是旧值 #

这个问题本质是SelectionChanged的触发时机问题——它在Text属性更新之前就跑了，所以你拿到的永远是上一次的值。

最佳解决方案：绑定Text到ViewModel属性 #

直接把ComboBox的Text绑定到ViewModel的一个属性，同时设置UpdateSourceTrigger=LostFocus（或者PropertyChanged，看你需求），这样命令执行时，ViewModel里的属性已经是最新值了，根本不需要从事件参数里拿：

<ComboBox ItemsSource="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type DataGrid}}, Path=DataContext.AppData.AlienTypeObjects}" 
          SelectedItem="{Binding InterestOrDividend}" 
          Text="{Binding NewInterestOrDividendValue, UpdateSourceTrigger=LostFocus}"
          DisplayMemberPath="FullName" 
          IsEditable="True">
    <i:Interaction.Triggers>
        <i:EventTrigger EventName="LostFocus">
            <i:InvokeCommandAction Command="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type DataGrid}}, Path=DataContext.InterestOrDividendAddCommand}" />
        </i:EventTrigger>
    </i:Interaction.Triggers>
</ComboBox>

然后在ViewModel里（记得实现INotifyPropertyChanged）：

public string NewInterestOrDividendValue { get; set; }

public ICommand InterestOrDividendAddCommand => new RelayCommand(() =>
{
    // 直接用NewInterestOrDividendValue，它已经是最新的输入值了
    if (!string.IsNullOrEmpty(NewInterestOrDividendValue))
    {
        // 把新值添加到AlienTypeObjects列表里的逻辑
        AppData.AlienTypeObjects.Add(new AlienType { FullName = NewInterestOrDividendValue });
    }
});

这样既解决了时机问题，也更符合MVVM的设计思路。

总结一下 #

1. 抛弃SelectionChanged，用DropDownClosed+LostFocus覆盖所有用户操作场景，解决滚动触发和新值不触发的问题。
2. 把ComboBox的Text绑定到ViewModel属性，避免依赖事件参数获取值，彻底解决旧值问题。

内容的提问来源于stack exchange，提问作者Istvan Heckl