使用EF从DB查询与List内名称匹配的数据，原有for循环方案失效如何解决？

原有代码问题分析 #

1. 闭包变量捕获异常：EF的表达式树解析时捕获的是循环变量i的引用而非当前迭代值，等到异步查询实际执行时i已经自增到_promoNames.Count，会触发索引越界。
2. N+1性能问题：每次循环都发起独立的数据库查询，列表元素较多时会产生大量无效IO，性能极差。

最优实现方案 #

推荐使用EF原生支持的Contains方法批量查询，仅需1次数据库请求即可获取所有匹配数据：

// 直接查询所有匹配的JsonParams字段，减少不必要的数据传输
var matchedJsonParams = await _mainDbContext.PromoCode
    .Where(prc => _promoNames.Contains(prc.Code))
    .Select(prc => prc.JsonParams)
    .ToListAsync();

_promoJsonParams.AddRange(matchedJsonParams);

场景适配补充 #

• 如果你需要最终结果的顺序和_promoNames的原始顺序一致，可以先查询为字典再匹配：

// 先构造Code-JsonParams的映射字典
var promoMap = await _mainDbContext.PromoCode
    .Where(prc => _promoNames.Contains(prc.Code))
    .ToDictionaryAsync(prc => prc.Code, prc => prc.JsonParams);

// 按原始列表顺序添加结果，不存在的Code可以自行处理跳过/报错
foreach (var name in _promoNames)
{
    if (promoMap.TryGetValue(name, out var jsonParam))
    {
        _promoJsonParams.Add(jsonParam);
    }
}

• 如果一定要保留循环写法，只需在循环内部声明局部变量接收当前索引即可修复闭包问题：

for (var i = 0; i < _promoNames.Count; i++)
{
    var currentIdx = i; // 局部变量不会被多次迭代共享
    var promo = await _mainDbContext.PromoCode
        .Where(prc => prc.Code == _promoNames[currentIdx])
        .SingleAsync();
    _promoJsonParams.Add(promo.JsonParams);
}

注：如果存在Code不在数据库中的情况，SingleAsync会抛出异常，可根据业务需要替换为SingleOrDefaultAsync做兼容处理。

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