为什么ORDER BY COUNT(lab_results.testid) DESC在SQL Server正常但C#无效

问题根因 #

核心错误是SQL片段拼接时没有预留空格，导致最终生成的SQL语法非法：
你定义的condition、groupby、orderby三个变量的开头都没有加空格，和前一段内容拼接后会直接连为一体，生成类似between '2024-01-01'GROUP BY labtests.testnameORDER BY COUNT(lab_results.testid) DESC的非法语句，SQL Server执行时报错，但你的代码没有捕获异常所以表现为没有输出。
之前不加ORDER BY能正常运行只是巧合：你测试时大概率没有选择日期范围，condition为空，sql末尾预留的空格刚好和后面的GROUP BY拼接后没有连写，一旦触发日期条件、或者加上ORDER BY片段就会触发语法错误。

修复方案 #

1. 给所有独立的SQL片段开头加空格，避免拼接后出现连写的非法语法：

// 每个SQL片段开头预留空格
string groupby = " GROUP BY labtests.testname";
// 也可以直接用查询别名简化ORDER BY写法，逻辑完全等价
// string orderby = " ORDER BY [Test Count] DESC";
string orderby = " ORDER BY COUNT(lab_results.testid) DESC";

2. 强烈建议替换直接拼接日期字符串的写法，改用参数化查询：既可以彻底避免SQL注入风险，也不需要额外处理日期格式、引号转义等问题，示例适配逻辑如下：

private void button2_Click(object sender, EventArgs e)
{
    DateTime fromDate;
    DateTime toDate;
    if (!DateTime.TryParse(dtFromDate.Value.ToString(), out fromDate))
    {
        MessageBox.Show("Invalid From Date");
        return;
    }
    if (!DateTime.TryParse(dtToDate.Value.ToString(), out toDate))
    {
        MessageBox.Show("Invalid to Date");
        return;
    }
    string sql = @"SELECT COUNT(lab_results.testid) as [Test Count] , labtests.TestName  
                    FROM LabTests
                    INNER JOIN lab_results ON labtests.testid = lab_results.testid
                    WHERE lab_results.approved_date IS NOT NULL 
                    AND cast(lab_results.approved_date as date) BETWEEN @FromDate AND @ToDate
                    GROUP BY labtests.testname
                    ORDER BY [Test Count] DESC";
    // 给你的data层传参即可，不需要手动拼接日期字符串
    DataTable dt = data.fireDatatable(sql, new SqlParameter("@FromDate", fromDate.Date), new SqlParameter("@ToDate", toDate.Date));
    OrdersDataGridView.DataSource = dt;
}

3. 建议在fireDatatable方法中增加异常捕获逻辑，调试时可以直接看到SQL执行的报错信息，快速定位问题。

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