WPF MVVM中ComboBox的SelectedValue为何无法通过OnPropertyChanged正确设置？

需求：选择ComboBox中除索引1（值为"2"）的任意项时，弹出提示{值} Is not available yet，并将SelectedValue重置为索引1的项。但实际运行时，提示框正常弹出，ComboBox的选中值却未被重置为目标项。

相关代码 #

ViewModel 代码 #

private static ObservableCollection<string> sourceTypeCollection = new ObservableCollection<string>() { "1", "2", "3", "4", "5", "6", "7", "8", "9" };
public ObservableCollection<string> SourceTypeCollection
{
    get => sourceTypeCollection;
    set => Set(ref sourceTypeCollection, value);
}


private string selectedValue = "2";
public string SelectedValue
{
    get => selectedValue;
    set
    {
        selectedValue = value;              
        if (selectedValue != SourceTypeCollection[1])
        {
            MessageBox.Show(value + " Is not available yet", "Source Info", MessageBoxButton.OK);
            selectedValue = SourceTypeCollection[1];                   
        }
        Set(ref selectedValue, value);
    }
}

XAML 代码 #

<ComboBox ItemsSource="{Binding SourceTypeCollection, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
          SelectedValue="{Binding SelectedValue, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
          IsSynchronizedWithCurrentItem="True"                                         
          Height="30" FontSize="18" Width="150"
          VerticalAlignment="Top" Grid.Row="1"
          HorizontalAlignment="Center">
</ComboBox>

ViewModel 基类代码 #

public abstract class ViewModel : INotifyPropertyChanged, IDisposable
{
    public event PropertyChangedEventHandler PropertyChanged;

    protected virtual void OnPropertyChanged([CallerMemberName] string PropertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(PropertyName));
    }

    protected virtual bool Set<T>(ref T field, T value, [CallerMemberName] string PropertyName = null)
    {
        if (Equals(field, value)) return false;
        field = value;
        OnPropertyChanged(PropertyName);
        return true;
    }
}

问题原因分析 #

1. Setter逻辑错误：在SelectedValue的setter中，你先将selectedValue改为传入的value，判断后又把selectedValue改成目标值，但最后调用Set(ref selectedValue, value)时，传入的参数还是原始的value，而非修改后的目标值。
   • 更关键的是，Set方法会先判断字段（此时是目标值）和传入值（原始选中值）是否相等，因为不等，会把字段重新改回原始value，同时触发的PropertyChanged通知也是基于原始值，导致UI无法更新到目标项。
2. 额外同步属性干扰：IsSynchronizedWithCurrentItem="True"会让ComboBox的选中项与集合的ICollectionView当前项强制同步，这会和你手动控制SelectedValue的逻辑产生冲突，进一步导致UI无法正确响应重置操作。

解决方案 #

1. 修正SelectedValue的Setter逻辑 #

直接在判断后设置目标值并触发通知，避免中间变量的错误赋值：

private string selectedValue = "2";
public string SelectedValue
{
    get => selectedValue;
    set
    {
        if (value != SourceTypeCollection[1])
        {
            MessageBox.Show($"{value} Is not available yet", "Source Info", MessageBoxButton.OK);
            // 直接设置为目标值并触发属性变更通知
            Set(ref selectedValue, SourceTypeCollection[1]);
        }
        else
        {
            // 正常选中目标项时的处理
            Set(ref selectedValue, value);
        }
    }
}

2. 移除不必要的同步属性 #

删除ComboBox的IsSynchronizedWithCurrentItem="True"属性，避免与SelectedValue的绑定逻辑冲突：

<ComboBox ItemsSource="{Binding SourceTypeCollection, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
          SelectedValue="{Binding SelectedValue, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"                                         
          Height="30" FontSize="18" Width="150"
          VerticalAlignment="Top" Grid.Row="1"
          HorizontalAlignment="Center">
</ComboBox>

修改后，当选择禁用项时，ViewModel会直接将SelectedValue设置为目标值并触发属性变更通知，UI会正确更新到索引1的项，同时避免了同步属性带来的逻辑冲突。

内容的提问来源于stack exchange，提问作者Евгений Федоров