var attr = (ExcelObjectAttribute)ProgramModelEntities[0]?.GetType() ?.GetCustomAttributes(typeof(ExcelObjectAttribute), false) ?.FirstOrDefault(); List<Guid> ids = _coreRepository.Context.Set<CProgramme>().Select(i => i.Id).ToList(); List<String> errorIds = new(); bool successFlag = true; foreach (var cProgramme in Programmes) { if (ids.Contains(cProgramme.Id)) { successFlag = false; errorIds.Add(cProgramme.Id.ToString()); _errorLogService.SaveErrorLog(cProgramme.Id.ToString(), "Program Id already exists", attr.WokeSheetName); continue; } else { ids.Add(cProgramme.Id); } var exist = _coreRepository.Context.Set<CProgramme>().Where(s => s.Id != cProgramme.Id && s.Name.ToLower() == cProgramme.Name.ToLower() && s.SchoolId == cProgramme.SchoolId).Select(i => i.Id).ToList(); if (exist.Count > 0) { errorIds.Add(cProgramme.Id.ToString()); _errorLogService.SaveErrorLog(cProgramme.Id.ToString(), "Program name already exists", attr.WokeSheetName); successFlag = false; } } return successFlag; 优化一下这段代码

可以尝试以下优化：

1. 使用HashSet代替List，可以提高ids的查找速度。

2. 使用LINQ的Any方法代替ToList+Count，可以提高exist的查询速度。

3. 将错误日志的保存放到一个集合中，在循环结束后一次性保存，可以减少数据库操作。

优化后的代码如下：

var attr = (ExcelObjectAttribute)ProgramModelEntities[0]?.GetType()
    ?.GetCustomAttributes(typeof(ExcelObjectAttribute), false)
    ?.FirstOrDefault();

HashSet<Guid> ids = new(_coreRepository.Context.Set<CProgramme>().Select(i => i.Id));
List<string> errorLogs = new();

foreach (var cProgramme in Programmes)
{
    if (ids.Contains(cProgramme.Id))
    {
        errorLogs.Add($"{cProgramme.Id}: Program Id already exists");
        continue;
    }

    if (_coreRepository.Context.Set<CProgramme>().Any(s =>
        s.Id != cProgramme.Id &&
        s.Name.ToLower() == cProgramme.Name.ToLower() &&
        s.SchoolId == cProgramme.SchoolId))
    {
        errorLogs.Add($"{cProgramme.Id}: Program name already exists");
        continue;
    }

    ids.Add(cProgramme.Id);
}

if (errorLogs.Any())
{
    _errorLogService.SaveErrorLogs(errorLogs, attr.WokeSheetName);
    return false;
}

return true;