ComboBox选中值异常：获取旧值而非新值的问题排查

看起来你遇到的核心问题是SelectionChanged事件中无法及时获取当前选中的新值，同时初始化后控件为空、IP显示滞后。我来帮你一步步排查并解决这些问题：

问题根源分析 #

1. Text属性更新滞后：WPF中ComboBox的Text属性在SelectionChanged事件触发时可能还未同步更新到新选中的值，导致你拿到的是旧值
2. 初始化配置冗余：你同时设置了DataContext和ItemsSource，属于重复配置，可能导致绑定异常
3. 硬编码初始值不匹配：XAML中设置Text="0"，但数据库中可能没有对应数据，导致初始状态为空
4. SQL查询存在风险：直接拼接字符串的查询方式存在SQL注入漏洞，且未处理空结果的情况

具体解决方案 #

1. 修正ComboBox的XAML配置

去掉硬编码的Text="0"，我们将在初始化代码中设置默认选中项：

<ComboBox Grid.Column="1" x:Name="stationComboBox_1" FontSize="25" Width="60" HorizontalAlignment="Left" 
          DisplayMemberPath="stationNumber_stations" 
          ItemsSource="{Binding}" 
          SelectionChanged="StationComboBox_1_SelectionChanged" />

2. 修改SelectionChanged事件处理逻辑

直接从SelectedItem获取当前选中的数据行，避免Text属性的更新延迟：

private void StationComboBox_1_SelectionChanged(object sender, SelectionChangedEventArgs e)
{
    // 确保选中项有效
    if (stationComboBox_1.SelectedItem is DataRowView selectedStationRow)
    {
        string currentStationNumber = selectedStationRow["stationNumber_stations"].ToString();
        GetIP(currentStationNumber);
    }
}

3. 重构GetIP方法（使用参数化查询+资源自动释放）

解决SQL注入风险，同时处理查询结果为空的情况：

private void GetIP(string stationNumber)
{
    string connectionString = "SERVER=localhost;DATABASE=db; UID=PC;Password=pw;";
    // 使用using自动释放数据库连接资源
    using (MySqlConnection connection = new MySqlConnection(connectionString))
    {
        // 参数化查询避免SQL注入
        string query = "SELECT IP_stations from stations WHERE stationNumber_stations=@StationNumber";
        MySqlCommand cmd = new MySqlCommand(query, connection);
        cmd.Parameters.AddWithValue("@StationNumber", stationNumber);
        
        connection.Open();
        object queryResult = cmd.ExecuteScalar();
        // 处理查询结果为空的情况，避免空引用异常
        DisplayIP.Text = queryResult != null ? queryResult.ToString() : "未找到对应IP";
    }
}

4. 优化初始化代码（SQLSetup）

去掉冗余的DataContext设置，添加默认选中逻辑，确保启动后立即显示第一个站点的IP：

public void SQLSetup()
{
    string connectionString = "SERVER=localhost;DATABASE=db; UID=PC;Password=pw;";
    using (MySqlConnection connection = new MySqlConnection(connectionString))
    {
        MySqlCommand cmd = new MySqlCommand("Select stationNumber_stations from stations", connection);
        connection.Open();
        DataTable dt = new DataTable();
        dt.Load(cmd.ExecuteReader());
        
        var stationsDataView = dt.DefaultView;
        // 仅设置ItemsSource即可，无需重复设置DataContext
        stationComboBox_1.ItemsSource = stationsDataView;
        // 其他ComboBox同理复制上述代码
        
        // 默认选中第一个站点（如果数据存在）
        if (stationsDataView.Count > 0)
        {
            stationComboBox_1.SelectedIndex = 0;
        }
    }
    
    // 下面这些初始化的stationSelection变量如果没有实际用途可以删除
    // string stationSelection_1 = stationComboBox_1.Text;
    // ...
}

为什么这样修改能解决问题？ #

• SelectedItem直接获取数据源：绕过了Text属性的更新延迟问题，直接从绑定的DataRowView中读取当前选中的站点编号
• 参数化查询更安全：避免了SQL注入攻击，同时兼容特殊字符的站点编号
• 默认选中逻辑：启动后自动选中第一个站点，触发SelectionChanged事件，立即显示对应IP
• 资源自动释放：using语句确保数据库连接及时关闭，避免资源泄漏

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