关于客户ID输入SQL查询的正确性及三位输入限制的技术咨询

一、现有查询的规范问题及修正 #

首先看你的现有代码，有一个明显的无效语句：SET @CID = @CID —— 因为DECLARE @CID int;之后变量默认是NULL，这条语句执行后变量依然是NULL，这会导致你的IF @CID > 0判断永远不成立（除非你手动替换这条语句赋值）。

如果是要接收用户输入的客户ID，更规范的做法是创建存储过程来封装逻辑，这样可以更安全地接收参数，也符合SQL的最佳实践。修正后的代码示例：

USE WideWorldImporters
GO

CREATE PROCEDURE GetCustomerTotalTransactions
    @CID int
AS
BEGIN
    -- 先做参数校验
    IF @CID IS NULL OR @CID < 100
    BEGIN
        PRINT 'Invalid ID'
        RETURN
    END

    -- 查询逻辑
    SELECT 
        CustomerID, 
        SUM(TransactionAmount) AS TotalAmount 
    FROM Sales.CustomerTransactions 
    WHERE CustomerID = @CID 
    GROUP BY CustomerID
END
GO

调用的时候直接传入参数即可：

EXEC GetCustomerTotalTransactions @CID = 123; -- 有效ID
EXEC GetCustomerTotalTransactions @CID = 99;  -- 会输出Invalid ID

如果暂时不想用存储过程，只是脚本形式，那也要确保变量被正确赋值，比如把用户输入的ID传给@CID，而不是写SET @CID = @CID这种无效语句。

二、如何限制用户输入的ID至少为三位数字 #

三位数字的最小有效值是100，所以只需要在参数校验的条件里加上@CID < 100的判断即可：

• 当ID是0、负数、或者小于100的正数时，都判定为无效
• 只有当ID >=100时，才执行查询

另外还要考虑NULL的情况，因为用户可能没有输入任何值，所以校验条件最好包含@CID IS NULL，避免空值导致的异常。

额外的优化建议 #

• 如果查询的客户ID不存在，当前代码会返回空结果集，你可以考虑加上判断，比如用IF EXISTS先检查是否有该客户的交易记录，再返回结果或者提示“Customer not found”
• 对于聚合查询SUM(TransactionAmount)，如果客户没有交易记录，会返回NULL，可以用ISNULL(SUM(TransactionAmount), 0)把结果转为0，更友好

优化后的查询示例（存储过程里的逻辑）：

IF @CID IS NULL OR @CID < 100
BEGIN
    PRINT 'Invalid ID'
    RETURN
END

IF NOT EXISTS(SELECT 1 FROM Sales.CustomerTransactions WHERE CustomerID = @CID)
BEGIN
    PRINT 'Customer not found'
    RETURN
END

SELECT 
    CustomerID, 
    ISNULL(SUM(TransactionAmount), 0) AS TotalAmount 
FROM Sales.CustomerTransactions 
WHERE CustomerID = @CID 
GROUP BY CustomerID

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