ComboBox选中失效求助：DataReader数据无法设置ComboBox选中项

我帮你梳理下核心问题和可行的解决方向：

1. 最关键的问题：事件触发顺序导致cboTeam设置被覆盖 #

你已经写了cboBase的SelectedIndexChanged事件逻辑（就是那段根据选中项填充cboTeam的代码）。当你从数据库读取数据后设置cboBase.SelectedIndex时，这个事件会立刻触发——它会清空cboTeam、重新添加选项，最后把cboTeam.SelectedIndex强制设为0。而你设置cboTeam选中索引的代码是在这之后执行的，自然会被事件里的SelectedIndex = 0覆盖掉！

解决办法：临时禁用事件，完成所有设置后再启用 #

在处理数据库数据之前，先把cboBase的SelectedIndexChanged事件暂时解绑，等所有控件都设置好后再重新绑定：

// 先解绑事件，避免提前触发填充逻辑
cboBase.SelectedIndexChanged -= cboBase_SelectedIndexChanged;

// 这里执行你的DataReader读取逻辑
// ...（省略读取代码）

// 先设置cboBase的选中索引
if (HomeBase == "JUR") { cboBase.SelectedIndex = 0; } 
else if (HomeBase == "LVR") { cboBase.SelectedIndex = 1; } 
else if (HomeBase == "SBW") { cboBase.SelectedIndex = 2; } 

// 手动调用一次cboBase的填充逻辑，确保cboTeam有正确的选项
cboBase_SelectedIndexChanged(null, EventArgs.Empty);

// 现在再设置cboTeam的选中索引（这时候不会被事件覆盖了）
int teamIndex = cboTeam.Items.IndexOf(TeamID);
if (teamIndex != -1)
{
    cboTeam.SelectedIndex = teamIndex;
}

// 最后重新绑定事件
cboBase.SelectedIndexChanged += cboBase_SelectedIndexChanged;

2. TeamID判断的冗余与错误 #

你写的TeamID判断里有重复的TeamID == "8"（分别对应索引5和2），这会导致后面的判断永远不会被执行。而且硬编码索引的方式很容易出错——比如当HomeBase切换时，cboTeam的选项列表变了，之前的索引对应关系就失效了。

优化方案：直接根据值查找索引 #

不用硬编码每个TeamID对应的索引，而是直接在cboTeam.Items里查找TeamID对应的位置：

int teamIndex = cboTeam.Items.IndexOf(TeamID);
if (teamIndex != -1)
{
    cboTeam.SelectedIndex = teamIndex;
}

这样不管cboTeam的选项列表怎么变，只要有对应的TeamID值，就能正确设置选中项，避免硬编码的错误。

3. RadioButton失效的修复 #

用PerformClick()模拟点击有时候会因为焦点、事件优先级等问题失效，直接设置Checked属性更可靠：

// 替换原来的PerformClick逻辑
radioPIC.Checked = (PIC == "1");
radioNo.Checked = (PIC == "0");

另外可以检查下PIC的值是否真的是"1"或"0"——比如数据库里是不是存的整数，转字符串的时候有没有额外的空格或者格式问题。

额外提醒：避免SQL注入风险 #

你的SQL语句用了字符串拼接："SELECT ... WHERE EmployeeID ='" + txtEmployeeID.Text + "'"，这存在严重的SQL注入漏洞。赶紧改成参数化查询：

SqlCommand cmd = new SqlCommand("SELECT EmployeeID, HomeBase, TeamID, StaffType, PIC From Employee WHERE EmployeeID = @EmployeeID", con);
cmd.Parameters.AddWithValue("@EmployeeID", txtEmployeeID.Text);

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