refactor(templateDistributor): добавлены комментарии

PARR.TemplateDistributor/TemplateDistributor.cs

@@ -13,7 +13,6 @@ namespace PARR.TemplateDistributor
         private readonly ILogger<TemplateDistributor> logger;
         private readonly ITemplateService templateService;
         private readonly IApplicationsInWorkService applicationsInWorkService;
-        private readonly IEsppSchTypeScheduleService esppSchTypeScheduleService;
         private readonly ICalendarService calendarService;
         private readonly IEsppScheduleTransformService esppScheduleTransformService;
         private readonly IWeekendDayService weekendDayService;
@@ -22,7 +21,6 @@ namespace PARR.TemplateDistributor
             ILogger<TemplateDistributor> logger,
             ITemplateService templateService,
             IApplicationsInWorkService applicationsInWorkService,
-            IEsppSchTypeScheduleService esppSchTypeScheduleService,
             ICalendarService calendarService,
             IEsppScheduleTransformService esppScheduleTransformService,
             IWeekendDayService weekendDayService
@@ -31,7 +29,6 @@ namespace PARR.TemplateDistributor
             this.logger = logger;
             this.templateService = templateService;
             this.applicationsInWorkService = applicationsInWorkService;
-            this.esppSchTypeScheduleService = esppSchTypeScheduleService;
             this.calendarService = calendarService;
             this.esppScheduleTransformService = esppScheduleTransformService;
             this.weekendDayService = weekendDayService;
@@ -40,57 +37,45 @@ namespace PARR.TemplateDistributor
 
         public async Task UpdateScheduleAsync(Guid applicationInWorkId)
         {
-            // все шаблоны по applicationInWorkId
-            // группирует по РР
-            // -> DistributeTemplateForPeriodAsync
-            // сохранить в БД
-            //applicationInWorkId = Guid.Parse("bdfefd77-bce3-4f62-a484-5042c06f4467");
-
             var appInWork = await applicationsInWorkService.GetAsync(applicationInWorkId);
-            //                                                .Include(aiw => aiw.EsppSchValues)
-            //                                                    .ThenInclude(esv => esv.EsppSchTypeValue)
-            //                                                    .ThenInclude(etv => etv!.DistributionPeriod)
-            //                                                .FirstOrDefaultAsync(t => t.Id == applicationInWorkId);
 
             var templates = await templateService.Get()
                                                     .Include(t => t.Host)
                                                         .ThenInclude(h => h.WorkGroup)
-                                                    .Where(t => t.ApplicationInWorkId == applicationInWorkId)
+                                                    .Where(t => t.ApplicationInWorkId == applicationInWorkId && t.Host!.WorkGroupId != null)
                                                     .ToListAsync();
 
-
+            //Группируем шаблоны по рабочим группам
             var workGroupsWithTemplates = templates.GroupBy(t => t.Host!.WorkGroupId);
 
             foreach (var workGroupWithTemplates in workGroupsWithTemplates)
             {
-                var wgId = workGroupWithTemplates.Key ?? Guid.NewGuid();//TODO сделать правильно
+                var wgId = (Guid)workGroupWithTemplates.Key!;
                 var values = workGroupWithTemplates.ToList();
 
-                if (!values.Any())
-                    continue;
-
-                //Обновляем сразу время при необходимости
-                values.ForEach(t =>
+                //Обновляем сразу время при необходимости, так как дальше будем работать только с датой
+                var timesList = values.Select(t => t.NextRun.TimeOfDay).Distinct().ToList();
+                if (timesList.Count() > 1 || (timesList.Count() == 1 && timesList[0] != appInWork!.ReferenceDate.TimeOfDay))
                 {
-                    if (t.NextRun.TimeOfDay != appInWork!.ReferenceDate.TimeOfDay)
-                        //t.NextRun = new DateTimeOffset(t.NextRun.DateTime, new TimeSpan(//(appInWork!.ReferenceDate.Hour, appInWork!.ReferenceDate.Minute, appInWork!.ReferenceDate.Second));
-                        t.NextRun = new DateTimeOffset(t.NextRun.Year, t.NextRun.Month, t.NextRun.Day,
-                            appInWork!.ReferenceDate.Hour, appInWork!.ReferenceDate.Minute, appInWork!.ReferenceDate.Second,
-                            new TimeSpan(0, 0, 0));
-                });
-
-                if (!await applicationsInWorkService.CommitAsync())
-                {
-                    logger.LogError($"Ошибка записи изменений в БД при актуализации времени NextRun шаблонов");
-                    return;
+                    values.ForEach(t =>
+                    {
+                        if (t.NextRun.TimeOfDay != appInWork!.ReferenceDate.TimeOfDay)
+                            t.NextRun = new DateTimeOffset(t.NextRun.Year, t.NextRun.Month, t.NextRun.Day,
+                                appInWork!.ReferenceDate.Hour, appInWork!.ReferenceDate.Minute, appInWork!.ReferenceDate.Second,
+                                new TimeSpan(0, 0, 0));
+                    });
+                    if (!await templateService.CommitAsync())
+                    {
+                        logger.LogError($"Ошибка записи изменений в БД при актуализации времени NextRun шаблонов РР({applicationInWorkId})");
+                        return;
+                    }
                 }
 
-
-                var distrTemplates = await DistributeTemplateAsync(values, applicationInWorkId, wgId);
+                var distrTemplates = await DistributeTemplateAsync(new List<Template>(), applicationInWorkId, wgId);
 
                 if (distrTemplates.Any())
-                    if (!await applicationsInWorkService.CommitAsync())
-                        logger.LogError($"Ошибка записи изменений в БД при перераспределении NextRun шаблонов");
+                    if (!await templateService.CommitAsync())
+                        logger.LogError($"Ошибка записи изменений в БД при перераспределении NextRun шаблонов РР({applicationInWorkId})");
             }
         }
 
@@ -101,12 +86,12 @@ namespace PARR.TemplateDistributor
                                                             .Include(aiw => aiw.EsppSchValues)
                                                                 .ThenInclude(esv => esv.EsppSchTypeValue)
                                                                 .ThenInclude(etv => etv!.DistributionPeriod)
-                                                            .FirstOrDefaultAsync(t => t.Id == applicationInWorkId);
+                                                            .FirstAsync(t => t.Id == applicationInWorkId);
 
-            var refDate = appInWork!.ReferenceDate;
+            var refDate = appInWork.ReferenceDate;
 
             //Проверяем наличие распределения РР на период
-            if (appInWork!.IsAutoDistributionEnabled)
+            if (appInWork.IsAutoDistributionEnabled)
             {
                 var existingTemplates = await templateService.Get()
                                                     .Include(t => t.Host)
@@ -114,10 +99,11 @@ namespace PARR.TemplateDistributor
                                                     .ToListAsync();
 
                 //Удалим из входных шаблонов уже существующие в БД
+                //TODO: сделать селект повторяющихся шаблонов, написать их в варнинг и потом удалить
                 templates.RemoveAll(t => existingTemplates.Any(et => et.Id == t.Id));
 
                 //Если распределенная РР получаем начало
-                var period = appInWork.EsppSchValues.FirstOrDefault()?.EsppSchTypeValue?.DistributionPeriod;
+                var period = appInWork.EsppSchValues.First()!.EsppSchTypeValue!.DistributionPeriod;
                 var periodType = ParseDistributionPeriodType(period!.Type);
 
                 var workDays = await GetWorkDaysAsync(appInWork);
@@ -162,7 +148,7 @@ namespace PARR.TemplateDistributor
 
                 return templates;
 
-
+                #region comments
 
                 //var d = new Dictionary<DateOnly, List<Template>>();
                 //foreach (var wd in workDays)
@@ -233,6 +219,8 @@ namespace PARR.TemplateDistributor
                 //        }
                 //    }
                 //}
+
+                #endregion
             }
             else
                 templates.ForEach(t => t.NextRun = appInWork!.ReferenceDate);
@@ -240,7 +228,12 @@ namespace PARR.TemplateDistributor
             return templates;
         }
 
-        private async Task<List<DateOnly>> GetWorkDaysAsync(ApplicationsInWork appInWork)
+        /// <summary>
+        /// Получить список рабочих дней из календаря и перенести прошедшие даты на следующий период по РР
+        /// </summary>
+        /// <param name="appInWork"></param>
+        /// <returns></returns>
+        public async Task<List<DateOnly>> GetWorkDaysAsync(ApplicationsInWork appInWork)
         {
 
             var period = appInWork.EsppSchValues.FirstOrDefault()?.EsppSchTypeValue?.DistributionPeriod;
@@ -255,6 +248,7 @@ namespace PARR.TemplateDistributor
                                             wd.Year, wd.Month, wd.Day,
                                             appInWork.ReferenceDate.Hour, appInWork.ReferenceDate.Minute, appInWork.ReferenceDate.Second,
                                             new TimeSpan(0, 0, 0));
+
                 if (wrokDateTimeOffset < DateTimeOffset.UtcNow)
                 {
                     wrokDateTimeOffset = esppScheduleTransformService.GetNextDateForDistributionRun(wrokDateTimeOffset, periodType, period.Duration);
@@ -265,16 +259,19 @@ namespace PARR.TemplateDistributor
             return result;
         }
 
-
+        /// <summary>
+        /// Проверить соответствие распределения существующих шаблонов нововму плану.
+        /// На выходе получаем шаблоны, которым требуется изменить дату следующего запуска.
+        /// </summary>
+        /// <param name="distributionPlan"></param>
+        /// <param name="templates"></param>
+        /// <returns></returns>
         private List<Template> GetTemplatesToDistribute(ref Dictionary<DateOnly, int> distributionPlan, List<Template> templates)
         {
             var result = new List<Template>();
 
             foreach (var wd in distributionPlan)
             {
-                //if (wd.Value < 1)
-                //    continue;
-
                 var templatesInWd = templates.Where(t => DateOnly.FromDateTime(t.NextRun.DateTime) == wd.Key).ToList();
                 var templatesToDistribute = templatesInWd.Skip(wd.Value).Take(templatesInWd.Count - wd.Value).ToList();
 
@@ -303,48 +300,51 @@ namespace PARR.TemplateDistributor
 
         private Dictionary<DateOnly, int> GetDistributionPlan(List<DateOnly> workDays, int templateCount)
         {
-            var d = new Dictionary<DateOnly, int>();
+            var result = new Dictionary<DateOnly, int>();
+            //Заполняем будующий план датами из полученных на входе данных
             foreach (var wd in workDays)
             {
-                var workDateTimeOffset = new DateOnly(wd.Year, wd.Month, wd.Day);
-                d.Add(workDateTimeOffset, 0);
+                var workDate = new DateOnly(wd.Year, wd.Month, wd.Day);
+                result.Add(workDate, 0);
             }
 
-            var templateDistributed = 0;
+            var templateDistributedCount = 0;//количество распределённых на текущий момент шаблонов
 
-            if (templateCount >= workDays.Count)
+            //Если количество шаблонов больше количества рабочих дней, то сразу распределяем их равным количеством по всем рабочим дням
+            if (templateDistributedCount >= workDays.Count)
             {
+                //количеством шаблонов на один рабочий день
                 var mainPartTemplateCountToDistribute = templateCount / workDays.Count;
-                foreach (var item in d)
-                {
-                    d[item.Key] = mainPartTemplateCountToDistribute;
 
-                    templateDistributed += mainPartTemplateCountToDistribute;
+                //перебираем рабочие дня и записываем количество шаблонов
+                foreach (var item in result)
+                {
+                    result[item.Key] = mainPartTemplateCountToDistribute;
+
+                    templateDistributedCount += mainPartTemplateCountToDistribute;
                 }
             }
 
+            var daysBetweenTemplates = (double)workDays.Count / (templateCount % workDays.Count);//количество дней между шаблонами, если распределить их равномерно
+            var currentStepBetweenTemplates = (double)Math.Floor(daysBetweenTemplates);//инициализация шага перехода между датами запуска двух шаблонов
+            var balance = (double)daysBetweenTemplates - Math.Floor(currentStepBetweenTemplates);//остаток между датами, потому что мы берём целые дни, его нужно учесть в следующей итирации
+            var currentDayIndex = 0;//инициализация индекса даты словаря рабочих дней
 
-            var templatesPerWorkDay = (double)workDays.Count / (templateCount % workDays.Count);
-            var currentStep = (double)Math.Floor(templatesPerWorkDay);//??????? точно double?
-            var balance = (double)templatesPerWorkDay - Math.Floor(currentStep);
-            var currentIndex = 0;
-
-            while (templateDistributed < templateCount)
+            while (templateDistributedCount < templateCount)
             {
-                if (currentStep > 0)
-                {
-                    d[d.ElementAt(currentIndex).Key] += 1;
-                    templateDistributed++;
-                    currentIndex += (int)Math.Floor(currentStep);
+                //лишняя проверка так как шагом ранее мы распределям шаблоны равным количеством и currentStepBetweenTemplates не может быть меньше одного дня
+                //if (currentStepBetweenTemplates > 0)
+                //{
+                result[result.ElementAt(currentDayIndex).Key] += 1;
+                templateDistributedCount++;
+                currentDayIndex += (int)Math.Floor(currentStepBetweenTemplates);
+                //}
 
-
-                } 
-
-                currentStep = templatesPerWorkDay + balance;
-                balance = templatesPerWorkDay - currentStep;
+                currentStepBetweenTemplates = daysBetweenTemplates + balance;
+                balance = daysBetweenTemplates - currentStepBetweenTemplates;
             }
 
-            return d;
+            return result;
         }
     }