ComboBox Item Template相关问题：重新初始化数据源后控件出现数据错误

嘿，我来帮你排查这个问题！你遇到的ComboBox数据错误既不是内存问题，也不是ItemTemplate的核心遗漏（不过ItemTemplate确实有可以优化的点），主要问题出在你重置数据源的操作方式上。

问题到底出在哪？ #

先看你按钮点击事件里的这段代码：

(DataContext as ViewModel).SortList.Clear();
(DataContext as ViewModel).SortList = new ObservableCollection<SortByType>(Enum.GetValues(typeof(SortByType)).OfType<SortByType>().ToList());

这里有两个关键问题：

1. 先清空再替换集合的操作逻辑冲突：当你调用Clear()时，原有的ObservableCollection会被清空，UI会立刻响应这个变化——ComboBox的选项会被清空，此时SelectedItem因为找不到对应的选项，会被自动设为null。紧接着你替换了整个SortList的集合实例，虽然触发了PropertyChanged通知UI更新，但ComboBox在经历了“清空→加载新集合”的过程后，可能无法自动重新匹配SelectedType对应的选中项，导致显示异常或者选中状态丢失。
2. ItemTemplate的显示逻辑有瑕疵：你现在的ItemTemplate直接绑定枚举值，所以ComboBox显示的是Y_Value这种枚举原始名称，而不是你通过[Display]特性设置的友好名称Y Value，这虽然不是数据错误的直接原因，但也是影响体验的小问题。

怎么解决？ #

1. 简化数据源重置逻辑

完全不需要先清空再替换集合！你的SortList属性已经实现了PropertyChanged通知，直接赋值新的ObservableCollection就足够了，UI会自动响应这个变化：

private void Button_Click(object sender, RoutedEventArgs e)
{
    var viewModel = DataContext as ViewModel;
    // 直接赋值新集合，UI会自动更新
    viewModel.SortList = new ObservableCollection<SortByType>(Enum.GetValues(typeof(SortByType)).OfType<SortByType>().ToList());
    
    // 如果需要保持原来的选中项，加下面这行就行（枚举是值类型，赋值会触发PropertyChanged）
    // viewModel.SelectedType = viewModel.SelectedType;
}

2. 让ComboBox显示友好名称

要让ComboBox显示[Display]特性里的文字，我们需要加一个转换器来读取枚举的Display属性：
首先创建一个转换器类：

public class EnumDisplayConverter : IValueConverter
{
    public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
    {
        if (value is Enum enumValue)
        {
            var field = enumValue.GetType().GetField(enumValue.ToString());
            var displayAttr = field.GetCustomAttribute<DisplayAttribute>();
            // 有Display属性就显示友好名称，没有就显示枚举原始名称
            return displayAttr?.Name ?? enumValue.ToString();
        }
        return value?.ToString() ?? string.Empty;
    }

    public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
    {
        throw new NotImplementedException();
    }
}

然后在XAML里引用这个转换器，修改ItemTemplate：

<Window.Resources>
    <!-- 注意替换local为你的项目命名空间 -->
    <local:EnumDisplayConverter x:Key="EnumDisplayConverter"/>
</Window.Resources>
<Grid>
    <StackPanel VerticalAlignment="Center">
        <ComboBox ItemsSource="{Binding SortList}" SelectedItem="{Binding SelectedType}">
            <ComboBox.ItemTemplate>
                <DataTemplate>
                    <Label Content="{Binding Converter={StaticResource EnumDisplayConverter}}"/>
                </DataTemplate>
            </ComboBox.ItemTemplate>
        </ComboBox>
        <Button Height="25" Width="150" Content="Details" Click="Button_Click"/>
    </StackPanel>
</Grid>

3. 额外的代码优化建议

你可以在窗口构造函数里把ViewModel存成私有字段，避免每次都强制转换DataContext，代码更简洁也更安全：

private ViewModel _viewModel;

public MainWindow()
{
    _viewModel = new ViewModel();
    DataContext = _viewModel;
    _viewModel.SelectedType = SortByType.Y_Value;
    InitializeComponent();
}

private void Button_Click(object sender, RoutedEventArgs e)
{
    _viewModel.SortList = new ObservableCollection<SortByType>(Enum.GetValues(typeof(SortByType)).OfType<SortByType>().ToList());
}

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