创建SQL存储过程调试错误日志表的方案可行性及最佳实践咨询

这个方案非常合理，是数据库侧业务操作异常兜底的行业常规实践，既能保证生产数据的事务一致性，又能留存完整的异常上下文，比单纯把错误抛给应用层更方便问题定位。

1. 错误日志表结构优化 #

• 字段长度和类型不合理：当前ErrorMessage仅为varchar(50)，完全不足以承载SQL系统报错和自定义业务报错的内容，极易出现截断丢失关键信息的问题，建议改为nvarchar(2000)或nvarchar(max)；ErrorLine、ErrorNumber、ErrorSeverity、ErrorState本身是系统返回的整数类型，无需存为字符串，改为对应int类型即可，既节省存储空间也避免类型转换问题。
• 时间字段优化：若使用SQL Server 2016及以上版本，ErrorDate可改为datetime2(3)获得更高时间精度，默认值改用GETUTCDATE()适配跨时区部署场景。

2. 存储过程逻辑优化 #

• 业务校验逻辑位置错误：当前你把业务规则校验放在了TRY块外部，RAISERROR抛出的报错不会被CATCH捕获，触发业务规则错误时根本不会写入错误日志，直接就抛出到上层，这是核心逻辑缺陷。建议把业务校验逻辑移到TRY块内部，同时SQL Server 2012及以上版本推荐用THROW代替RAISERROR，无需手动指定严重级别和状态码，使用更简单。
• 事务回滚增加判断：执行ROLLBACK前先判断XACT_STATE() <> 0，避免事务未开启/已自动回滚时，执行回滚语句产生二次错误覆盖真实报错。
• 错误日志写入逻辑抽象：把写入ErrorLog的逻辑封装为通用存储过程（比如usp_ErrorLog_Write），所有业务存储过程的CATCH块直接调用即可，避免重复写大量INSERT代码，后续调整日志字段也只需修改一次通用存储过程。
  优化后的存储过程核心逻辑示例：

BEGIN TRY
    DECLARE @BusinessRule_1_Fail int = (SELECT IIF(@A < 3, 1, 0))
    DECLARE @BusinessRule_2_Fail int = (SELECT IIF(@C < 7, 1, 0))

    IF @BusinessRule_1_Fail = 1
        THROW 50001, 'Rule #1 failed because XYZ', 1;
    ELSE IF @BusinessRule_2_Fail = 1
        THROW 50002, 'Rule #2 failed because ZYX', 1;

    BEGIN TRANSACTION
        INSERT INTO [INV].[Table_A] (ColA, ColB, ColC, ColD)
        VALUES (@A, @B, @C, @D)
    COMMIT TRANSACTION
END TRY
BEGIN CATCH
    IF XACT_STATE() <> 0
        ROLLBACK TRANSACTION;

    -- 调用通用写日志存储过程
    EXEC usp_ErrorLog_Write 
        @ProcName = '[dbo].[proc_xxx]',
        @InputParams = @InputParams -- 拼接好的参数字符串
END CATCH

你计划新增大字段存储格式化参数的思路完全可行，生产环境落地建议如下：

• 字段类型优先选nvarchar(max)，无需担心长度不足问题，SQL Server对大字段的存储优化已经非常成熟，不会产生额外性能开销。
• 优先用JSON格式序列化参数，不要自定义分隔符格式：SQL Server 2016及以上版本自带JSON序列化能力，无需引入额外工具，拼接逻辑示例如下：

DECLARE @InputParams NVARCHAR(MAX) = (
    SELECT @A AS A, @B AS B, @C AS C, @D AS D
    FOR JSON PATH, WITHOUT_ARRAY_WRAPPER
)

生成的参数格式为{"A":10,"B":2,"C":323,"D":42}，后续排查时可以直接用JSON_VALUE函数提取指定参数值，比自定义分隔符拆分效率高很多。

• 低版本不支持JSON的场景可以用XML格式序列化参数，同样比自定义分隔符更易解析。
• 若要减少重复代码，可以写一个小工具查询sys.parameters系统表，自动生成每个存储过程的参数序列化代码，无需手动编写。

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