WPF项目中集合Getter应使用IEnumerable<T>还是ObservableCollection<T>

结论 #

你目前提供的两种写法都存在设计缺陷，既不要直接返回未做通知处理的IEnumerable<T>，也不要每次属性读取都新建ObservableCollection<T>实例，具体分析和最佳实践如下：

现有写法的问题 #

1. 直接返回IEnumerable<T>的问题

• 确实存在内存泄漏风险：WPF的项控件（DataGrid/ListView等）绑定未实现INotifyCollectionChanged接口的集合时，内部会创建静态强引用缓存，只要控件未销毁，集合实例就无法被GC回收，在视图频繁创建销毁的场景下内存泄漏问题会被放大。
• 功能异常：该属性返回的是Linq查询的临时结果，既没有集合变更通知，也没有属性变更通知，无论原Collection内容变更还是SomeValue变更，UI都不会自动刷新，完全不符合动态变更的需求。

2. 每次读取都新建ObservableCollection<T>的问题

这种写法问题比第一种更严重：

• 每次UI访问该属性都会生成全新的集合实例，触发全量UI重绘，性能损耗极大，用户操作产生的选中、排序等临时状态也会全部丢失。
• 频繁创建的集合实例会产生大量浮动内存，反而会加剧内存占用问题。

推荐实现方案 #

方案1：使用CollectionViewSource做过滤（优先选择，适配原集合是ObservableCollection<T>的场景）

这是WPF官方推荐的集合过滤方案，不需要生成新集合，复用原集合的视图层做过滤，性能最优，也完全规避内存泄漏问题：

// 原集合建议改为ObservableCollection，支持自动同步增删变更
public ObservableCollection<T> Collection { get; set; }
public ICollectionView PartOfCollection { get; }

private int _someValue;
public int SomeValue
{
    get => _someValue;
    set
    {
        _someValue = value;
        // 过滤条件变更时刷新视图
        PartOfCollection.Refresh();
        // 这里要触发你自己实现的INotifyPropertyChanged通知
        OnPropertyChanged();
    }
}

// 构造函数中初始化过滤视图
public YourViewModel()
{
    PartOfCollection = CollectionViewSource.GetDefaultView(Collection);
    PartOfCollection.Filter = item => ((T)item).SomeField == SomeValue;
}

XAML绑定不需要修改，仍然绑定PartOfCollection即可。

方案2：维护单例ObservableCollection<T>（适配原集合是固定List<T>的场景）

如果你的原集合不会动态增删，仅在过滤条件变更时才需要更新结果，就维护唯一的ObservableCollection实例，仅在需要时更新内容：

private ObservableCollection<T> _partOfCollection = new();
public ObservableCollection<T> PartOfCollection
{
    get => _partOfCollection;
    set
    {
        _partOfCollection = value;
        OnPropertyChanged();
    }
}

private int _someValue;
public int SomeValue
{
    get => _someValue;
    set
    {
        _someValue = value;
        OnPropertyChanged();
        UpdateFilteredCollection();
    }
}

// 统一更新过滤结果的方法
private void UpdateFilteredCollection()
{
    var filteredItems = Collection.Where(item => item.SomeField == SomeValue).ToList();
    PartOfCollection.Clear();
    foreach (var item in filteredItems)
    {
        PartOfCollection.Add(item);
    }
}

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