如何通过参数/变量动态调用对应编号的ComboBox控件？

你的需求完全可行，以下是两种实用的实现方案，同时会修正原代码中的SQL注入风险：

方案一：通过控件名称动态查找（WinForms环境） #

利用WinForms控件容器的Controls.Find方法，根据拼接的控件名称找到对应的ComboBox：

public void ComboText(string text, int i)
{
    try
    {
        // 拼接目标ComboBox的名称：cmb + 数字i
        string comboName = $"cmb{i}";
        // 在当前窗体的控件集合中查找，第二个参数true表示递归查找子控件
        var comboBox = Controls.Find(comboName, true).FirstOrDefault() as ComboBox;
        
        if (comboBox == null)
        {
            MessageBox.Show($"未找到名为{comboName}的ComboBox控件");
            return;
        }

        string connectionString = @"Data Source=DESKTOP-V0HE9JH\SQLEXPRESS01;Initial Catalog=Parts_Shop;Integrated Security=True";
        using (SqlConnection connection = new SqlConnection(connectionString))
        {
            connection.Open();
            // 使用参数化查询避免SQL注入
            string sql = "select manufacturer from Parts where item_type = @ItemType";
            using (SqlCommand command = new SqlCommand(sql, connection))
            {
                command.Parameters.AddWithValue("@ItemType", text);
                using (SqlDataReader srd = command.ExecuteReader())
                {
                    comboBox.Items.Clear(); // 先清空原有项，避免重复添加
                    while (srd.Read())
                    {
                        comboBox.Items.Add(srd.GetString(0));
                    }
                }
            }
        }
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
}

方案二：提前将ComboBox存入数组/列表（更高效） #

如果这些ComboBox是固定存在的，提前把它们放到数组或List中，直接通过索引访问，性能比动态查找更好：

// 先在窗体类中定义一个ComboBox数组，初始化时把所有cmb1、cmb2...cmbN放进去
private ComboBox[] _comboBoxes;

// 在窗体的构造函数或Load事件中初始化
public YourFormName()
{
    InitializeComponent();
    _comboBoxes = new ComboBox[] { cmb1, cmb2, cmb3, ..., cmb10 }; // 按顺序添加对应控件
}

// 修改后的ComboText方法
public void ComboText(string text, int i)
{
    try
    {
        // 注意：如果i是从1开始的，要减1对应数组索引（数组从0开始）
        if (i < 1 || i > _comboBoxes.Length)
        {
            MessageBox.Show("无效的ComboBox编号");
            return;
        }
        var comboBox = _comboBoxes[i - 1];

        string connectionString = @"Data Source=DESKTOP-V0HE9JH\SQLEXPRESS01;Initial Catalog=Parts_Shop;Integrated Security=True";
        using (SqlConnection connection = new SqlConnection(connectionString))
        {
            connection.Open();
            string sql = "select manufacturer from Parts where item_type = @ItemType";
            using (SqlCommand command = new SqlCommand(sql, connection))
            {
                command.Parameters.AddWithValue("@ItemType", text);
                using (SqlDataReader srd = command.ExecuteReader())
                {
                    comboBox.Items.Clear();
                    while (srd.Read())
                    {
                        comboBox.Items.Add(srd.GetString(0));
                    }
                }
            }
        }
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
}

关键说明 #

1. 两种方案都解决了动态获取控件的问题，方案二更适合控件数量固定的场景，性能更优；
2. 原代码存在严重的SQL注入风险，必须改用参数化查询，上面的代码已经修正了这个问题；
3. 使用using语句自动释放数据库连接、命令和阅读器资源，避免资源泄漏；
4. 添加了控件存在性检查，防止因控件名称错误或编号越界导致的异常。

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