feat(templateMatcher):
- добавлен режим DryRun - изменен подход к фильтрации шаблонов по рабочей группе - перенесено из этапа основной фильтрации в этап посторение подргупп/шаблонов - при использовании тегов альтернативных рабочих групп фильтрация по списку пропускается
This commit is contained in:
@@ -18,15 +18,17 @@ internal class BuildGroupsStage : IGroupedSyncStage
|
||||
|
||||
public async Task<GroupedSyncContext> ExecuteAsync(GroupedSyncContext context, CancellationToken ct = default)
|
||||
{
|
||||
var groups = await _builder.BuildAsync(context.ReverseMapping, context.JobGroup, context.MaxJob);
|
||||
var templateGroups = await _builder.BuildAsync(
|
||||
context.ReverseMapping,
|
||||
context.JobGroup,
|
||||
context.MaxJob,
|
||||
context.UnitNames,
|
||||
ct);
|
||||
|
||||
if (!groups.Any())
|
||||
throw new GroupedSyncEarlyExitException("Нет данных после построения групп");
|
||||
|
||||
context.TemplateGroups = groups;
|
||||
context.TemplateGroups = templateGroups;
|
||||
|
||||
_logger.LogDebug("JobGroup '{JobGroupName}' ({JobGroupId}): построено {Count} групп",
|
||||
context.JobGroupName, context.JobGroupId, groups.Count);
|
||||
context.JobGroupName, context.JobGroupId, templateGroups.Count);
|
||||
|
||||
return context;
|
||||
}
|
||||
|
||||
@@ -22,6 +22,9 @@ public class GroupedSyncContext
|
||||
public List<GroupedTemplateGroup> TemplateGroups { get; set; } = new();
|
||||
public HashSet<(Guid JobId, Guid UnitId, int Index)> ExpectedTemplateKeys { get; set; } = new();
|
||||
|
||||
// Режим только для чтения — без записи в БД и отправки в MQ
|
||||
public bool DryRun { get; set; } = false;
|
||||
|
||||
/// <summary>
|
||||
/// Имена юнитов для логирования. Заполняется на этапе фильтрации.
|
||||
/// </summary>
|
||||
|
||||
@@ -16,24 +16,28 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
private readonly IUnitInValueRepository unitInValueRepository;
|
||||
private readonly IUnitFieldRepository unitFieldRepository;
|
||||
private readonly IShortcodesService shortcodesService;
|
||||
private readonly IUnitRegionalEkPtkGroupRepository regionalEkPtkGroupRepository;
|
||||
|
||||
public GroupedTemplateBuilder(
|
||||
ILogger<GroupedTemplateBuilder> logger,
|
||||
IUnitInValueRepository unitInValueRepository,
|
||||
IUnitFieldRepository unitFieldRepository,
|
||||
IShortcodesService shortcodesService
|
||||
IShortcodesService shortcodesService,
|
||||
IUnitRegionalEkPtkGroupRepository regionalEkPtkGroupRepository
|
||||
)
|
||||
{
|
||||
this.logger = logger;
|
||||
this.unitInValueRepository = unitInValueRepository;
|
||||
this.unitFieldRepository = unitFieldRepository;
|
||||
this.shortcodesService = shortcodesService;
|
||||
this.regionalEkPtkGroupRepository = regionalEkPtkGroupRepository;
|
||||
}
|
||||
|
||||
public async Task<List<GroupedTemplateGroup>> BuildAsync(
|
||||
Dictionary<Guid, List<Guid>> initialReverseMapping,
|
||||
JobGroup jobGroup,
|
||||
Job maxJob,
|
||||
Dictionary<Guid, string> unitNames,
|
||||
CancellationToken ct = default)
|
||||
{
|
||||
logger.LogDebug("Начало построения структуры групп для JobGroup {JobGroupId}.", jobGroup.Id);
|
||||
@@ -42,7 +46,6 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
return new List<GroupedTemplateGroup>();
|
||||
|
||||
// 1. Определяем стратегию внутренней группировки
|
||||
// Если IsGroupByResponsible != true, используем WorkGroupMask через ShortcodesService
|
||||
bool useWorkGroupMask = jobGroup.IsGroupByResponsible != true;
|
||||
|
||||
logger.LogDebug("Стратегия внутренней группировки: {Strategy}",
|
||||
@@ -102,22 +105,50 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
|
||||
if (useWorkGroupMask)
|
||||
{
|
||||
// Используем ShortcodesService для получения финальных значений WorkGroupMask
|
||||
// Определяем, нужно ли выполнять фильтрацию по списку рабочих групп ПТК.
|
||||
// Если маска содержит тег с "_в_ЗО_РГ", фильтрация не выполняется.
|
||||
var skipRegionalFilter = maxJob.WorkGroupMask.Contains("_в_ЗО_РГ", StringComparison.OrdinalIgnoreCase);
|
||||
|
||||
HashSet<string>? allowedWorkGroupSet = null;
|
||||
|
||||
if (!skipRegionalFilter)
|
||||
{
|
||||
// Загружаем разрешённые значения из regionalEkPtkGroupRepository
|
||||
var allowedWorkGroupValues = await regionalEkPtkGroupRepository.Get()
|
||||
.AsNoTracking()
|
||||
.Include(g => g.FieldValue)
|
||||
.Where(g => g.FieldValue != null && g.FieldValue.Value != null)
|
||||
.Select(g => g.FieldValue!.Value!)
|
||||
.ToListAsync(ct);
|
||||
|
||||
allowedWorkGroupSet = new HashSet<string>(allowedWorkGroupValues, StringComparer.OrdinalIgnoreCase);
|
||||
|
||||
logger.LogDebug("Загружено {Count} разрешённых значений рабочих групп из RegionalEkPtkGroup.", allowedWorkGroupSet.Count);
|
||||
}
|
||||
else
|
||||
{
|
||||
logger.LogDebug("Маска содержит тег с '_в_ЗО_РГ'. Фильтрация по списку рабочих групп ПТК пропущена.");
|
||||
}
|
||||
|
||||
// Применяем WorkGroupMask для каждого юнита отдельно
|
||||
unitIdToGroupingValueMap = new Dictionary<Guid, string>();
|
||||
|
||||
foreach (var potentialUnitId in reverseMapping.Keys)
|
||||
{
|
||||
var relatedUnitIds = reverseMapping[potentialUnitId].Select(e => e.UnitId).Distinct().ToList();
|
||||
var relatedUnitIds = reverseMapping[potentialUnitId]
|
||||
.Select(e => e.UnitId)
|
||||
.Distinct()
|
||||
.ToList();
|
||||
|
||||
foreach (var relatedUnitId in relatedUnitIds)
|
||||
{
|
||||
// Создаем временный Template для применения шорткодов
|
||||
// Формируем шаблон с одним связанным юнитом
|
||||
var tempTemplate = new Template
|
||||
{
|
||||
Id = Guid.NewGuid(),
|
||||
Name = "temp",
|
||||
JobId = maxJob.Id,
|
||||
UnitId = potentialUnitId, // Родительский юнит шаблона
|
||||
UnitId = potentialUnitId,
|
||||
Job = maxJob,
|
||||
UnitsInTemplate = new List<UnitsInTemplate>
|
||||
{
|
||||
@@ -130,6 +161,27 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
tempTemplate,
|
||||
nameof(GroupedTemplateBuilder));
|
||||
|
||||
if (string.IsNullOrEmpty(workGroupValue))
|
||||
{
|
||||
logger.LogDebug(
|
||||
"Юнит {UnitId} исключён: маска вернула пустую строку.",
|
||||
relatedUnitId);
|
||||
continue;
|
||||
}
|
||||
|
||||
// Фильтрация по списку рабочих групп ПТК выполняется только если не используется тег с "_в_ЗО_РГ"
|
||||
if (!skipRegionalFilter && !allowedWorkGroupSet!.Contains(workGroupValue))
|
||||
{
|
||||
logger.LogDebug( // Изменено с LogDebug на LogInformation для тестирования
|
||||
"[FILTER] Юнит {UnitId} ИСКЛЮЧЁН: рабочая группа '{WorkGroup}' не найдена в списке разрешённых ({AllowedCount} значений).",
|
||||
relatedUnitId, workGroupValue, allowedWorkGroupSet!.Count);
|
||||
continue;
|
||||
}
|
||||
|
||||
logger.LogDebug( // Добавляем лог для успешного прохождения
|
||||
"[FILTER] Юнит {UnitId} ПРОШЁЛ: рабочая группа '{WorkGroup}' для PotentialUnit {PotentialUnitId}.",
|
||||
relatedUnitId, workGroupValue, potentialUnitId);
|
||||
|
||||
unitIdToGroupingValueMap[relatedUnitId] = workGroupValue;
|
||||
}
|
||||
}
|
||||
@@ -163,7 +215,18 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
var potentialUnitId = kvp.Key;
|
||||
var unitsInTemplateForThisPotentialUnitId = kvp.Value;
|
||||
|
||||
var innerGroupedUnits = unitsInTemplateForThisPotentialUnitId
|
||||
// Фильтруем записи: оставляем только те, которые прошли проверку
|
||||
var filteredUnits = unitsInTemplateForThisPotentialUnitId
|
||||
.Where(entry => unitIdToGroupingValueMap.ContainsKey(entry.UnitId))
|
||||
.ToList();
|
||||
|
||||
if (!filteredUnits.Any())
|
||||
{
|
||||
logger.LogDebug("PotentialUnitId {PotentialUnitId} исключён: нет юнитов после фильтрации.", potentialUnitId);
|
||||
continue;
|
||||
}
|
||||
|
||||
var innerGroupedUnits = filteredUnits
|
||||
.GroupBy(entry => unitIdToGroupingValueMap.GetValueOrDefault(entry.UnitId, "Нет данных"))
|
||||
.OrderBy(g => g.Key, StringComparer.Ordinal)
|
||||
.ToList();
|
||||
@@ -179,21 +242,23 @@ internal class GroupedTemplateBuilder : IGroupedTemplateBuilder
|
||||
|
||||
var unitsInInnerGroup = innerGroup.ToList();
|
||||
|
||||
var splitSubGroups = unitsInInnerGroup
|
||||
// Сортируем по имени юнита, потом по UnitFieldValueId
|
||||
var sortedUnitsInInnerGroup = unitsInInnerGroup
|
||||
.OrderBy(e => unitNames.GetValueOrDefault(e.UnitId, e.UnitId.ToString()))
|
||||
.ThenBy(e => e.UnitFieldValueId)
|
||||
.ToList();
|
||||
|
||||
var splitSubGroups = sortedUnitsInInnerGroup
|
||||
.Select((entry, index) => new { entry, groupIndex = index / maxValueForSplitting })
|
||||
.GroupBy(x => x.groupIndex)
|
||||
.Select(g => g.Select(x => x.entry).ToList())
|
||||
.ToList();
|
||||
|
||||
// ВАЖНО: добавляем подгруппы в subGroups
|
||||
foreach (var subGroupEntries in splitSubGroups)
|
||||
{
|
||||
var sortedEntries = subGroupEntries
|
||||
.OrderBy(e => e.UnitId)
|
||||
.ThenBy(e => e.UnitFieldValueId)
|
||||
.ToList();
|
||||
|
||||
subGroups.Add(new GroupedTemplateSubGroup(
|
||||
Entries: sortedEntries,
|
||||
Entries: subGroupEntries,
|
||||
InnerGroupName: innerGroupName,
|
||||
GlobalIndex: globalIndex
|
||||
));
|
||||
|
||||
@@ -9,19 +9,13 @@ internal class GroupedTemplateUnitFilter : IGroupedTemplateUnitFilter
|
||||
{
|
||||
private readonly ILogger<GroupedTemplateUnitFilter> logger;
|
||||
private readonly IUnitInValueRepository unitInValueRepository;
|
||||
private readonly IUnitRegionalEkPtkGroupRepository regionalEkPtkGroupRepository;
|
||||
private readonly IUnitFieldRepository unitFieldRepository;
|
||||
|
||||
public GroupedTemplateUnitFilter(
|
||||
ILogger<GroupedTemplateUnitFilter> logger,
|
||||
IUnitInValueRepository unitInValueRepository,
|
||||
IUnitRegionalEkPtkGroupRepository regionalEkPtkGroupRepository,
|
||||
IUnitFieldRepository unitFieldRepository)
|
||||
IUnitInValueRepository unitInValueRepository)
|
||||
{
|
||||
this.logger = logger;
|
||||
this.unitInValueRepository = unitInValueRepository;
|
||||
this.regionalEkPtkGroupRepository = regionalEkPtkGroupRepository;
|
||||
this.unitFieldRepository = unitFieldRepository;
|
||||
}
|
||||
|
||||
public async Task<List<UnitFilterResultDto>> FilterAsync(
|
||||
@@ -47,53 +41,24 @@ internal class GroupedTemplateUnitFilter : IGroupedTemplateUnitFilter
|
||||
var groupingFieldId = jobGroup.GroupingUnitFieldId.Value;
|
||||
logger.LogDebug("Фильтрация по GroupingUnitFieldId (FieldId={FieldId}).", groupingFieldId);
|
||||
|
||||
// 2. Фильтрация по GroupingUnitFieldId
|
||||
// 2. Загружаем значения поля группировки для всех юнитов
|
||||
var allUnitIds = unitsList.Select(u => u.Id).ToList();
|
||||
var groupingValues = await unitInValueRepository.GetByUnitIdsAndFieldIdsAsync(allUnitIds, new HashSet<Guid> { groupingFieldId });
|
||||
var groupingValues = await unitInValueRepository.GetByUnitIdsAndFieldIdsAsync(
|
||||
allUnitIds, new HashSet<Guid> { groupingFieldId }, ct);
|
||||
|
||||
// 3. Определяем юниты, у которых есть непустое значение поля группировки
|
||||
var validUnitIdsAfterGrouping = groupingValues
|
||||
.Where(uv => uv.Value != null && !string.IsNullOrEmpty(uv.Value.Value))
|
||||
.Select(uv => uv.UnitId)
|
||||
.ToHashSet();
|
||||
|
||||
// 4. Фильтруем исходный список
|
||||
var filteredByGrouping = unitsList
|
||||
.Where(u => validUnitIdsAfterGrouping.Contains(u.Id))
|
||||
.ToList();
|
||||
|
||||
logger.LogDebug("После фильтрации по GroupingUnitFieldId осталось {Count} юнитов.", filteredByGrouping.Count);
|
||||
|
||||
if (!filteredByGrouping.Any())
|
||||
{
|
||||
return filteredByGrouping;
|
||||
}
|
||||
|
||||
// 3. Фильтрация по РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК
|
||||
var workGroupField = await unitFieldRepository.GetByAihitNameAsync("РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК")
|
||||
?? throw new InvalidOperationException("Поле 'РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК' не найдено в справочнике полей.");
|
||||
|
||||
var workGroupFieldId = workGroupField.Id;
|
||||
logger.LogDebug("Фильтрация по полю 'РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК' (FieldId={FieldId}).", workGroupFieldId);
|
||||
|
||||
var filteredUnitIds = filteredByGrouping.Select(u => u.Id).ToList();
|
||||
var workGroupValues = await unitInValueRepository.GetByUnitIdsAndFieldIdsAsync(filteredUnitIds, new HashSet<Guid> { workGroupFieldId });
|
||||
|
||||
var allowedValueIds = regionalEkPtkGroupRepository.Get()
|
||||
.Select(g => g.FieldValueId)
|
||||
.ToHashSet();
|
||||
|
||||
logger.LogDebug("Найдено {Count} разрешенных значений для поля 'РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК'.", allowedValueIds.Count);
|
||||
|
||||
var validUnitIdsAfterWorkGroup = workGroupValues
|
||||
.Where(uv => uv.Value != null && allowedValueIds.Contains(uv.Value.Id))
|
||||
.Select(uv => uv.UnitId)
|
||||
.ToHashSet();
|
||||
|
||||
var finalFiltered = filteredByGrouping
|
||||
.Where(u => validUnitIdsAfterWorkGroup.Contains(u.Id))
|
||||
.ToList();
|
||||
|
||||
logger.LogDebug("После фильтрации по 'РАБОЧАЯ_ГР_ОТВ_ЗА_ЭК' осталось {Count} юнитов.", finalFiltered.Count);
|
||||
|
||||
return finalFiltered;
|
||||
return filteredByGrouping;
|
||||
}
|
||||
}
|
||||
@@ -15,5 +15,6 @@ public interface IGroupedTemplateBuilder
|
||||
Dictionary<Guid, List<Guid>> initialReverseMapping,
|
||||
JobGroup jobGroup,
|
||||
Job maxJob,
|
||||
Dictionary<Guid, string> unitNames,
|
||||
CancellationToken ct = default);
|
||||
}
|
||||
Reference in New Issue
Block a user