C#中Foreach与查询性能优化问题求助

看起来你这段生成全国企业对比得分的代码遇到了性能瓶颈，结合你的代码片段和这类场景的常见问题，我给你梳理几个实用的优化方向：

1. 减少数据库查询次数（最关键的优化点） #

你当前的写法是先查Visit，再通过导航属性取Company，如果EF上下文默认开启延迟加载的话，这会触发两次数据库查询（一次查Visit，一次查关联的Company）。如果后面还要用到MainEconomicCNAE，还会触发第三次查询——这种N+1查询是性能杀手。

建议用Include和ThenInclude一次性加载所有需要的关联数据，把多次数据库往返变成一次：

var visit = Context.Visit
    .Include(v => v.Company)
    .ThenInclude(c => c.MainEconomicCNAE)
    .FirstOrDefault(v => v.Id == VisitId);

if (visit == null)
{
    // 处理Visit不存在的情况，避免空引用
    return new List<object>();
}

另外，Find方法虽然会先查缓存，但对于单条查询，用FirstOrDefault配合Include更可控，也能明确加载关联数据。

2. 避免不必要的对象创建与装箱开销 #

你返回的是List<object>，这意味着每个得分项都会发生装箱操作（如果是值类型的话），而且代码可读性差，后续维护也麻烦。建议定义一个强类型的实体类来存储得分结果：

// 定义强类型实体
public class EnterpriseComparisonScore
{
    public string Theme { get; set; }
    public decimal Score { get; set; }
    // 其他需要的字段，比如企业ID、CNAE分类等
}

// 修改方法返回类型
public List<EnterpriseComparisonScore> GetCNAEBRCycleChart(int VisitId)
{
    var result = new List<EnterpriseComparisonScore>();
    // 后续逻辑直接添加强类型对象
    result.Add(new EnterpriseComparisonScore { Theme = "Finance", Score = 85.5m });
    return result;
}

这样不仅消除了装箱拆箱的性能损耗，代码的可读性和可维护性也会大幅提升。

3. 优化固定资源的复用 #

你的Themes数组是在方法内部初始化的，如果这个主题列表是固定不变的，完全可以把它定义成类的静态只读字段，避免每次调用方法都重新创建数组：

private static readonly string[] _themes = new[] { "Finance", "Market", /* 剩余主题 */ };

public List<EnterpriseComparisonScore> GetCNAEBRCycleChart(int VisitId)
{
    // 直接使用_themes即可
}

4. 缓存计算结果（如果数据不是实时更新的） #

如果企业的CNAE分类和对比得分不会频繁变化（比如一天更新一次），可以把计算好的结果缓存起来，避免重复计算和数据库查询。比如用.NET自带的MemoryCache：

private readonly IMemoryCache _cache;

// 通过构造函数注入缓存
public YourService(IMemoryCache cache)
{
    _cache = cache;
}

public List<EnterpriseComparisonScore> GetCNAEBRCycleChart(int VisitId)
{
    var visit = Context.Visit
        .Include(v => v.Company)
        .ThenInclude(c => c.MainEconomicCNAE)
        .FirstOrDefault(v => v.Id == VisitId);
    
    if (visit == null) return new List<EnterpriseComparisonScore>();
    
    var cnae = visit.Company.MainEconomicCNAE.IdentifyCNAE;
    var cacheKey = $"EnterpriseComparison_{VisitId}_{cnae}";
    
    // 先查缓存
    if (_cache.TryGetValue(cacheKey, out List<EnterpriseComparisonScore> cachedResult))
    {
        return cachedResult;
    }
    
    // 执行你的得分计算逻辑...
    var result = new List<EnterpriseComparisonScore>();
    // 填充result的代码
    
    // 存入缓存，设置过期时间
    _cache.Set(cacheKey, result, new MemoryCacheEntryOptions
    {
        SlidingExpiration = TimeSpan.FromHours(24),
        AbsoluteExpirationRelativeToNow = TimeSpan.FromDays(1)
    });
    
    return result;
}

5. 用性能分析工具定位瓶颈 #

如果做完上面的优化还是有性能问题，建议用Visual Studio自带的性能探查器（Debug -> Performance Profiler）来跑一下，它能帮你精准定位是数据库查询慢，还是内存中的计算逻辑耗时，或者是循环操作拖慢了速度——毕竟没有工具的话，优化都是凭经验猜，效率很低。

内容的提问来源于stack exchange，提问作者Lucas Hipólito