CheckBoxList选SQL列后新窗体录入数据仅最后值有效其余为NULL的原因排查

问题根源 #

你遇到的这个问题，核心是窗体加载方法Test_Load里的数组初始化逻辑错误——每次循环都重新创建了txtBox和labels数组，导致之前循环生成的TextBox实例全部被丢弃，最终只有最后一次循环创建的数组元素是有效的，前面的位置全是null。

看这段出问题的代码：

for (int i = 0; i < DM.checkedListBox1.CheckedItems.Count; i++) {
    txtBox = new TextBox[DM.checkedListBox1.CheckedItems.Count]; // 每次循环都重新new数组
    labels = new Label[DM.checkedListBox1.CheckedItems.Count];
    // ... 给当前i索引的元素赋值
}

比如第一次循环i=0时，你创建了数组并给txtBox[0]赋值；第二次循环i=1时，又重新new了一个全新的数组，之前的txtBox[0]就直接丢失了，新数组里只有txtBox[1]有值，其他位置都是null。循环结束后，txtBox数组里只有最后一个索引的元素是有效的，所以拼接SQL时，除了最后一个值，其他都是空字符串或者NULL。

修复方案 #

1. 先修复窗体加载的数组初始化逻辑

把数组的创建移到循环外面，确保整个循环复用同一个数组，不会覆盖之前的元素：

private void Test_Load(object sender, EventArgs e) {
    TableLayoutPanel tableLayoutPanel = new TableLayoutPanel() { AutoSize = true };
    tableLayoutPanel.ColumnStyles.Add(new ColumnStyle(SizeType.AutoSize));
    int n = 0;
    
    // 关键：提前创建数组，不要在循环里反复new
    int checkedItemCount = DM.checkedListBox1.CheckedItems.Count;
    txtBox = new TextBox[checkedItemCount];
    labels = new Label[checkedItemCount];

    for (int i = 0; i < checkedItemCount; i++) {
        labels[i] = new Label();
        for (int j = i; j < DM.dataGridView1.Columns.Count; j++) {
            if (DM.dataGridView1.Columns[j].Visible) {
                labels[i].Text = DM.dataGridView1.Columns[j].HeaderText;
                break;
            }
        }
        tableLayoutPanel.RowStyles.Add(new RowStyle(SizeType.AutoSize));
        tableLayoutPanel.SetCellPosition(labels[i], new TableLayoutPanelCellPosition(0, n++));
        tableLayoutPanel.Controls.Add(labels[i]);
        
        txtBox[i] = new TextBox();
        tableLayoutPanel.RowStyles.Add(new RowStyle(SizeType.AutoSize));
        tableLayoutPanel.SetCellPosition(txtBox[i], new TableLayoutPanelCellPosition(0, n++));
        tableLayoutPanel.Controls.Add(txtBox[i]);
    }
    Controls.Add(tableLayoutPanel);
}

2. 优化SQL拼接（强烈推荐）

你当前用字符串拼接SQL的方式存在SQL注入风险，而且遇到单引号这类特殊字符会直接报错。改成参数化查询既能解决这些问题，也让代码更健壮：

private void Button1_Click(object sender, EventArgs e) {
    string targetTable = DM.comboBox1.SelectedItem.ToString();
    int checkedCount = DM.checkedListBox1.CheckedItems.Count;

    // 构建参数化的INSERT语句
    string columnNames = string.Join(", ", DM.checkedListBox1.CheckedItems.Cast<string>());
    string paramPlaceholders = string.Join(", ", Enumerable.Range(0, checkedCount).Select(x => $"@Col{x}"));
    string query = $"INSERT INTO {targetTable} ({columnNames}) VALUES ({paramPlaceholders})";

    // 使用using自动释放资源，比手动Close更可靠
    using (SqlConnection conn = new SqlConnection($"Server = {Fm1.ServerBox.Text }; Database = { Fm1.DBBox.Text}; Trusted_Connection = True")) {
        using (SqlCommand cmd = new SqlCommand(query, conn)) {
            // 逐个添加参数
            for (int i = 0; i < checkedCount; i++) {
                cmd.Parameters.AddWithValue($"@Col{i}", string.IsNullOrEmpty(txtBox[i].Text) ? DBNull.Value : txtBox[i].Text);
            }
            conn.Open();
            cmd.ExecuteNonQuery();
        }
    }
}

注意：如果CheckedItems里存的不是直接的表列名，你需要调整columnNames的生成逻辑，确保和目标表的列名完全匹配。

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