SQL Server获取Color列Top10高频颜色值的问题排查

核心问题分析 #

• ExecuteScalar()仅返回查询结果的第一行第一列，无法获取多行数据，这是你只能拿到单个值的根本原因。
• 原SQL语句存在三处问题：
  • 排序方向错误：要取高频颜色，需用ORDER BY ValueFrequency DESC（降序），而非ASC（升序）
  • 缺少Top10限制：需添加TOP 10筛选出频率最高的10个颜色
  • 列名不一致：原统计行数用Date列，获取颜色时用QueriedAt，请确认表中实际日期列名，避免查询错误

修正后的完整代码 #

var connection = new SqlConnection(connString);

using (connection)
{
    connection.Open();

    // 统计当日新增行数（优化为参数化查询，避免SQL注入）
    using (var cmd = new SqlCommand("SELECT COUNT(*) FROM [dbo].[Cars] WHERE Date LIKE @today + '%'", connection))
    {
        cmd.Parameters.AddWithValue("@today", today);
        var total = Convert.ToInt32(cmd.ExecuteScalar());
        Console.WriteLine($"Total car orders today: {total}");
    }

    // 获取当日Top10高频颜色
    using (var cmd = new SqlCommand(
        "SELECT TOP 10 Color, COUNT(Color) AS ValueFrequency FROM [dbo].[Cars] WHERE Date LIKE @today + '%' GROUP BY Color ORDER BY ValueFrequency DESC", 
        connection))
    {
        cmd.Parameters.AddWithValue("@today", today);
        using (var reader = cmd.ExecuteReader())
        {
            Console.WriteLine("Top 10 colors today:");
            while (reader.Read())
            {
                int color = reader.GetInt32(0);
                int frequency = reader.GetInt32(1);
                Console.WriteLine($"Color {color}: {frequency} times");
            }
        }
    }
}

关键修改说明 #

• 改用SqlDataReader遍历查询结果，完整获取Top10的颜色及对应频率
• SQL语句添加TOP 10并将排序改为DESC，确保取频率最高的前10个颜色
• 替换字符串拼接为参数化查询，彻底避免SQL注入风险，同时提升代码健壮性
• 统一表名拼写（原代码中[db].[Cars]与[dbo].[Cars]混用，已统一为[dbo].[Cars]，请根据实际数据库架构调整）

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