C#使用SQL声明变量实现参数化查询：故障分类表插入、主键生成及并发问题

执行层问题适配 #

• 关于返回值获取：你当前用ExecuteNonQuery拿不到结果的核心原因是，这个方法的设计用途是执行无返回结果集的SQL（比如纯插入、更新、删除），返回值仅代表SQL执行影响的行数，比如你执行插入后这个方法会返回1（代表插了1行），根本不会读取SELECT语句输出的内容。你这条SQL最后有SELECT @id + 1的标量输出，直接改用ExecuteScalar方法即可，该方法会自动读取结果集的第一行第一列内容，完全匹配你取新生成id的需求。
• 关于参数化适配：你不需要把SQL内部的@id做成C#端的外部参数——这个变量是数据库内部基于表数据计算出来的可信值，不存在注入风险。你需要做参数化的只有从程序传入的动态业务值fault_catagory（例如设参数名@FaultCategory），把这个占位符嵌到SQL的VALUES部分即可，完全满足防注入的安全要求，不需要改动原有SQL的内部逻辑。

并发风险的根本性优化 #

你当前手动取max(主键)+1的方案，在多用户并发场景下存在本质缺陷：两个并发会话可能同时查询到相同的最大id值，最终导致插入时撞主键失败，仅靠单条SQL语句无法完全规避这个问题，推荐两个层级的优化方案：

• 最优方案：直接将fault_catagory_ident字段配置为数据库自增主键（IDENTITY列）。配置完成后，你完全不需要手动计算下一个id值，插入操作时不需要给这个字段传值，数据库会自动生成全局唯一的不重复id。插入完成后，用SCOPE_IDENTITY()函数获取当前会话插入生成的主键值返回给C#即可，该函数仅返回当前会话、当前作用域内生成的id值，不会被其他并发操作干扰，安全性远高于手动计算max+1的逻辑。
• 历史遗留兼容方案：如果因为线上表结构已经固化，无法改动成自增列，就需要把原有的max+1逻辑调整为带表锁保护的事务操作，彻底锁断并发带来的id重复问题，参考SQL写法：

BEGIN TRANSACTION
DECLARE @nextId int
SELECT @nextId = ISNULL(MAX(fault_catagory_ident), 0) + 1 
FROM fault_catagory_list WITH (UPDLOCK, HOLDLOCK)
INSERT INTO fault_catagory_list (fault_catagory_ident, fault_catagory)
VALUES (@nextId, @FaultCategory)
SELECT @nextId
COMMIT TRANSACTION

其中WITH (UPDLOCK, HOLDLOCK)会在查询最大id时就给目标表加排他锁，其他并发的同类查询必须等当前事务提交后才能执行，从机制上避免id重复。

C#端代码适配参考 #

核心逻辑不需要复杂改动，重点注意两点：

• 所有动态传入的业务值必须通过参数对象传入，不要直接拼接SQL字符串
• 拿到SQL返回的id值时直接用ExecuteScalar，不要用ExecuteNonQuery
  示例核心代码片段：

using var conn = new SqlConnection("数据库连接字符串");
conn.Open();
// 这里放优化后的SQL语句，其中@FaultCategory是占位的参数
var cmd = new SqlCommand(优化后的安全SQL, conn);
// 把程序里拿到的分类值传入参数，完成参数化
cmd.Parameters.AddWithValue("@FaultCategory", 你的业务侧分类值);
// 直接读取SQL返回的新插入的id值
var newInsertedId = (int)cmd.ExecuteScalar();

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