diff --git a/PARR.EsppScheduleSync/ScheduleSyncher.cs b/PARR.EsppScheduleSync/ScheduleSyncher.cs index 1d47c10e..9081eba1 100644 --- a/PARR.EsppScheduleSync/ScheduleSyncher.cs +++ b/PARR.EsppScheduleSync/ScheduleSyncher.cs @@ -55,7 +55,7 @@ namespace PARR.EsppScheduleSync if (!isConnected) throw new Exception("Ошибка при подключении к RabbitMq"); - logger.LogInformation($"Запущена проверка очереди {globalSettings.MqSettings!.QueueName}."); + logger.LogInformation("Запущена проверка очереди {QueueName}.", globalSettings.MqSettings!.QueueName); } @@ -63,7 +63,7 @@ namespace PARR.EsppScheduleSync { await mqService.DisposeAsync(); - logger.LogInformation($"=== === === Соединение с очередью {globalSettings.MqSettings!.QueueName} закрыто === === ==="); + logger.LogInformation("=== === === Соединение с очередью {QueueName} закрыто === === ===", globalSettings.MqSettings!.QueueName); } @@ -202,7 +202,7 @@ namespace PARR.EsppScheduleSync ScheduleName = template.Name, IsActive = template.IsActiveSchedule, //ResponseArea = template.Host!.ResponseArea!.Name,//TODO Migration to job - //WorkGroup = template.Host!.WorkGroup!, + //WorkGroup = template.Host!.WorkGroup!,//TODO Migration to job //WorkGroup = template.Host!.WorkGroup!.Name,//TODO Migration to job //Мы решили, что для всех расписаний "Нет исключений", если что-то поменяется, тут нужно переделать //TypeV60calendar = settingsFromDb.ScheduleExcludeType == "Нет исключений" ? "NONE" : "", @@ -234,14 +234,14 @@ namespace PARR.EsppScheduleSync // мы знаем, что у нас точно в шаблоне есть инклуды до ScheduleExcludeType и ScheduleExcludeTypeCalendar if (template.Job?.Group?.ScheduleExcludeTypeCalendar == null) { - logger.LogDebug("Для шаблона шаблона {templte}, {templateId} нет исключений календаря", template.Name, template.Id); + logger.LogDebug("Для шаблона шаблона {TemplateName}, {TemplateId} нет исключений календаря", template.Name, template.Id); //TODO:!!!!!!!!!!! Вот тут null или string.Empty??? Спросить у Андрея что он нам вернет! return string.Empty; } - logger.LogDebug("Для шаблона шаблона {templte}, {templateId} установлено исключений календаря \"{name}\", EsppValue: {esppValue}", template.Name, template.Id, template.Job.Group.ScheduleExcludeTypeCalendar.Title, template.Job.Group.ScheduleExcludeTypeCalendar.EsppValue); + logger.LogDebug("Для шаблона шаблона {TemplateName}, {TemplateId} установлено исключений календаря \"{Name}\", EsppValue: {EsppValue}", template.Name, template.Id, template.Job.Group.ScheduleExcludeTypeCalendar.Title, template.Job.Group.ScheduleExcludeTypeCalendar.EsppValue); return template.Job.Group.ScheduleExcludeTypeCalendar.EsppValue; } @@ -258,11 +258,11 @@ namespace PARR.EsppScheduleSync // на всякий конечно же проверим if (template.Job?.Group?.ScheduleExcludeType == null) { - logger.LogError("Для шаблона {templte}, {templateId} не смог получить тип исключения, установил значение по умолчанию \"Без исключения\"", template.Name, template.Id); + logger.LogError("Для шаблона {TemplateName}, {TemplateId} не смог получить тип исключения, установил значение по умолчанию \"Без исключения\"", template.Name, template.Id); return "NONE"; } - logger.LogDebug("Для шаблона шаблона {templte}, {templateId} тип исключения \"{name}\", EsppValue: {esppValue}", template.Name, template.Id, template.Job.Group.ScheduleExcludeType.Title, template.Job.Group.ScheduleExcludeType.EsppValue); + logger.LogDebug("Для шаблона шаблона {TemplateName}, {TemplateId} тип исключения \"{Name}\", EsppValue: {EsppValue}", template.Name, template.Id, template.Job.Group.ScheduleExcludeType.Title, template.Job.Group.ScheduleExcludeType.EsppValue); return template.Job.Group.ScheduleExcludeType.EsppValue; @@ -297,7 +297,7 @@ namespace PARR.EsppScheduleSync var esppSchedule = esppSchTypeConfigService.GetEsppScheduleDto(template.Job!.GroupId); if (esppSchedule == null) { - logger.LogError($"Не смог получить расписание из БД для шаблона templateId: {template.Id}, {template.Name}"); + logger.LogError("Не смог получить расписание из БД для шаблона templateId: {TemplateId}, {TemplateName}", template.Id, template.Name); return; } @@ -361,7 +361,7 @@ namespace PARR.EsppScheduleSync var splittedContent = str.Split(globalSettings.ParsingSeparator); if (splittedContent.Length != 24) { - logger.LogError($"Входная строка после сплита не содержит 24 объекта (факт: {splittedContent.Length})."); + logger.LogError("Входная строка после сплита не содержит 24 объекта (факт: {Length}).", splittedContent.Length); return null; } @@ -437,7 +437,7 @@ namespace PARR.EsppScheduleSync if (typeSchedule == EsppSchTypeScheduleEnum.Annually2.ToString().ToLower()) return EsppSchTypeScheduleEnum.Annually2; - logger.LogError($"Не смог преобразовать Тип повторения из ЕСПП в EsppSchTypeScheduleEnum. Получено значение {typeSchedule}"); + logger.LogError("Не смог преобразовать Тип повторения из ЕСПП в EsppSchTypeScheduleEnum. Получено значение {TypeSchedule}", typeSchedule); return null; } @@ -458,7 +458,7 @@ namespace PARR.EsppScheduleSync if (!string.IsNullOrEmpty(settingsFromDb.TemplatePrefixWithoutVariable) && name.Contains(settingsFromDb.TemplatePrefixWithoutVariable)) // , StringComparison.CurrentCultureIgnoreCase return true; - logger.LogWarning($"Имя шаблона или расписания не соответствует обязательному префиксу({settingsFromDb.TemplatePrefixWithoutVariable}). {name} игнорирован"); + logger.LogWarning("Имя шаблона или расписания не соответствует обязательному префиксу({Prefix}). {Name} игнорирован", settingsFromDb.TemplatePrefixWithoutVariable, name); return false; } @@ -562,4 +562,4 @@ namespace PARR.EsppScheduleSync } } -} +} \ No newline at end of file diff --git a/PARR.EsppSync/SyncService.cs b/PARR.EsppSync/SyncService.cs index b0bb0108..6e41fcf2 100644 --- a/PARR.EsppSync/SyncService.cs +++ b/PARR.EsppSync/SyncService.cs @@ -31,7 +31,7 @@ namespace PARR.EsppSync ConvertDbObjToComparisonObjHandlerDelegate converterToEsppObject ) { - logger.LogDebug($"Получил строку. Начинаю работать. Строка: {str}"); + logger.LogDebug("Получил строку. Начинаю работать. Строка: {String}", str); if (string.IsNullOrEmpty(str)) { @@ -85,7 +85,7 @@ namespace PARR.EsppSync if (template == null) { - logger.LogWarning($"Найден объект в ЕСПП с именем шаблона {esppObject.TemplateName} незарегистрированный в ПАРР."); + logger.LogWarning("Найден объект в ЕСПП с именем шаблона {TemplateName} незарегистрированный в ПАРР. Строка: {String}", esppObject.TemplateName, str); return; } @@ -116,7 +116,7 @@ namespace PARR.EsppSync if (dbObjectInEsppObject.IsActive == false) { - logger.LogDebug($"Объект деактивирован в ПАРР. Сравниваем только обязательные поля. {esppObject.TemplateName}"); + logger.LogDebug("Объект деактивирован в ПАРР. Сравниваем только обязательные поля. {TemplateName}", esppObject.TemplateName); var lightDbObj = new EsppLightObject(dbObjectInEsppObject); var lightEsppObject = new EsppLightObject(esppObject); @@ -124,13 +124,13 @@ namespace PARR.EsppSync } else { - logger.LogDebug($"Объект активирован в ПАРР. Сравниваем все поля. {esppObject.TemplateName}"); + logger.LogDebug("Объект активирован в ПАРР. Сравниваем все поля. {TemplateName}", esppObject.TemplateName); isChanged = IsChanged(esppObject, dbObjectInEsppObject, esppObject.TemplateName); } if (isChanged) { - logger.LogDebug($"Есть изменения, требуется обновление. {esppObject.TemplateName}"); + logger.LogDebug("Есть изменения, требуется обновление. {TemplateName}", esppObject.TemplateName); var config = robotConfigurationService.GetFromTemplateByRobotCode(esppObject.Robot, template); @@ -141,14 +141,14 @@ namespace PARR.EsppSync SetUpdateStatus(ref template, robotConfigurationService, esppObject.Robot); if (!await templateService.CommitAsync()) - logger.LogError($"Не удалось изменить запись Template {template.Name}, Robot: {esppObject.Robot}"); + logger.LogError("Не удалось изменить запись Template {TemplateName}, Robot: {Robot}", template.Name, esppObject.Robot); else - logger.LogInformation($"Установлен принудительный статус {TaskStatusEnum.Updating}, Template {template.Name}, Robot: {esppObject.Robot}"); + logger.LogInformation("Установлен принудительный статус {TaskStatus}, Template {TemplateName}, Robot: {Robot}", TaskStatusEnum.Updating, template.Name, esppObject.Robot); } else { // Если пред статус был Update, то ничего не делаем, так его и оставляем, не сбрасывам кол-во попыток и ошибок - logger.LogInformation($"Есть изменения в Template {template.Name}, но предыдущий статус TaskStatusCode: {(TaskStatusEnum)config.TaskStatusCode}. Не меняем статус, будем разбираться вручную."); + logger.LogInformation("Есть изменения в Template {TemplateName}, но предыдущий статус TaskStatusCode: {TaskStatusCode}. Не меняем статус, будем разбираться вручную.", template.Name, (TaskStatusEnum)config.TaskStatusCode); } #region Old logic @@ -163,7 +163,7 @@ namespace PARR.EsppSync }//надо ли проверять если не изменился, но был статус Updating не понятно. Доверяем роботу пока, что после окончания работ он точно сообщит else { - logger.LogDebug($"Нет изменений, обновление не требуется. {esppObject.TemplateName}"); + logger.LogDebug("Нет изменений, обновление не требуется. {TemplateName}", esppObject.TemplateName); //если все поля совпали //проверяем, какой был статус предыдущий статус в БД, если он был не Ок, то ставим ему ОК @@ -172,15 +172,15 @@ namespace PARR.EsppSync { robotConfigurationService.ChangeTaskStatus(TaskStatusEnum.Ok, robotConfig); if (!await templateService.CommitAsync()) - logger.LogError($"Не удалось изменить запись Template {template.Name}, Robot: {esppObject.Robot}"); + logger.LogError("Не удалось изменить запись Template {TemplateName}, Robot: {Robot}", template.Name, esppObject.Robot); else - logger.LogInformation($"Установлен принудительный статус {TaskStatusEnum.Ok}, Template {template.Name}, Robot: {esppObject.Robot}"); + logger.LogInformation("Установлен принудительный статус {TaskStatus}, Template {TemplateName}, Robot: {Robot}", TaskStatusEnum.Ok, template.Name, esppObject.Robot); } } } catch (Exception ex) { - logger.LogError(ex, $"Ошибка синхронизации объекта АСУ ЕСПП {esppObject.TemplateName}"); + logger.LogError(ex, "Ошибка синхронизации объекта АСУ ЕСПП {TemplateName}", esppObject.TemplateName); } } } @@ -223,7 +223,7 @@ namespace PARR.EsppSync if (dbValueStr != esppValueStr) { - logger.LogInformation($"Не совпадают поля ({prop.Name}). dbValueStr: {dbValueStr}, esppValueStr: {esppValueStr}. Имя шаблона: {templateName}"); + logger.LogInformation("Не совпадают поля ({PropertyName}). dbValueStr: {DbValueStr}, esppValueStr: {EsppValueStr}. Имя шаблона: {TemplateName}", prop.Name, dbValueStr, esppValueStr, templateName); return true; } @@ -247,4 +247,4 @@ namespace PARR.EsppSync return str.ToLower(); } } -} +} \ No newline at end of file diff --git a/PARR.EsppTemplateSync/TemplateMQSyncer.cs b/PARR.EsppTemplateSync/TemplateMQSyncer.cs index a488e1e0..2a91f675 100644 --- a/PARR.EsppTemplateSync/TemplateMQSyncer.cs +++ b/PARR.EsppTemplateSync/TemplateMQSyncer.cs @@ -46,14 +46,14 @@ namespace PARR.EsppTemplateSync if (!isConnected) throw new Exception("Ошибка при подключении к RabbitMq"); - logger.LogInformation($"Запущена проверка очереди {globalSettings.MqSettings!.QueueName}."); + logger.LogInformation("Запущена проверка очереди {QueueName}.", globalSettings.MqSettings!.QueueName); } public async Task StopAsync() { await mqService.DisposeAsync(); - logger.LogInformation($"=== === === Соединение с очередью {globalSettings.MqSettings!.QueueName} закрыто === === ==="); + logger.LogInformation("=== === === Соединение с очередью {QueueName} закрыто === === ===", globalSettings.MqSettings!.QueueName); } @@ -118,7 +118,7 @@ namespace PARR.EsppTemplateSync var splittedContent = str.Split(globalSettings.ParsingSeparator); if (splittedContent.Length != 17) { - logger.LogError($"Входная строка после сплита не содержит 17 объектов (факт: {splittedContent.Length})."); + logger.LogError("Входная строка после сплита не содержит 17 объектов (факт: {Length}).", splittedContent.Length); return null; } @@ -129,7 +129,7 @@ namespace PARR.EsppTemplateSync //if (!string.IsNullOrEmpty(settingsFromDb.TemplatePrefixName) || !templateName.StartsWith(settingsFromDb.TemplatePrefixName, true, currentCulture)) if (string.IsNullOrEmpty(settingsFromDb.TemplatePrefixWithoutVariable) || !templateName.Contains(settingsFromDb.TemplatePrefixWithoutVariable)) // , StringComparison.CurrentCultureIgnoreCase { - logger.LogWarning($"Имя шаблона не соответствует обязательному префиксу({settingsFromDb.TemplatePrefixWithoutVariable}). Шаблон {templateName} игнорирован"); + logger.LogWarning("Имя шаблона не соответствует обязательному префиксу({Prefix}). Шаблон {TemplateName} игнорирован", settingsFromDb.TemplatePrefixWithoutVariable, templateName); return null; } @@ -176,5 +176,4 @@ namespace PARR.EsppTemplateSync return templateFromEspp; } } -} - +} \ No newline at end of file