通用计数存储过程返回两个结果集问题求助

你的存储过程返回两个结果集的核心原因是执行了两次输出结果的操作：

1. EXEC(@table) 会直接执行动态拼接的SQL语句，返回COUNT(列名)的真实统计结果；
2. 随后的SELECT @Rowcount会输出@@ROWCOUNT的值——而@@ROWCOUNT在这里表示的是动态SQL语句返回的行数（COUNT函数永远只会返回1行结果），所以这个值始终是1。

另外你的代码还有几个潜在问题：

• 参数名拼写错误：@ColunName应该是@ColumnName；
• @table变量长度仅VARCHAR(30)，很容易因为表名/列名较长导致SQL语句被截断，引发执行错误；
• 存在SQL注入风险，直接拼接用户传入的表名/列名是不安全的。

我们可以通过sp_executesql来安全地执行动态SQL，并将统计结果赋值给变量，最后只返回一次结果：

ALTER PROCEDURE [dbo].[GenericCountAll] 
    @TableName NVARCHAR(100), 
    @ColumnName NVARCHAR(100) 
AS 
BEGIN 
    SET NOCOUNT ON; -- 关闭影响行数的默认输出，避免额外结果

    DECLARE @DynamicSQL NVARCHAR(MAX);
    DECLARE @Rowcount INT;

    -- 用QUOTENAME包裹表名和列名，防止SQL注入和标识符转义
    SET @DynamicSQL = N'SELECT @Result = COUNT(' + QUOTENAME(@ColumnName) + N') FROM ' + QUOTENAME(@TableName);

    -- 执行动态SQL并获取输出参数
    EXEC sp_executesql 
        @DynamicSQL,
        N'@Result INT OUTPUT',
        @Result = @Rowcount OUTPUT;

    -- 只返回一次统计结果
    SELECT @Rowcount AS TotalCount;
END

执行语句保持不变：

EXEC GenericCountAll 'T_User', 'Id';

• 使用QUOTENAME函数包裹表名和列名，避免SQL注入，同时处理包含特殊字符的标识符；
• 改用NVARCHAR(MAX)存储动态SQL，避免语句截断；
• 通过sp_executesql的输出参数获取统计结果，避免多次输出结果集；
• 添加SET NOCOUNT ON，防止存储过程默认返回影响行数的信息。

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