存储过程参数为空时查询全部数据的SQL实现求助

嘿，我一眼就看出问题出在哪了——你的WHERE子句里逻辑运算符的优先级没处理好，导致IS NULL的条件完全没按你预期的方式工作！

问题根源 #

SQL里AND的优先级比OR高，你原来的写法会被数据库解析成：

@Brand IN (...) OR (@Brand IS NULL AND @Category IN (...)) OR @Category IS NULL

这显然和你想要的「参数为NULL则匹配该字段全部值，否则按传入值筛选」逻辑完全不符。

修正后的完整代码 #

我帮你把逻辑调整好，同时也修正了临时表创建的语法错误（原来的create table #table1(VALUES)是不合法的，得明确列结构）：

CREATE PROC PROCNAME 
    @Brand nvarchar(max), 
    @Category nvarchar(max)
AS
BEGIN
    -- 开启NOCOUNT避免返回额外的行数消息
    SET NOCOUNT ON;

    -- 正确定义临时表结构，和要插入的字段类型匹配
    CREATE TABLE #table1 (Amount int);

    INSERT INTO #table1(Amount)
    SELECT Amount 
    FROM OriginalTable
    WHERE 
        -- 每个字段的判断用括号包裹，确保逻辑分组正确
        (@Brand IS NULL OR Brand IN (SELECT value FROM fn_split(@Brand, ',')))
        AND 
        (@Category IS NULL OR Categories IN (SELECT value FROM fn_split(@Category, ',')))
END

关键修正点说明 #

1. 括号分组逻辑：把每个字段的「参数为NULL」和「匹配拆分后的值」用括号包起来，这样每个字段的判断是一个独立的逻辑单元，AND会正确连接两个字段的筛选条件：

   • 如果@Brand为NULL，Brand的条件整体为true，只看Category的筛选逻辑
   • 如果@Category为NULL，Category的条件整体为true，只看Brand的筛选逻辑
   • 两个参数都不为NULL时，必须同时满足两个字段的匹配
   • 两个参数都为NULL时，返回所有记录

2. 临时表语法修正：明确指定临时表的列名和数据类型，避免插入数据时出现类型不匹配或语法错误

3. 额外优化：加上SET NOCOUNT ON;可以减少存储过程执行时返回的无关消息，让输出更干净

额外建议 #

如果你的SQL Server版本是2016及以上，建议用官方内置的STRING_SPLIT函数代替自定义的fn_split，它更稳定且性能更好，用法和你的fn_split一致：

SELECT value FROM STRING_SPLIT(@Brand, ',')

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