WinForm级联ComboBox更新客户时首次打开弹窗City下拉框不回显已选值

根本原因 #

完全匹配你描述的现象逻辑：

• 你在UpdateCustomer方法里的控件赋值顺序写反了：先给City下拉框设置历史值，再给Country下拉框设置历史值。
• 第一次打开弹窗时，Country下拉框默认是空值状态，给它设置国家值的操作，会因和当前状态不一致立刻触发CBxCountry_SelectedIndexChanged事件，连带执行refreshcity()给City下拉框绑定全新的国家对应城市列表。在绑定新数据源的过程中，City下拉框会自动把选中项重置为列表第一项，你之前提前写的CbxCity.Text = City的赋值会被直接覆盖丢失，最终只显示列表第一项。
• 第二次打开弹窗时，Country下拉框在上次关闭前已经保留了之前设置的国家值，这次再赋完全一样的国家文本，控件判定值没有变化，不会触发SelectedIndexChanged事件，自然不会执行refreshcity()替换City的数据源，之前设置的City历史值可以正常保留，就出现了第二次打开才正常的奇怪现象。

另外你代码里有个无效的空值判断：if(CBxCountry.SelectedValue.ToString() != null)，当SelectedValue为null时，调用.ToString()会直接抛出空引用异常，这个判断根本不会生效。
refreshcity方法末尾的City = this.CbxCity.GetItemText(this.CbxCity.SelectedItem);也会在每次刷新City列表时，把存储的历史City值覆盖成列表默认选中的项，进一步加剧回显失败的问题。

具体修复步骤 #

只需要调整赋值顺序，加一个加载状态标记屏蔽联动事件，不需要改动原有数据库查询逻辑：

1. 在窗体类中添加一个全局布尔标记，用来标识当前正在给控件加载初始值，暂时屏蔽下拉框的联动触发逻辑：

// 窗体类的全局字段
private bool _isLoadingInitialValue = false;

2. 修改CBxCountry_SelectedIndexChanged事件，在方法开头加上状态判断，修正原有的空值判断逻辑：

private void CBxCountry_SelectedIndexChanged(object sender, EventArgs e)
{
    // 初始加载阶段不触发联动刷新
    if (_isLoadingInitialValue) return;
    
    // 修正空值判断，避免空引用
    if (CBxCountry.SelectedValue == null || CBxCountry.SelectedValue == DBNull.Value)
    {
        return;
    }
    countryId = Convert.ToInt32(CBxCountry.SelectedValue);
    refreshcity(countryId);
}

3. 修改UpdateCustomer方法的赋值顺序，按照「加载状态标记置位 → 给Country赋值 → 手动加载对应城市列表 → 给City赋值回显 → 恢复加载状态标记」的流程执行：

public void UpdateCustomer() 
{
    // 进入初始加载模式，屏蔽联动事件
    _isLoadingInitialValue = true;
    
    LblAddUpdateCustomer.Text = "Update Customer";
    BtnUpdateSave.Text = "Update";
    TxtNameAdd.Text = CustomerName;
    TxtAddressAdd.Text = Address;
    TxtZipAdd.Text = PostalCode;
    TxtPhoneAdd.Text = Phone;
    
    // 先设置Country的历史值
    CBxCountry.Text = Country;
    countryId = Convert.ToInt32(CBxCountry.SelectedValue);
    
    // 手动调用方法加载对应国家的城市列表
    refreshcity(countryId);
    
    // 最后给City设置历史值回显，此时数据源已经加载完成，可以正常匹配
    CbxCity.Text = City;
    
    // 退出初始加载模式，恢复用户操作时的级联联动效果
    _isLoadingInitialValue = false;
}

4. 从refreshcity方法中移除那行自动覆盖City值的代码，City的值只在用户手动选择或者弹窗初始化时设置即可。

额外优化建议：不要全局复用同一个MySQL连接对象，建议在每次数据库操作时新建连接、操作完成后及时释放，避免连接状态异常导致的偶发报错。

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