如何为ListViewModel的计算属性IsAllSelected实现双向绑定？

问题分析 #

你当前的代码存在两个核心问题：

1. UI更新不及时：IsAllSelected的getter是实时计算集合选中状态，但当集合内ItemViewModel的IsSelected属性变化时，ListViewModel不会主动触发IsAllSelected的PropertyChanged通知，导致UI无法感知状态变化。
2. 逻辑冗余矛盾：定义了私有字段isAllSelected但getter从未使用它，setter更新该字段的操作完全无效。

解决方案 #

要实现双向绑定的全选功能，需要完成三个关键步骤：监听集合内项的属性变化、监听集合本身的增删变化、修正全选属性的逻辑。

1. 完善ListViewModel的构造与事件监听

在构造函数中，不仅初始化集合，还要为现有项绑定PropertyChanged事件，同时监听集合的CollectionChanged事件，确保新增/移除项时同步处理事件绑定：

public class ListViewModel : ViewModelBase
{
    private readonly ObservableCollection<ItemViewModel> itemList;
    public IEnumerable<ItemViewModel> Items => itemList;

    public ListViewModel(IEnumerable<ItemViewModel> items)
    {
        itemList = new ObservableCollection<ItemViewModel>(items);
        // 为现有项绑定PropertyChanged事件
        foreach (var item in itemList)
        {
            item.PropertyChanged += Item_PropertyChanged;
        }
        // 监听集合的增删变化
        itemList.CollectionChanged += ItemList_CollectionChanged;
    }
}

2. 实现集合与项的事件处理方法

• 处理集合增删：新增项时绑定事件，移除项时解绑事件，避免内存泄漏
• 处理项的属性变化：当IsSelected变化时，触发IsAllSelected的属性变更通知

private void ItemList_CollectionChanged(object sender, System.Collections.Specialized.NotifyCollectionChangedEventArgs e)
{
    // 为新增的项绑定事件
    if (e.NewItems != null)
    {
        foreach (ItemViewModel newItem in e.NewItems)
        {
            newItem.PropertyChanged += Item_PropertyChanged;
        }
    }
    // 为移除的项解绑事件
    if (e.OldItems != null)
    {
        foreach (ItemViewModel oldItem in e.OldItems)
        {
            oldItem.PropertyChanged -= Item_PropertyChanged;
        }
    }
    // 集合变化后，更新全选状态的UI
    OnPropertyChanged(nameof(IsAllSelected));
}

private void Item_PropertyChanged(object sender, PropertyChangedEventArgs e)
{
    // 仅当IsSelected属性变化时，触发全选状态的更新
    if (e.PropertyName == nameof(ItemViewModel.IsSelected))
    {
        OnPropertyChanged(nameof(IsAllSelected));
    }
}

3. 修正IsAllSelected属性的逻辑

移除无用的私有字段，getter直接计算集合状态；setter批量设置项的选中状态，同时添加防重复触发的判断（避免批量设置时反复触发事件）：

public bool IsAllSelected
{
    get => itemList.All(item => item.IsSelected);
    set
    {
        // 如果当前状态和要设置的状态一致，直接返回，避免无效操作
        if (IsAllSelected == value) return;
        
        foreach (var item in itemList)
        {
            // 同样判断，避免重复触发Item的PropertyChanged
            if (item.IsSelected != value)
            {
                item.IsSelected = value;
            }
        }
        // 触发属性变更通知，更新UI
        OnPropertyChanged(nameof(IsAllSelected));
    }
}

额外注意事项 #

• 如果ItemViewModel的IsSelected是直接绑定到Model的isSelected属性，要确保ItemViewModel在Model属性变化时也会触发自身的PropertyChanged通知，否则ListViewModel无法感知底层Model的状态变化。
• 当集合规模较大时，批量设置IsSelected可能会频繁触发事件，可以考虑临时取消事件监听，设置完成后再恢复，提升性能。

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