C# CheckedListBox数据库查询实现求助及索引越界错误解决

问题根源 #

你碰到的起始索引不能小于0错误，几乎可以确定是代码里误用了CheckedListBox.SelectedIndex属性——当没有任何项被选中时，这个属性的值是-1，如果直接用它去访问Items集合（比如chkbx.Items[chkbx.SelectedIndex]），就会触发这个参数异常。

另外，你的需求是基于被勾选的项执行查询，但SelectedIndexChanged事件是响应"选中的高亮项"变化，和"勾选状态"完全不是一回事，这也是功能没实现的核心原因。

修正方案 #

我们需要调整两个核心点：

1. 针对勾选状态处理逻辑，而不是选中的索引
2. 安全拼接SQL语句，避免SQL注入风险

完整可运行的修正代码 #

推荐使用ItemCheck事件，因为它直接响应勾选状态的变化：

private void chkbx_ItemCheck(object sender, ItemCheckEventArgs e)
{
    // 延迟执行确保勾选状态已更新
    Task.Delay(100).ContinueWith(_ =>
    {
        this.Invoke((Action)(() =>
        {
            string query;
            var checkedItems = chkbx.CheckedItems.Cast<string>().ToList();

            if (checkedItems.Count == 0)
            {
                // 无勾选项时查询全部
                query = "SELECT * FROM SME";
            }
            else
            {
                // 使用参数化查询避免SQL注入
                var paramList = new List<string>();
                var conditionList = new List<string>();

                for (int i = 0; i < checkedItems.Count; i++)
                {
                    string paramName = $"@CheckedItem{i}";
                    paramList.Add(paramName);
                    // 替换下面的"YourTargetColumn"为你要匹配的数据库列名
                    conditionList.Add($"YourTargetColumn = {paramName}");
                }

                query = $"SELECT * FROM SME WHERE {string.Join(" OR ", conditionList)}";
            }

            // 执行数据库查询逻辑（示例）
            using (var connection = new SqlConnection("你的数据库连接字符串"))
            {
                connection.Open();
                using (var command = new SqlCommand(query, connection))
                {
                    // 为勾选项添加参数
                    for (int i = 0; i < checkedItems.Count; i++)
                    {
                        command.Parameters.AddWithValue($"@CheckedItem{i}", checkedItems[i]);
                    }

                    // 这里可以用SqlDataAdapter填充DataTable，或者用ExecuteReader读取数据
                    // 示例：
                    // var adapter = new SqlDataAdapter(command);
                    // var dataTable = new DataTable();
                    // adapter.Fill(dataTable);
                    // 绑定到DataGridView或其他控件...
                }
            }
        }));
    });
}

关键细节说明 #

• 使用CheckedItems集合：这个集合专门存储所有被勾选的项，不会出现索引为-1的问题，完全匹配你的需求
• 参数化查询：绝对不要直接把勾选的文本拼接到SQL语句里，这会导致严重的SQL注入漏洞，参数化查询是安全的标准做法
• 延迟与Invoke：ItemCheck事件触发时，勾选状态还没完全更新，所以用Task.Delay+Invoke确保获取到最新的勾选状态；如果是通过按钮点击触发查询，就不需要这部分，直接遍历CheckedItems即可

若坚持使用SelectedIndexChanged事件 #

如果你不想更换事件，也可以调整逻辑（但注意这个事件和勾选状态无关，只是高亮项变化）：

private void chkbx_SelectedIndexChanged(object sender, EventArgs e)
{
    try
    {
        var checkedItems = chkbx.CheckedItems.Cast<string>().ToList();
        string query;

        if (checkedItems.Count == 0)
        {
            query = "SELECT * FROM SME";
        }
        else
        {
            var paramList = new List<string>();
            var conditionList = new List<string>();

            for (int i = 0; i < checkedItems.Count; i++)
            {
                string paramName = $"@Item{i}";
                paramList.Add(paramName);
                conditionList.Add($"YourTargetColumn = {paramName}");
            }

            query = $"SELECT * FROM SME WHERE {string.Join(" OR ", conditionList)}";
        }

        // 执行数据库操作...
    }
    catch (Exception ex)
    {
        MessageBox.Show($"操作出错：{ex.Message}");
    }
}

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