WPF/C#中如何阻止ListBox选中项在数据输入时变更？

嘿，我完全懂你现在的烦恼——每次在TextBox回车添加数据，ListBox的选中项就乱跑，导致你没法精准指定数据要存到哪个班级里对吧？你尝试保存SelectedIndex再恢复没成功，大概率是因为重新加载数据时索引对应的项已经变了，或者绑定逻辑把选中状态重置了，咱们一步步来解决：

问题根源分析 #

你之前的代码里保存了SelectedIndex，但在执行ShowClass()后，ListBox的数据源是重新从数据库加载的，这时候原来的索引可能已经不对应之前选中的班级了（比如数据库里新增了班级，或者顺序变了），直接设置回索引自然没用。另外，TextBox获得焦点时ListBox会失去焦点，但默认情况下不会取消选中，所以核心问题还是重新加载数据后如何正确恢复选中状态。

修正后的解决方案 #

咱们换个思路：不要保存索引，而是保存选中项的唯一标识（比如KlassenId），等重新加载数据后，根据这个标识找到对应的项再选中，这样就不会受索引变化的影响了。修改后的代码如下：

// 不需要全局变量y，直接在事件里处理
private void TxbSchüler_KeyDown(object sender, KeyEventArgs e)
{
    if (e.Key == Key.Enter)
    {
        // 先保存选中班级的ID，避免重新加载后找不到
        int selectedClassId = 0;
        if (listClasses.SelectedItem != null)
        {
            // 这里根据你ListBox绑定的数据类型调整：
            // 如果是绑定的实体类（比如ClassModel），就转成实体取ID
            // var selectedClass = listClasses.SelectedItem as ClassModel;
            // selectedClassId = selectedClass?.KlassenId ?? 0;
            
            // 如果是直接绑定DataTable/DataSet，用SelectedValue更方便
            selectedClassId = Convert.ToInt32(listClasses.SelectedValue);
        }

        // 先判断是否选中了班级，避免无效插入
        if (selectedClassId == 0)
        {
            MessageBox.Show("请先选择一个班级！");
            return;
        }

        try
        {
            string query = "insert into Schüler values (@Name, @KlassenId)";
            // 使用using自动释放SqlCommand资源
            using (SqlCommand sqlCommand = new SqlCommand(query, sqlConnection))
            {
                sqlConnection.Open();
                // 对输入文本做Trim，避免空内容或多余空格
                sqlCommand.Parameters.AddWithValue("@Name", txbSchüler.Text.Trim());
                sqlCommand.Parameters.AddWithValue("@KlassenId", selectedClassId);
                // 插入操作用ExecuteNonQuery更合适，返回受影响的行数
                int affectedRows = sqlCommand.ExecuteNonQuery();
                if (affectedRows > 0)
                {
                    MessageBox.Show("添加成功！");
                }
            }
        }
        catch (Exception ex)
        {
            MessageBox.Show($"添加失败：{ex.Message}");
        }
        finally
        {
            // 确保连接关闭
            if (sqlConnection.State == ConnectionState.Open)
                sqlConnection.Close();
            // 重新加载数据
            ShowStudent();
            ShowClass();

            // 重新加载后，根据保存的ID找到对应的班级并选中
            foreach (var item in listClasses.Items)
            {
                // 同样根据绑定的数据类型调整：
                // 实体类版本：
                // var classItem = item as ClassModel;
                // if (classItem != null && classItem.KlassenId == selectedClassId)
                
                // DataTable版本：
                var row = item as DataRowView;
                if (row != null && Convert.ToInt32(row["KlassenId"]) == selectedClassId)
                {
                    listClasses.SelectedItem = item;
                    break;
                }
            }
        }

        // 清空TextBox，方便下一次输入
        txbSchüler.Clear();
        // 可选：让TextBox重新获得焦点，提升输入体验
        txbSchüler.Focus();
    }
}

额外的优化建议 #

• 避免全局变量：你之前用了全局变量y，其实完全可以在事件内部处理，减少不必要的全局状态。
• 使用using管理资源：SqlCommand、SqlConnection这类资源最好用using包裹，确保用完自动释放，避免内存泄漏。
• 增加输入校验：比如判断TextBox是否为空，是否选中了班级，避免无效的数据库操作。
• 优先用SelectedValue/SelectedItem：相比SelectedIndex，用实际的数据标识更可靠，尤其是在数据源会动态变化的场景下。

内容的提问来源于stack exchange，提问作者Benedikt J Schlegel