WPF中编辑ObservableCollection内的项时ViewModel数据未更新的问题排查与解决

在我的应用中，我需要展示ObservableCollection中的项并允许用户编辑它们。网上很多讨论都是关于如何向集合中添加或删除项，但我找不到关于如何编辑项的内容。

我的ViewModel里有这个集合：

private ObservableCollection<ExerciseModel> exercises = new();
public ObservableCollection<ExerciseModel> Exercises 
{  
    get { return exercises; } 
    set
    {
        exercises = value;
        OnPropertyChanged();
    }
}

ExerciseModel实现了INotifyPropertyChanged，并且会初始化默认值：

public class ExerciseModel: INotifyPropertyChanged
{
    private static readonly int maxWeights = 5;
    public string IdExercise { get; set; } = string.Empty;
    public string IdExerciseSet { get; set; } = string.Empty;
    public string ExerciseName { get; set; } = string.Empty;
    public ObservableCollection<string> exerciseWeights = new();
    public ObservableCollection<string> ExerciseWeights
    {
        get { return exerciseWeights; }
        set { exerciseWeights = value; OnPropertyChanged(); }
    }

    public event PropertyChangedEventHandler? PropertyChanged;
    public void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
    public ExerciseModel() 
    {
            for (int i = 0; i < maxWeights; i++)
            {
                ExerciseWeights.Add("10");
        }
    }
}

我有点搞不懂绑定TextBox的部分，我用了Binding Path=.的方式，但不确定是否正确：

<ItemsControl Grid.Row="3" Grid.Column="0"  Grid.ColumnSpan="7" Grid.RowSpan="5" VerticalAlignment="Top" ItemsSource="{Binding Exercises}">
    <ItemsControl.ItemTemplate>
        <DataTemplate>
            <StackPanel Orientation="Horizontal">
                <TextBox Width="75" Text="{Binding ExerciseName}"/>
                <ItemsControl ItemsSource="{Binding ExerciseWeights}">
                    <ItemsControl.ItemsPanel>
                        <ItemsPanelTemplate>
                            <StackPanel Orientation="Horizontal"/>
                        </ItemsPanelTemplate>
                    </ItemsControl.ItemsPanel>
                    <ItemsControl.ItemTemplate>
                        <DataTemplate>
                            <StackPanel>
                                <TextBox Width="30"  Text="{Binding Path=.}"/>
                                <!-- Path=. Not really sure if this is the right way to bind -->
                            </StackPanel>
                        </DataTemplate>
                    </ItemsControl.ItemTemplate>
                </ItemsControl>
            </StackPanel>
        </DataTemplate>
    </ItemsControl.ItemTemplate>
</ItemsControl>

现在这些控件能正常显示默认值，但当我修改任何值时，它们都不会同步到我的ViewModel中，问题出在哪里呢？

问题分析与解决办法 #

我来帮你拆解一下核心问题和对应的解决方案：

1. 核心问题：不可变类型的绑定限制

你遇到的主要问题出在ExerciseWeights的绑定上：

• 你用ObservableCollection<string>存储权重值，而字符串是不可变引用类型，一旦创建就无法修改内容，只能生成新的字符串实例。
• 当你用Path=.绑定集合中的字符串项时，默认的绑定模式是OneWay，UI上的修改无法同步回集合；即使设置TwoWay，因为字符串不可变，新输入的内容会生成新的字符串实例，无法自动替换集合中原有的项。

另外，你的ExerciseName、IdExercise等属性是自动属性，没有在setter中调用OnPropertyChanged，虽然UI修改能同步到模型，但模型代码修改这些属性时，UI不会自动更新，这也是需要完善的点。

2. 具体解决方案

方案一：用可绑定包装类包裹字符串（推荐，符合MVVM）

创建一个简单的包装类，把字符串包装成可通知的属性，这样就能实现双向绑定：

public class StringWrapper : INotifyPropertyChanged
{
    private string _value = string.Empty;
    public string Value
    {
        get => _value;
        set
        {
            if (_value != value)
            {
                _value = value;
                OnPropertyChanged();
            }
        }
    }

    public event PropertyChangedEventHandler? PropertyChanged;
    public void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

然后修改ExerciseModel中的ExerciseWeights类型，并调整初始化逻辑：

public class ExerciseModel: INotifyPropertyChanged
{
    private static readonly int maxWeights = 5;

    // 完善自动属性的通知逻辑
    private string _idExercise = string.Empty;
    public string IdExercise
    {
        get => _idExercise;
        set
        {
            if (_idExercise != value)
            {
                _idExercise = value;
                OnPropertyChanged();
            }
        }
    }

    private string _idExerciseSet = string.Empty;
    public string IdExerciseSet
    {
        get => _idExerciseSet;
        set
        {
            if (_idExerciseSet != value)
            {
                _idExerciseSet = value;
                OnPropertyChanged();
            }
        }
    }

    private string _exerciseName = string.Empty;
    public string ExerciseName
    {
        get => _exerciseName;
        set
        {
            if (_exerciseName != value)
            {
                _exerciseName = value;
                OnPropertyChanged();
            }
        }
    }

    // 改为包装类的集合，并且设为只读属性（通常不需要替换整个集合）
    public ObservableCollection<StringWrapper> ExerciseWeights { get; } = new();

    public event PropertyChangedEventHandler? PropertyChanged;
    public void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    public ExerciseModel() 
    {
        for (int i = 0; i < maxWeights; i++)
        {
            ExerciseWeights.Add(new StringWrapper { Value = "10" });
        }
    }
}

最后修改XAML中的绑定，指向包装类的Value属性：

<ItemsControl Grid.Row="3" Grid.Column="0"  Grid.ColumnSpan="7" Grid.RowSpan="5" VerticalAlignment="Top" ItemsSource="{Binding Exercises}">
    <ItemsControl.ItemTemplate>
        <DataTemplate>
            <StackPanel Orientation="Horizontal">
                <TextBox Width="75" Text="{Binding ExerciseName, Mode=TwoWay}"/>
                <ItemsControl ItemsSource="{Binding ExerciseWeights}">
                    <ItemsControl.ItemsPanel>
                        <ItemsPanelTemplate>
                            <StackPanel Orientation="Horizontal"/>
                        </ItemsPanelTemplate>
                    </ItemsControl.ItemsPanel>
                    <ItemsControl.ItemTemplate>
                        <DataTemplate>
                            <StackPanel>
                                <TextBox Width="30" Text="{Binding Value, Mode=TwoWay}"/>
                            </StackPanel>
                        </DataTemplate>
                    </ItemsControl.ItemTemplate>
                </ItemsControl>
            </StackPanel>
        </DataTemplate>
    </ItemsControl.ItemTemplate>
</ItemsControl>

方案二：手动处理UI更新（适合简单场景，非MVVM友好）

如果不想创建包装类，可以给TextBox绑定LostFocus事件，手动把UI输入的值更新到集合中。但这种方式需要在代码后台写逻辑，不符合MVVM的设计思想，只推荐临时使用。

3. 额外优化建议

• 对于ObservableCollection类型的属性，通常不需要提供setter（因为我们一般只修改集合内的项，不会替换整个集合），设为只读属性可以简化代码，避免意外赋值。
• 显式指定绑定的Mode=TwoWay，让代码逻辑更清晰，虽然TextBox的Text绑定默认是TwoWay。

备注：内容来源于stack exchange，提问作者kkalwa