将SQL数据库绑定到ComboBox：跨类填充失败求助

嘿，我明白你的问题出在哪了——你在Database类里新建了一个Keukenhulp的实例，然后往这个新实例的cboxUser里加数据，但你实际运行的窗体是另一个完全不同的实例呀！这就像你往一个空购物袋里装东西，却转头去看另一个购物袋找东西，肯定看不到嘛 😄

核心问题拆解 #

你写的这段代码：

Keukenhulp KH = new Keukenhulp();

会创建一个全新的、从未显示过的窗体对象，你往这个对象的ComboBox里加数据，对正在运行的那个窗体完全没有影响。

解决方案：传递现有窗体实例 #

我们需要让Database类操作已经打开的那个窗体实例，而不是新建一个。具体可以这么做：

1. 修改Database类，添加接收窗体实例的方法

把你的数据填充逻辑封装成一个方法，让它接收Keukenhulp的现有实例作为参数：

public class Database
{
    // 假设你的连接字符串已定义在这里
    private string _connectionString = "你的连接字符串";

    public void LoadUsersToComboBox(Keukenhulp activeForm)
    {
        // 使用using语句自动释放资源，比手动Close更安全
        using (SqlConnection connection = new SqlConnection(_connectionString))
        {
            connection.Open();
            string queryText = "SELECT Gebruikersnaam FROM Gebruikers";
            
            using (SqlCommand query = new SqlCommand(queryText, connection))
            using (SqlDataReader dr = query.ExecuteReader())
            {
                // 先清空ComboBox避免重复数据
                activeForm.cboxUser.Items.Clear();
                
                while (dr.Read())
                {
                    // 记得把数据库值转成字符串再添加
                    activeForm.cboxUser.Items.Add(dr["Gebruikersnaam"].ToString());
                }
            }
        }
    }
}

2. 在窗体类里调用这个方法

在Keukenhulp窗体的加载事件（或者你需要触发数据加载的地方），把当前窗体实例（this）传递给Database的方法：

private void Keukenhulp_Load(object sender, EventArgs e)
{
    Database db = new Database();
    // 把正在显示的窗体实例传进去
    db.LoadUsersToComboBox(this);
}

（可选）更符合封装原则的优化

如果不想直接暴露窗体的ComboBox控件（毕竟直接操作控件违反封装），可以在Keukenhulp里写一个公开的方法来添加用户：

// 在Keukenhulp类里添加
public void AddUserToComboBox(string username)
{
    cboxUser.Items.Add(username);
}

然后修改Database类的方法：

while (dr.Read())
{
    activeForm.AddUserToComboBox(dr["Gebruikersnaam"].ToString());
}

（进阶）用数据绑定替代手动Add

如果想让代码更简洁、更易维护，建议用DataTable和数据绑定：

public void LoadUsersToComboBox(Keukenhulp activeForm)
{
    using (SqlConnection connection = new SqlConnection(_connectionString))
    {
        string queryText = "SELECT Gebruikersnaam FROM Gebruikers";
        SqlDataAdapter adapter = new SqlDataAdapter(queryText, connection);
        DataTable userTable = new DataTable();
        
        adapter.Fill(userTable);
        
        // 绑定数据源，自动填充ComboBox
        activeForm.cboxUser.DataSource = userTable;
        activeForm.cboxUser.DisplayMember = "Gebruikersnaam";
    }
}

这样不仅代码更短，还能自动处理数据刷新，后续如果需要修改显示字段也更方便。

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