SQL自定义函数内插入@OBV值到数据表的正确实现方法

SQL Server 的*用户自定义函数（UDF）*有严格的只读限制：禁止执行任何修改数据库状态的操作，包括对用户表的 INSERT/UPDATE/DELETE 写入，这就是你代码中 INSERT 语句执行报错的核心原因。
另外代码还有一处隐藏错误：你定义函数返回char(4)类型，但实际返回的@OBV是money类型，类型不匹配也会触发执行错误。

根据你的使用场景，可以选择以下两种常用实现方式：

方案1：改用存储过程实现（最推荐） #

如果需要同时完成OBV计算、插入表、返回计算结果的需求，存储过程是标准实现方式：

CREATE PROCEDURE CalculateOnBalanceVolume
    @operation varchar(3),
    @volume money,
    @CalculatedOBV money OUTPUT
AS
BEGIN
    DECLARE @prevOBV money, @OBV money
    -- 取最新的OBV值，ISNULL处理表为空时的默认值0
    SELECT TOP 1 @prevOBV = OnBalanceVolume 
    FROM OnBalanceVolume 
    ORDER BY EventTime DESC
    SET @prevOBV = ISNULL(@prevOBV, 0)

    IF @operation = 'add'
        SET @OBV = @prevOBV + @volume
    ELSE IF @operation = 'sub'
        SET @OBV = @prevOBV - @volume

    -- 明确指定插入列名，避免表结构变更后执行出错
    INSERT INTO OBVTable (OBV列名) VALUES (@OBV)

    SET @CalculatedOBV = @OBV
END

调用示例：

DECLARE @ResultOBV money
EXEC CalculateOnBalanceVolume 'add', 1000, @ResultOBV OUTPUT
SELECT @ResultOBV AS FinalOBV

方案2：拆分计算和写入逻辑 #

如果必须保留自定义函数的实现，可以把计算和插入操作拆分执行：

第一步：创建仅做计算的纯函数 #

ALTER FUNCTION CalculateOnBalanceVolume (
    @operation varchar(3),
    @volume money
)
RETURNS money -- 修正返回值类型和实际返回结果匹配
AS
BEGIN
    DECLARE @prevOBV money, @OBV money
    SELECT TOP 1 @prevOBV = OnBalanceVolume 
    FROM OnBalanceVolume 
    ORDER BY EventTime DESC
    SET @prevOBV = ISNULL(@prevOBV, 0)

    IF @operation = 'add'
        SET @OBV = @prevOBV + @volume
    ELSE IF @operation = 'sub'
        SET @OBV = @prevOBV - @volume

    RETURN @OBV 
END

第二步：调用函数后单独执行插入 #

DECLARE @CalculatedOBV money = dbo.CalculateOnBalanceVolume('add', 1000)
INSERT INTO OBVTable VALUES (@CalculatedOBV)

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