疑问：ComboBox的SelectedItem变更为何会修改ItemsSource？附代码

嘿，我之前也碰到过类似的坑，结合你的代码来看，这个问题大概率是几个细节没处理好导致的，咱们一步步拆解解决：

问题根源分析 #

1. IsSynchronizedWithCurrentItem="True" 的“副作用”：这个属性会让ComboBox的选中项和ItemsSource对应的CollectionView的CurrentItem强制同步。如果你的FamilyList是ObservableCollection这类支持CollectionView的集合，选中项变更时，CollectionView的CurrentItem也会跟着切换，要是你在ViewModel里对CurrentItem有修改操作，就会间接影响到ItemsSource里的元素。而且多数单纯选择项的场景，这个属性其实是多余的。
2. 引用类型绑定的特性：如果SelectedFamily和FamilyList里的元素都是自定义的引用类型（比如你的Family实体类），那么SelectedItem绑定的是对象的引用——也就是说，你修改SelectedFamily的属性时，本质上是在修改FamilyList里对应对象的属性，看起来就像是ItemsSource被修改了。

具体解决方案 #

1. 先移除多余的IsSynchronizedWithCurrentItem #

如果你的场景不需要和其他控件共享当前项（比如没有用CollectionView的上/下导航功能），直接把这个属性删掉就行，这是最容易解决的情况：

<ComboBox SelectedItem="{Binding SelectedFamily}" ItemsSource="{Binding FamilyList}"/>

2. 给SelectedItem绑定对象副本（避免修改原集合） #

如果你的需求是修改选中项的属性，但不想影响原集合里的元素，那你需要给SelectedFamily创建一个对象副本，而不是直接绑定原对象的引用。比如先给你的Family类加个克隆方法：

public class Family
{
    public string Name { get; set; }
    public int Age { get; set; }

    // 实现克隆方法，复制所有属性
    public Family Clone()
    {
        return new Family
        {
            Name = this.Name,
            Age = this.Age
        };
    }
}

然后在ViewModel的SelectedFamily setter里创建副本：

private Family _selectedFamily;
public Family SelectedFamily
{
    get => _selectedFamily;
    set
    {
        if (_selectedFamily != value)
        {
            // 用副本代替原对象，修改时就不会影响原集合了
            _selectedFamily = value?.Clone();
            OnPropertyChanged(nameof(SelectedFamily));
        }
    }
}

3. 检查ViewModel里的setter逻辑 #

仔细看看你ViewModel中SelectedFamily的setter，有没有不小心修改FamilyList的代码——比如误加了移除元素、修改集合的逻辑，这类代码会直接导致ItemsSource变化。举个错误示例：

// 错误：在setter里修改了原集合
public Family SelectedFamily
{
    get => _selectedFamily;
    set
    {
        if (_selectedFamily != value)
        {
            _selectedFamily = value;
            FamilyList.Remove(_selectedFamily); // 这行就会直接修改ItemsSource
            OnPropertyChanged(nameof(SelectedFamily));
        }
    }
}

如果有这类代码，直接移除或者调整逻辑就好。

4. 换成ObservableCollection管理集合 #

如果你的FamilyList还是普通的List<T>，建议换成ObservableCollection<T>，它会自动处理集合变更的UI通知，也能避免一些因为集合类型导致的意外同步问题：

public ObservableCollection<Family> FamilyList { get; set; } = new ObservableCollection<Family>();

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