WPF MVVM应用中ComboBox选中值无法传递至计算方法的问题

问题根源 #

你在MainWindow的构造函数中创建了两个完全独立的MyViewModel实例：

• 一个赋值给了DataContext，界面的ComboBox绑定的是这个实例的SelectedCompoundInterval
• 另一个赋值给了字段_myViewModel，点击按钮时调用的是这个实例的Calculate方法

这两个实例没有任何关联，所以按钮调用的Calculate方法访问的是未被界面更新过的SelectedCompoundInterval（始终为初始的null）。

修复方案 #

1. 统一ViewModel实例

修改MainWindow构造函数，只创建一个ViewModel实例，同时赋值给DataContext和_myViewModel字段：

public MainWindow()
{
    InitializeComponent();
    // 只创建一个实例，同时绑定到DataContext和字段
    _myViewModel = new MyViewModel();
    DataContext = _myViewModel;
}

2. 可选优化：避免初始值为null

在MyViewModel的构造函数中给SelectedCompoundInterval设置默认值，防止用户未选择选项时出现异常：

internal class MyViewModel : INotifyPropertyChanged
{
    public MyViewModel()
    {
        // 默认选中第一个复利周期选项
        SelectedCompoundInterval = CompoundIntervals[0];
    }

    // 原有代码...
}

3. 可选优化：简化绑定代码

ComboBox的SelectedItem绑定默认就是TwoWay模式，可以去掉冗余的Mode=TwoWay声明：

<ComboBox Grid.Column="1" 
          ItemsSource="{Binding CompoundIntervals}" 
          SelectedItem="{Binding SelectedCompoundInterval}" 
          Margin="10,5,0,0" Width="100" Height="20"/>

4. 可选优化：完善异常处理

在Calculations类中添加参数非空检查，避免null值导致的未预期异常：

public decimal CalculationsAmounts(decimal initialAmount, decimal interestRate, int years, int months, string compoundInterval, string interestRateType)
{
    if (string.IsNullOrWhiteSpace(compoundInterval))
        throw new ArgumentNullException(nameof(compoundInterval), "复利周期不能为空");
    if (string.IsNullOrWhiteSpace(interestRateType))
        throw new ArgumentNullException(nameof(interestRateType), "利率类型不能为空");

    // 原有逻辑代码...
}

5. 可选优化：遵循MVVM原则

去掉后台代码中直接切换ContentTemplate的逻辑，完全通过ViewModel的CurrentTemplate属性控制：

<!-- 修改XAML中的ContentControl绑定 -->
<ContentControl x:Name="MyContentControl" ContentTemplate="{Binding CurrentTemplate}"/>

// 简化按钮点击事件
private void CalculateButton_Click(object sender, RoutedEventArgs e)
{
    _myViewModel.Calculate();
}

private void BackButton_Click(object sender, RoutedEventArgs e)
{
    // 在ViewModel中添加切换回输入模板的方法
    _myViewModel.ShowInputTemplate();
}

在MyViewModel中添加对应方法：

public void ShowInputTemplate()
{
    _showResults = false;
    OnPropertyChanged(nameof(CurrentTemplate));
}

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