WPF MVVM：获取绑定控件值并实现对象更新的问题

嘿，刚接触MVVM和数据绑定确实容易踩这些小坑，我来帮你理清楚问题出在哪以及怎么一步步解决～

1. 先解决眼前的报错：空引用异常 #

你在Category = SelectedCategory.Category;这行报错，大概率是**SelectedCategory为null**导致的空引用异常。原因可能是：

• ViewModel里的SelectedCategory没有初始化，比如构造函数里没给它赋值；
• 界面上没有选中对应的分类项（比如绑定了ListBox的话，用户没选任何项，SelectedCategory就会是null）。

解决办法： #

在ClickUpdate方法开头先加个判断，避免空引用：

private async void ClickUpdate() {
    ButtonIsEnabled = false;
    try
    {
        if(SelectedCategory == null)
        {
            ShowMessageBox("请先选择一个分类，或确保分类对象已初始化");
            return;
        }
        // 后续逻辑...
    }
    catch(Exception ex)
    {
        ShowMessageBox($"操作出错：{ex.Message}");
    }
    finally
    {
        ButtonIsEnabled = true; // 不管成功失败，都要恢复按钮可用状态
    }
}

另外，建议在ViewModel的构造函数里初始化SelectedCategory，避免它一开始就是null：

public YourViewModel()
{
    SelectedCategory = new EventCategory();
}

2. 理顺数据绑定逻辑，不需要手动赋值控件内容 #

你的TextBox绑定Text="{Binding SelectedCategory.Category, Mode=TwoWay}"是正确的——TwoWay模式下，TextBox的输入内容会自动同步到SelectedCategory.Category属性里，根本不需要再写Category = SelectedCategory.Category;这行代码，这一步完全多余，反而容易混淆逻辑。

正确的Update调用逻辑： #

你需要把SelectedCategory里的最新值，赋值给从数据库获取的实体，再调用Update方法：

// 移除这行多余的代码：Category = SelectedCategory.Category;
Id = SelectedCategory.Id;
IsActive = SelectedCategory.IsActive; // 这行如果是ViewModel自身的属性，也可以不用，直接用SelectedCategory的属性

var service = new DataService<EventCategory>(new TimeKeeprDbContextFactory());
EventCategory eventCategory = await service.Get(SelectedCategory.Id);
if (eventCategory == null) {
    ShowMessageBox("There is no such category. Please add a new one or chose another");
} else {
    // 把界面上修改后的属性值同步到数据库实体
    eventCategory.Category = SelectedCategory.Category;
    eventCategory.IsActive = SelectedCategory.IsActive;
    // 调用Update方法更新数据库
    await service.Update(SelectedCategory.Id, eventCategory);
    ShowMessageBox("分类更新成功！");
}

3. CheckBox和TextBlock的内容获取方式 #

• CheckBox：如果你的CheckBox绑定是IsChecked="{Binding SelectedCategory.IsActive, Mode=TwoWay}"，那么SelectedCategory.IsActive会自动同步CheckBox的勾选状态，直接用这个属性就能拿到最新值，和TextBox的逻辑一样。
• TextBlock：TextBlock默认是单向绑定（OneWay），主要用于展示内容。如果它绑定的是Text="{Binding SelectedCategory.SomeProperty}"，直接读取SelectedCategory.SomeProperty就能拿到展示的内容；如果需要编辑，建议换成TextBox，TextBlock不适合输入操作。

4. 给新手的额外小建议 #

• 确保你的ViewModel正确实现了INotifyPropertyChanged接口（从你的代码里的OnPropertyChanged来看应该是实现了，但可以确认一下，这是数据绑定生效的基础）。
• 尽量使用异步Command（比如自定义AsyncCommand）来替代BaseCommand里的async void，因为async void在Command中可能导致异常无法被捕获。如果暂时没有异步Command，一定要在ClickUpdate里加try-catch，像我上面代码里那样。
• 你的另一个TextBox绑定用了Source={StaticResource viewModel}，如果你的Window/UserControl的DataContext已经设置为这个ViewModel，完全可以去掉Source，直接写Text="{Binding Password, ...}"，这样代码更简洁。

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