自定义CalculateWarranty函数返回异常NULL问题排查求助

问题情况 #

我写了一个标量函数CalculateWarranty用于INSERT查询，逻辑是：计算购买日期到当前日期的天数差warranty_remaining，和3年（1095天）的保修时长对比，若差值≤1095则返回该天数，否则返回NULL表示保修过期。但执行INSERT后，warranty字段一直是NULL，其他字段都正常。测试过把函数返回值改成固定1000，字段能正常显示，说明不是函数调用或类型匹配的问题，应该是逻辑或参数传递出错。

已知条件 #

• 每次修改函数前都会先删除原有标量函数实例
• warranty字段是INT类型，函数声明返回INT，无类型不匹配问题
• 主键product_id已设为自增
• 保修时长默认3年
• 实在修不好就打算直接在VALUES里计算

当前代码 #

函数代码 #

USE YourDatabaseName; -- 替换为实际数据库名

GO

CREATE FUNCTION dbo.CalculateWarranty 
    (@purchase_date DATE)
RETURNS INT
AS
BEGIN
    DECLARE @warranty_duration INT;
    DECLARE @warranty_remaining INT;

    -- 保修时长设为3年（天数）
    SET @warranty_duration = 1095;

    -- 计算剩余保修天数
    SET @warranty_remaining = DATEDIFF(DAY, @purchase_date, GETDATE());

    -- 剩余保修在3年内则返回天数，否则返回NULL
    IF @warranty_remaining <= @warranty_duration
        RETURN @warranty_remaining;

    -- 默认返回NULL（保修过期）
    RETURN NULL;        
END;   

INSERT查询代码 #

INSERT INTO product(supplier_id, category_id, serial_no,
    product_brand, product_name, purchase_price,
    purchase_date, warranty, description) 
VALUES (1, 1, 'PF-2HKJ16',
        'Lenovo', 'ThinkPad X1 Carbon 8 / i7-10510U', NULL, -- 后续补充purchase_price
        '2021-11-01',
        dbo.CalculateWarranty(CONVERT(DATE, GETDATE())), -- 调用CalculateWarranty函数
        'The ThinkPad X1 Carbon 8th generation with an Intel Core i7-10510U processor is a premium ultrabook laptop developed by Lenovo. It is designed to deliver exceptional performance, portability, and durability for professionals and business users.');

问题根源 #

1. 参数传递错误：INSERT里调用函数时，传入的是当前日期CONVERT(DATE, GETDATE())，而非要插入的购买日期'2021-11-01'！以2024年为例，DATEDIFF(DAY, '2021-11-01', GETDATE())的结果会超过1095天，此时函数会返回NULL。
2. 逻辑颠倒：函数里计算的warranty_remaining是购买日期到当前日期已经过的天数，并非实际剩余的保修天数，和预期逻辑不符。

解决办法 #

1. 修正INSERT的函数参数 #

把函数调用的参数改成实际要插入的购买日期：

INSERT INTO product(supplier_id, category_id, serial_no,
    product_brand, product_name, purchase_price,
    purchase_date, warranty, description) 
VALUES (1, 1, 'PF-2HKJ16',
        'Lenovo', 'ThinkPad X1 Carbon 8 / i7-10510U', NULL, -- 后续补充purchase_price
        '2021-11-01',
        dbo.CalculateWarranty('2021-11-01'), -- 传入实际购买日期
        'The ThinkPad X1 Carbon 8th generation with an Intel Core i7-10510U processor is a premium ultrabook laptop developed by Lenovo. It is designed to deliver exceptional performance, portability, and durability for professionals and business users.');

2. 修正函数的剩余保修计算逻辑（按需调整） #

如果要返回实际剩余的保修天数，修改函数：

USE YourDatabaseName; -- 替换为实际数据库名

GO

CREATE FUNCTION dbo.CalculateWarranty 
    (@purchase_date DATE)
RETURNS INT
AS
BEGIN
    DECLARE @warranty_duration INT = 1095;
    DECLARE @days_passed INT = DATEDIFF(DAY, @purchase_date, GETDATE());
    DECLARE @warranty_remaining INT = @warranty_duration - @days_passed;

    -- 剩余天数≥0则返回，否则返回NULL
    IF @warranty_remaining >= 0
        RETURN @warranty_remaining;

    RETURN NULL;        
END;   

3. 替代方案：直接在VALUES中计算（不用函数） #

如果不想用函数，直接在INSERT语句里完成计算：

INSERT INTO product(supplier_id, category_id, serial_no,
    product_brand, product_name, purchase_price,
    purchase_date, warranty, description) 
VALUES (1, 1, 'PF-2HKJ16',
        'Lenovo', 'ThinkPad X1 Carbon 8 / i7-10510U', NULL,
        '2021-11-01',
        CASE WHEN DATEDIFF(DAY, '2021-11-01', GETDATE()) <= 1095 THEN (1095 - DATEDIFF(DAY, '2021-11-01', GETDATE())) ELSE NULL END,
        'The ThinkPad X1 Carbon 8th generation with an Intel Core i7-10510U processor is a premium ultrabook laptop developed by Lenovo. It is designed to deliver exceptional performance, portability, and durability for professionals and business users.');

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