mikhail/parr_api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(esppScheduleSync, esppSync): вынесена логика сравнения nextRun из EsppSync в EsppScheduleSync. Рефакторинг EsppScheduleSync - логика проверки имени шаблона

Browse Source
This commit is contained in:
Mikhail Trubnikov
2026-02-18 11:40:25 +10:00
parent d17f6cf409
commit 517d446bcf
6 changed files with 272 additions and 255 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
PARR.EsppSync/Domain/EsppObjectSchedule.cs
View File

@@ -11,6 +11,9 @@ namespace PARR.EsppSync.Domain
public required string TemplateName { get; set; }
/// <summary>
/// ИД расписания
/// </summary>
public string Code { get; set; } = string.Empty;
public string ScheduleName { get; set; } = string.Empty;

25
PARR.EsppSync/Helpers/EsppSyncHelpers.cs Normal file
View File

@@ -0,0 +1,25 @@
namespace PARR.EsppSync.Helpers
{
/// <summary>
/// Хелперы для EsppSync
/// </summary>
public static class EsppSyncHelpers
{
/// <summary>
/// Првести строку в стандарт сравнения (для сравнения объектов)
/// </summary>
/// <param name="str"></param>
/// <returns></returns>
public static string Normalize(string? str)
{
if (str == null)
return string.Empty;
str = str.Replace("\r", string.Empty);
str = str.Replace("\n", string.Empty);
str = str.Replace(" ", string.Empty);
return str.ToLower();
}
}
}

39
PARR.EsppSync/ISyncService.cs
View File

@@ -8,7 +8,7 @@ namespace PARR.EsppSync
/// <typeparam name="EsppObject"></typeparam>
/// <param name="str"></param>
/// <returns></returns>
public delegate EsppObject? ParserHandlerDelegate<EsppObject>(string str) where EsppObject : class, IEsppObject;
public delegate EsppObject? ParseStringToEsppObject<EsppObject>(string str) where EsppObject : class, IEsppObject;
/// <summary>
/// Конвертирует Template в модель для сравнения
@@ -16,17 +16,44 @@ namespace PARR.EsppSync
/// <typeparam name="EsppObject"></typeparam>
/// <param name="templateName"></param>
/// <returns></returns>
public delegate EsppObject ConvertDbObjToComparisonObjHandlerDelegate<EsppObject>(Template template) where EsppObject : class, IEsppObject;
public delegate EsppObject ConvertDbToEsppObject<EsppObject>(Template template) where EsppObject : class, IEsppObject;
/// <summary>
/// Дополнительная проверка полей, если проверка прошла, то true
/// </summary>
/// <typeparam name="EsppObject"></typeparam>
/// <param name="esppObject"></param>
/// <param name="dbObject"></param>
/// <param name="templateId"></param>
/// <returns>true - проверка пройдена, false - проверка не пройдена</returns>
public delegate Task<bool> CustomComparisionCheckAsync<EsppObject>(EsppObject esppObject, EsppObject dbObject, Guid templateId) where EsppObject : class, IEsppObject;
/// <summary>
/// Выполнить произвольную логику после парсинга строки в объект ЕСПП
/// </summary>
/// <typeparam name="EsppObject"></typeparam>
/// <param name="esppObject"></param>
/// <returns></returns>
public delegate Task AfterParseStringToEsppObjectAsync<EsppObject>(EsppObject esppObject) where EsppObject : class, IEsppObject;
public interface ISyncService<EsppObject> where EsppObject : class, IEsppObject
{
/// <summary>
///
/// Синхронизировать объекты
/// </summary>
/// <param name="str">Строка из RabbitMQ</param>
/// <param name="parser">Делегат парсинга из строки в модель EsppObject</param>
/// <param name="converterToEsppObject"> Конвертирует BD Template в модель для сравнения</param>
/// <param name="parseStringToEsppObject">Делегат парсинга из строки в модель EsppObject</param>
/// <param name="converterDbToEsppObject"> Конвертирует BD Template в модель для сравнения</param>
/// <param name="customComparisionAsync">Кастомное сравнение полей</param>
/// <param name="afterParseStringToEsppObjectAsync">Выполнить произвольный метод после парсинга строки в модель EsppObject</param>
/// <returns></returns>
Task SyncEsppObjectAsync(string str, ParserHandlerDelegate<EsppObject> parser, ConvertDbObjToComparisonObjHandlerDelegate<EsppObject> converterToEsppObject);
Task SyncEsppObjectAsync(string str,
ParseStringToEsppObject<EsppObject> parseStringToEsppObject,
ConvertDbToEsppObject<EsppObject> converterDbToEsppObject,
CustomComparisionCheckAsync<EsppObject>? customComparisionAsync = null,
AfterParseStringToEsppObjectAsync<EsppObject>? afterParseStringToEsppObjectAsync = null
);
}
}

134
PARR.EsppSync/SyncService.cs
View File

@@ -1,14 +1,12 @@
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
using PARR.BLL.Helpers;
using PARR.Constants;
using PARR.DAL.Contracts;
using PARR.DAL.DomainServices.Shortcodes;
using PARR.DAL.Models;
using PARR.DAL.NextRunServices;
using PARR.DAL.Services.Interfaces;
using PARR.EsppSync.Domain;
using PARR.EsppSync.Helpers;
using System.Reflection;
namespace PARR.EsppSync
@@ -30,8 +28,10 @@ namespace PARR.EsppSync
public async Task SyncEsppObjectAsync(
string str,
ParserHandlerDelegate<EsppObject> parser,
ConvertDbObjToComparisonObjHandlerDelegate<EsppObject> converterToEsppObject
ParseStringToEsppObject<EsppObject> parseStringToEsppObject,
ConvertDbToEsppObject<EsppObject> converterDbToEsppObject,
CustomComparisionCheckAsync<EsppObject>? customComparisionAsync = null,
AfterParseStringToEsppObjectAsync<EsppObject>? afterParseStringToEsppObjectAsync = null
)
{
logger.LogDebug("Получил строку. Начинаю работать. Строка: {String}", str);
@@ -42,7 +42,7 @@ namespace PARR.EsppSync
return;
}
var esppObject = parser.Invoke(str);
var esppObject = parseStringToEsppObject.Invoke(str);
if (esppObject == null)
{
@@ -50,12 +50,16 @@ namespace PARR.EsppSync
return;
}
// Вызовем кастомный метод, если он есть
if (afterParseStringToEsppObjectAsync != null)
await afterParseStringToEsppObjectAsync.Invoke(esppObject);
using (var scope = serviceProvider.CreateScope())
{
var templateService = GetServiceInScope<ITemplateService>(scope);
var robotConfigurationService = GetServiceInScope<IRobotConfigurationService>(scope);
var shortcodesService = GetServiceInScope<IShortcodesService>(scope);
var nextRunService = GetServiceInScope<INextRunServiceV2>(scope);
try
{
@@ -92,7 +96,7 @@ namespace PARR.EsppSync
return;
}
var dbObjectInEsppObject = converterToEsppObject.Invoke(template);
var dbObjectInEsppObject = converterDbToEsppObject.Invoke(template);
await ApplyShortcodesAsync(dbObjectInEsppObject, template, shortcodesService);
@@ -113,14 +117,28 @@ namespace PARR.EsppSync
logger.LogDebug("Объект активирован в ПАРР. Сравниваем все поля. {TemplateName}", esppObject.TemplateName);
isChanged = IsChanged(esppObject, dbObjectInEsppObject, esppObject.TemplateName);
// если это сравнение расписаний, рассчитать nextRun, и сравнить все три nextRun, БД - ЕСПП - Расчитанное
// будем считать, только если НЕТ изменений в остальных полях, если изменения есть, зачем лишний раз напрягать, потом в контроллере все пересчитается
if (!isChanged && dbObjectInEsppObject is EsppObjectSchedule dbValues && esppObject is EsppObjectSchedule esppValues)
// выполняем кастомную дополнительную проверку (только если isChanged==false, чтоб лишний раз не гонять)
if (!isChanged)
{
var isEqualNextRun = await IsEqualNextRunAsync(nextRunService, dbValues, esppValues, template.Id);
if (isEqualNextRun == false)
isChanged = true;
if (customComparisionAsync != null)
{
// добавлена дополнительная проверка
var isCustomComparision = await customComparisionAsync.Invoke(esppObject, dbObjectInEsppObject, template.Id);
if (isCustomComparision)
{
logger.LogDebug("Дополнительная проверка прошла.");
}
else
{
// если дополнительная проверка не прошла, то говорим что есть изменения
isChanged = true;
logger.LogDebug("Дополнительная проверка не прошла, ставим статус isChanged: {isChanged}", isChanged);
}
}
else
{
logger.LogDebug("Дополнительная проверка отсутствует");
}
}
}
@@ -252,8 +270,8 @@ namespace PARR.EsppSync
continue;
//Replace("\r","").Replace("\n","") - в подробном описании могут быть переносы строк, в Rabbit прилетает без переносов. Убираем переносы для стравнения
var dbValueStr = Normalize(dbValue!.ToString()!);
var esppValueStr = Normalize(esppValue!.ToString()!);
var dbValueStr = EsppSyncHelpers.Normalize(dbValue!.ToString());
var esppValueStr = EsppSyncHelpers.Normalize(esppValue!.ToString());
if (dbValueStr != esppValueStr)
{
@@ -266,87 +284,5 @@ namespace PARR.EsppSync
return false;
}
/// <summary>
/// Удаляет ненужные символы из строки
/// </summary>
/// <param name="str"></param>
/// <returns></returns>
private string Normalize(string str)
{
str = str.Replace("\r", string.Empty);
str = str.Replace("\n", string.Empty);
str = str.Replace(" ", string.Empty);
return str.ToLower();
}
/// <summary>
/// Рассчитать nextRun в часовом поясе УЗ робота
/// </summary>
/// <param name="nextRunService"></param>
/// <param name="templateId"></param>
/// <returns></returns>
private async Task<DateTimeOffset?> CalcNextRunAsync(INextRunServiceV2 nextRunService, Guid templateId)
{
var nextRun = await nextRunService.GetNextRunForTemplateAsync(templateId, isNew: false);
if (nextRun.HasValue)
{
var nextRunWithEsppAccountTz = nextRun.Value.Add(nextRunService.GetEsppAccountOffset());
logger.LogDebug("Расчитанный nextRun для шаблона {templateId}, UTC: {nextRun}, EsppAccountTz: {nextRunWithEsppAccountTz}", templateId, nextRun, nextRunWithEsppAccountTz);
return nextRunWithEsppAccountTz;
}
else
{
logger.LogError("При расчете nextRun для templateId: {templateId} верунлся null", templateId);
return null;
}
}
/// <summary>
/// Сравнить NextRun из БД, ЕСПП, рассчитанный
/// </summary>
/// <param name="dbValues"></param>
/// <param name="esppValues"></param>
/// <param name="templateId"></param>
/// <returns></returns>
private async Task<bool> IsEqualNextRunAsync(INextRunServiceV2 nextRunService, EsppObjectSchedule dbValues, EsppObjectSchedule esppValues, Guid templateId)
{
// в часовой зоне робота
var calculatedNextRun = await CalcNextRunAsync(nextRunService, templateId);
if (calculatedNextRun == null)
{
return false;
}
var scheduledCalculated = EsppScheduleHelpers.GetNextRun(calculatedNextRun.Value);
var basisTimeCalculated = EsppScheduleHelpers.GetGenerationTime(calculatedNextRun.Value);
logger.LogDebug("Рассчитанные значения для шаблона '{templateName}', {templateId}, следующее срабатывание {scheduledCalculated}, время создания наряда: {basisTimeCalculated}",
dbValues.TemplateName, templateId, scheduledCalculated, basisTimeCalculated);
if (Normalize(dbValues.Scheduled) != Normalize(esppValues.Scheduled) || Normalize(dbValues.Scheduled) != scheduledCalculated || Normalize(esppValues.Scheduled) != scheduledCalculated)
{
logger.LogInformation("Не совпадают поля ({propName}), dbValueStr: {dbValueStr}, esppValueStr: {esppValueStr}, scheduledCalculated: {scheduledCalculated}. Имя шаблона: {templateName}",
nameof(dbValues.Scheduled), dbValues.Scheduled, esppValues.Scheduled, scheduledCalculated, dbValues.TemplateName);
return false;
}
if (Normalize(dbValues.BasisTime) != Normalize(esppValues.BasisTime) || Normalize(dbValues.BasisTime) != basisTimeCalculated || Normalize(esppValues.BasisTime) != basisTimeCalculated)
{
logger.LogInformation("Не совпадают поля ({propName}), dbValueStr: {dbValueStr}, esppValueStr: {esppValueStr}, basisTimeCalculated: {basisTimeCalculated}. Имя шаблона: {templateName}",
nameof(dbValues.BasisTime), dbValues.BasisTime, esppValues.BasisTime, basisTimeCalculated, dbValues.TemplateName);
return false;
}
logger.LogDebug("Значения nextRun в БД, ЕСПП, расчитанное, все совпадают. Scheduled: {scheduled}, basisTime: {basisTime}", scheduledCalculated, basisTimeCalculated);
return true;
}
}
}