如何优化ComboBox选中后禁用其余关联控件的冗余代码？

核心优化思路 #

避免硬编码多分支判断，通过控件关联映射统一处理所有ComboBox和对应文本框的状态，实现逻辑复用，提升可维护性。

具体实现方案 #

方案1：使用字典存储控件配对关系（推荐，关联关系清晰）

适合控件数量较多或需要明确管理配对关系的场景。

1. 定义全局映射字典

// 存储ComboBox与其对应TextBox的关联关系
private Dictionary<ComboBox, TextBox> _comboTextBoxPairs;

2. 初始化控件关联与事件绑定
   在窗体加载（或初始化逻辑）中完成映射和事件绑定：

private void Form_Load(object sender, EventArgs e)
{
    // 建立ComboBox和对应TextBox的映射
    _comboTextBoxPairs = new Dictionary<ComboBox, TextBox>
    {
        { comboBox1, textBox1 },
        { comboBox2, textBox2 },
        { comboBox3, textBox3 },
        // 按需添加更多控件配对
    };

    // 将所有ComboBox存入InputCheck列表
    InputCheck = _comboTextBoxPairs.Keys.ToList();

    // 为每个ComboBox绑定SelectedIndexChanged事件
    foreach (var combo in InputCheck)
    {
        combo.SelectedIndexChanged += Check;
    }
}

3. 重构Check方法，统一处理状态逻辑

private void Check(object sender, EventArgs e)
{
    var currentCombo = sender as ComboBox;
    if (currentCombo == null) return;

    // 判断当前ComboBox是否选中有效项
    bool isCurrentSelected = currentCombo.SelectedIndex > 0;

    // 遍历所有控件对，批量设置状态
    foreach (var pair in _comboTextBoxPairs)
    {
        var targetCombo = pair.Key;
        var targetTextBox = pair.Value;

        // 确定当前控件是否应该启用：仅触发事件的ComboBox保持启用（若已选中），否则全部启用
        bool shouldEnable = targetCombo == currentCombo;
        if (!isCurrentSelected)
        {
            shouldEnable = true;
        }

        // 设置控件启用状态
        targetCombo.Enabled = shouldEnable;
        targetTextBox.Enabled = shouldEnable;

        // 禁用时清空内容（按需调整）
        if (!shouldEnable)
        {
            targetCombo.SelectedIndex = 0;
            targetTextBox.Text = string.Empty;
        }
    }
}

方案2：使用控件Tag属性绑定（轻量实现）

适合简单场景，无需额外字典存储，直接通过控件的Tag属性关联对应文本框。

1. 初始化时绑定Tag属性

private void Form_Load(object sender, EventArgs e)
{
    // 将每个ComboBox的Tag设置为对应的TextBox
    comboBox1.Tag = textBox1;
    comboBox2.Tag = textBox2;
    comboBox3.Tag = textBox3;

    // 存入InputCheck列表并绑定事件
    InputCheck = new List<ComboBox> { comboBox1, comboBox2, comboBox3 };
    foreach (var combo in InputCheck)
    {
        combo.SelectedIndexChanged += Check;
    }
}

2. 重构Check方法

private void Check(object sender, EventArgs e)
{
    var currentCombo = sender as ComboBox;
    if (currentCombo == null) return;

    bool isCurrentSelected = currentCombo.SelectedIndex > 0;

    foreach (var combo in InputCheck)
    {
        bool shouldEnable = combo == currentCombo;
        if (!isCurrentSelected) shouldEnable = true;

        // 获取绑定的TextBox
        var textBox = combo.Tag as TextBox;
        
        combo.Enabled = shouldEnable;
        if (!shouldEnable) combo.SelectedIndex = 0;

        if (textBox != null)
        {
            textBox.Enabled = shouldEnable;
            if (!shouldEnable) textBox.Text = string.Empty;
        }
    }
}

优化后的优势 #

• 消除硬编码：新增/删除控件时，仅需修改初始化阶段的关联配置，无需改动核心逻辑代码。
• 逻辑复用：通过遍历统一处理所有控件状态，避免重复的条件判断和设置代码。
• 可维护性提升：控件关联关系清晰，需求变更时仅需调整遍历中的状态判断逻辑。

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