FreeSql中Update SetSource自动添加主键条件的问题分析

问题背景

在使用FreeSql进行数据更新操作时，开发者发现当使用Update().SetSource()方法配合Where条件时，SQL语句会自动添加上主键的条件判断，即使开发者并没有在Where条件中显式指定主键条件。

问题复现

考虑以下代码示例：

fsql.Update<Resume_Category_ModulesModel>()
    .SetSource(model)
    .Where(m => m.categoryid == model.categoryid && m.modulename == model.modulename)
    .ExecuteAffrows()>0;

生成的SQL语句会变成：

UPDATE [resume_category_modules] 
SET [categoryid] = @p_0, [modulename] = @p_1, [title] = @p_2, 
    [body] = @p_3, [visible] = @p_4, [sort] = @p_5, [data] = @p_6
WHERE ([moduleid] = 0) AND ([categoryid] = 1 AND [modulename] = N'intention')

这里可以看到，虽然开发者只指定了categoryid和modulename作为条件，但SQL中自动添加了主键moduleid的条件。

原因分析

这种现象是FreeSql的默认行为设计。当实体类中包含被标记为[Column(IsIdentity = true)]的主键属性时，FreeSql会认为这是一个需要特别处理的关键字段。

在SetSource方法中，FreeSql会尝试"智能地"确定更新条件，默认会包含主键条件，这是为了防止误操作导致大规模数据更新。这种设计在很多ORM框架中都很常见，是一种安全机制。

解决方案

如果开发者确实需要避免自动添加主键条件，可以采用以下替代方案：

1. 使用Set方法替代SetSource：

   fsql.Update<Resume_Category_ModulesModel>()
       .Set(m => m.categoryid, model.categoryid)
       .Set(m => m.modulename, model.modulename)
       // 其他字段...
       .Where(m => m.categoryid == model.categoryid && m.modulename == model.modulename)
       .ExecuteAffrows();
   

2. 使用SetDto方法：

   fsql.Update<Resume_Category_ModulesModel>()
       .SetDto(new { 
           categoryid = model.categoryid,
           modulename = model.modulename
           // 其他字段...
       })
       .Where(m => m.categoryid == model.categoryid && m.modulename == model.modulename)
       .ExecuteAffrows();
   

3. 修改实体定义（不推荐）： 如果确实不需要主键参与条件判断，可以考虑移除IsIdentity标记，但这会影响其他操作的行为，一般不推荐。

最佳实践建议

1. 理解ORM的安全机制：自动添加主键条件是一种安全措施，防止意外的大规模更新。

2. 明确更新意图：如果确实需要基于非主键条件更新数据，应该明确使用Set或SetDto方法。

3. 考虑性能影响：基于非主键条件的更新可能会影响更多行数据，需要注意性能问题。

4. 事务处理：当执行可能影响多行的更新时，考虑使用事务确保数据一致性。

总结

FreeSql的SetSource方法自动添加主键条件是其内置的安全机制，开发者可以通过使用Set或SetDto方法来避免这种行为。理解ORM框架的设计理念和安全考虑，有助于开发者更有效地使用这些工具，同时保证数据操作的安全性和准确性。