C#处理List集合时如何使用out参数返回多条记录的主键值

解决方案 #

out参数仅支持返回单个值，你需要用集合类型承载多条生成的主键ID，以下是两种常用实现方案，同时修正你现有代码的隐含问题：

现有代码隐患说明：Model obj = new Model() 实例化逻辑写在循环外，每次循环修改的都是同一个对象的属性，最终仓储中添加的所有记录都会指向同一个实例，属性值全部为最后一次循环的赋值，需要把实例化逻辑移到循环内部。另外原代码存在变量未声明、变量名不匹配的问题，已在示例中修正。

方案1：调整方法返回值，直接返回生成的ID集合 #

如果不需要保留原有的bool返回值约定，直接返回List<int>是最简洁的方式，返回空列表可代表操作失败：

public List<int> Add(List<Model> model)
{
    List<int> generatedIds = new List<int>();
    foreach (Model m in model)
    {
        int totalCount = _repo.AsQueryable().Count();
        int maxId = totalCount <= 0 ? 1 : _repo.AsQueryable().Max(x => x.Id) + 1;
        // 把Model实例化移到循环内
        Model obj = new Model
        {
            Id = maxId,
            Name = m.Name,
            Dept = m.Dept
        };
        _repo.Add(obj);
        generatedIds.Add(maxId);
    }
    // 可结合你的实际持久化逻辑调整，比如保存失败时返回空列表
    return generatedIds;
}

控制器侧直接接收返回的ID列表即可做后续处理。

方案2：用out参数传递集合，保留bool返回值 #

如果需要保留原有方法的bool返回约定，可以用out参数传递集合类型：

public bool Add(List<Model> model, out List<int> generatedIds)
{
    generatedIds = new List<int>();
    bool isSaved = false;
    foreach (Model m in model)
    {
        int totalCount = _repo.AsQueryable().Count();
        int maxId = totalCount <= 0 ? 1 : _repo.AsQueryable().Max(x => x.Id) + 1;
        Model obj = new Model
        {
            Id = maxId,
            Name = m.Name,
            Dept = m.Dept
        };
        _repo.Add(obj);
        generatedIds.Add(maxId);
        isSaved = true;
    }
    return isSaved;
}

控制器调用示例：

List<int> ids;
bool saveSuccess = YourService.Add(modelList, out ids);
if(saveSuccess)
{
    // 用ids做后续处理
}

优化建议 #

你当前手动算MaxId+1生成主键的逻辑存在并发风险：多请求同时调用接口时，可能会取到相同的MaxId，导致主键冲突报错。建议改用数据库自增主键，插入完成后直接取数据库返回的自增ID，安全性更高。

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