ComboBox选择ALL选项时无法从MySQL查询全量数据求助

我来帮你搞定这个问题！你的代码里有几个核心问题导致全量数据无法正常加载，还存在SQL注入的安全风险，下面是修复后的完整代码和详细说明：

问题根源分析 #

1. 逻辑判断错误：你用了字符串连接符&代替逻辑与运算符AndAlso，导致ComboBox1.Text = "ALL TYRES" & ComboBox2.Text = "ALL SIZES"这个条件永远不会成立，全量查询的代码块根本没执行过。
2. Query覆盖问题：就算全量查询的代码侥幸执行了，后面的Query变量赋值和Command重新初始化会直接覆盖之前的全量查询命令，最终跑的还是特定筛选的SQL。
3. SQL注入风险：直接把ComboBox的文本拼进SQL语句是非常危险的操作，很容易被恶意用户利用进行注入攻击。

修复后的完整代码 #

Private Sub btnSearch_Click(sender As Object, e As EventArgs) Handles btnSearch.Click
    ' 使用Using语句自动管理连接资源，不用手动Dispose和Close
    Using MysqlConn As New MySqlConnection("server=localhost;userid=root;password=root;database=golden_star")
        Dim dbdataset As New DataTable
        Dim bSource As New BindingSource

        Try
            MysqlConn.Open()
            Dim queryBuilder As New StringBuilder()
            Dim cmd As New MySqlCommand()

            ' 先写基础查询语句
            queryBuilder.Append("SELECT sale_id, date, brand, size, selling_unit_price, cost_unit_price, quantity, cost_of_goods, profit, total_cost_price FROM golden_star.sales")

            ' 动态收集筛选条件
            Dim filterConditions As New List(Of String)()
            If ComboBox1.Text <> "ALL TYRES" Then
                filterConditions.Add("brand = @Brand")
                cmd.Parameters.AddWithValue("@Brand", ComboBox1.Text)
            End If
            If ComboBox2.Text <> "ALL SIZES" Then
                filterConditions.Add("size = @Size")
                cmd.Parameters.AddWithValue("@Size", ComboBox2.Text)
            End If

            ' 如果有筛选条件，拼接WHERE子句
            If filterConditions.Count > 0 Then
                queryBuilder.Append(" WHERE ").Append(String.Join(" AND ", filterConditions))
            End If

            ' 配置命令并执行查询
            cmd.CommandText = queryBuilder.ToString()
            cmd.Connection = MysqlConn

            Dim Sda As New MySqlDataAdapter(cmd)
            Sda.Fill(dbdataset)
            bSource.DataSource = dbdataset
            DataGridView1.DataSource = bSource

        Catch ex As MySqlException
            MessageBox.Show($"数据库查询出错啦：{ex.Message}")
        End Try
    End Using ' 代码块结束后自动关闭并释放连接资源
End Sub

关键改进说明 #

• 正确的逻辑判断：通过检查ComboBox文本是否不等于ALL选项，动态构建筛选条件，摆脱了复杂的多分支判断逻辑，代码更清晰。
• 参数化查询：用@Brand和@Size参数代替直接字符串拼接，彻底杜绝SQL注入风险，还能避免特殊字符导致的SQL语法错误。
• 自动资源管理：Using语句会自动处理数据库连接的关闭和资源释放，不用再手动调用Close()或Dispose()，减少出错概率。
• 灵活的SQL构建：用StringBuilder和List(Of String)动态拼接查询条件，后续要加新的筛选条件也很方便维护。

额外小建议 #

• 可以在ComboBox初始化的时候，把"ALL TYRES"和"ALL SIZES"设为默认选中项，提升用户体验。
• 最好把品牌和尺寸的下拉选项直接从数据库里加载唯一值，不要手动硬编码，这样数据更准确也更灵活。

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