fix: Обработаны замечания SonarQube.

arkuznetsov · May 31, 2022 · 5bfa9a4 · 5bfa9a4
1 parent 9586cfe
commit 5bfa9a4
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 14 deletions.
diff --git a/src/cmd/Классы/КомандаВыполнитьПакет.os b/src/cmd/Классы/КомандаВыполнитьПакет.os
@@ -92,10 +92,11 @@
 		Попытка
 			ПрочитатьСценарииИзФайла(Сценарии, ТекФайл);
 		Исключение
+			ТекстОшибки = ПодробноеПредставлениеОшибки(ИнформацияОбОшибке());
 			ТекстОшибки = СтрШаблон("Ошибка чтения файла сценариев ""%1"":%2%3",
 			                        ТекФайл,
 			                        Символы.ПС,
-			                        ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+			                        ТекстОшибки);
 			ВызватьИсключение ТекстОшибки;
 		КонецПопытки;
 

diff --git a/src/cmd/Классы/ЧтениеОпцийКоманды.os b/src/cmd/Классы/ЧтениеОпцийКоманды.os
@@ -69,10 +69,11 @@
 			ПараметрыИзФайла = ПрочитатьJSON(Чтение, Истина);
 
 		Исключение
+			ТекстОшибки = ПодробноеПредставлениеОшибки(ИнформацияОбОшибке());
 			ТекстОшибки = СтрШаблон("Ошибка чтения файла параметров ""%1"":%2%3",
 			                        ТекФайл,
 			                        Символы.ПС,
-			                        ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+			                        ТекстОшибки);
 			ВызватьИсключение ТекстОшибки;
 		КонецПопытки;
 

diff --git a/src/core/Классы/ПодключениеMSSQL.os b/src/core/Классы/ПодключениеMSSQL.os
@@ -1665,7 +1665,6 @@
 	КомандаРК.ДобавитьПараметр(СтрШаблон("-o ""%1""", ПутьКФайлуРезультата));
 
 	КомандаРК.УстановитьИсполнениеЧерезКомандыСистемы(Ложь);
-	КомандаРК.УстановитьОбработчикОжидания(Неопределено, 10);
 	КомандаРК.ПоказыватьВыводНемедленно(Ложь);
 
 	КодВозврата = КомандаРК.Исполнить();

diff --git a/src/core/Классы/РаботаСЯндексДиск.os b/src/core/Классы/РаботаСЯндексДиск.os
@@ -62,7 +62,6 @@
 		ВызватьИсключение ТекстОшибки;
 	КонецЕсли;
 
-	ТекущийПуть = "";
 	Если НЕ КаталогНайден Тогда
 		Попытка
 			Соединение.СоздатьПапку(ЦелевойПуть);

diff --git a/src/core/Классы/СтруктураХраненияИБ.os b/src/core/Классы/СтруктураХраненияИБ.os
@@ -1002,22 +1002,41 @@
 		ОписаниеКонфигурации.Имя = Совпадения[0].Группы[1].Значение;
 	КонецЕсли;
 
+	Синоним_Индекс                      = 0;
+	ПодробнаяИнформация_Индекс          = 1;
+	КраткаяИнформация_Индекс            = 2;
+	АвторскиеПрава_Индекс               = 3;
+	АдресИнформацииОПоставщике_Индекс   = 4;
+	АдресИнформацииОКонфигурации_Индекс = 5;
+
 	Совпадения = РВ_Текст.НайтиСовпадения(ТекстОписания);
 
 	Если Совпадения.Количество() > 0 Тогда
 		ПустоеЗначение = "{0}";
 		ОписаниеКонфигурации.Синоним =
-			?(Совпадения[0].Значение = ПустоеЗначение, "", Совпадения[0].Группы[1].Значение);
+			?(Совпадения[Синоним_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[Синоним_Индекс].Группы[1].Значение);
 		ОписаниеКонфигурации.ПодробнаяИнформация =
-			?(Совпадения[1].Значение = ПустоеЗначение, "", Совпадения[1].Группы[1].Значение);
+			?(Совпадения[ПодробнаяИнформация_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[ПодробнаяИнформация_Индекс].Группы[1].Значение);
 		ОписаниеКонфигурации.КраткаяИнформация =
-			?(Совпадения[2].Значение = ПустоеЗначение, "", Совпадения[2].Группы[1].Значение);
+			?(Совпадения[КраткаяИнформация_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[КраткаяИнформация_Индекс].Группы[1].Значение);
 		ОписаниеКонфигурации.АвторскиеПрава =
-			?(Совпадения[3].Значение = ПустоеЗначение, "", Совпадения[3].Группы[1].Значение);
+			?(Совпадения[АвторскиеПрава_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[АвторскиеПрава_Индекс].Группы[1].Значение);
 		ОписаниеКонфигурации.АдресИнформацииОПоставщике =
-			?(Совпадения[4].Значение = ПустоеЗначение, "", Совпадения[4].Группы[1].Значение);
+			?(Совпадения[АдресИнформацииОПоставщике_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[АдресИнформацииОПоставщике_Индекс].Группы[1].Значение);
 		ОписаниеКонфигурации.АдресИнформацииОКонфигурации =
-			?(Совпадения[5].Значение = ПустоеЗначение, "", Совпадения[5].Группы[1].Значение);
+			?(Совпадения[АдресИнформацииОКонфигурации_Индекс].Значение = ПустоеЗначение,
+			  "",
+			  Совпадения[АдресИнформацииОКонфигурации_Индекс].Группы[1].Значение);
 	КонецЕсли;
 
 КонецПроцедуры // ЗаполнитьТекстыВОписанииКонфигурации()
@@ -1037,12 +1056,16 @@
 		Новый РегулярноеВыражение("^\{?\d+,\d+\},\d+," +
 	                              "(?:(?i)(?:[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}),){4}(\d+)");
 
+	Поставщик_Индекс               = 1;
+	Версия_Индекс                  = 2;
+	АдресКаталогаОбновлений_Индекс = 3;
+
 	Совпадения = РВ_Версия.НайтиСовпадения(ТекстОписания);
 
 	Если Совпадения.Количество() > 0 Тогда
-		ОписаниеКонфигурации.Поставщик               = Совпадения[0].Группы[1].Значение;
-		ОписаниеКонфигурации.Версия                  = Совпадения[0].Группы[2].Значение;
-		ОписаниеКонфигурации.АдресКаталогаОбновлений = Совпадения[0].Группы[3].Значение;
+		ОписаниеКонфигурации.Поставщик               = Совпадения[0].Группы[Поставщик_Индекс].Значение;
+		ОписаниеКонфигурации.Версия                  = Совпадения[0].Группы[Версия_Индекс].Значение;
+		ОписаниеКонфигурации.АдресКаталогаОбновлений = Совпадения[0].Группы[АдресКаталогаОбновлений_Индекс].Значение;
 	КонецЕсли;
 
 	Совпадения = РВ_РежимСовместимости.НайтиСовпадения(ТекстОписания);

diff --git a/src/core/Модули/РаботаСИБ.os b/src/core/Модули/РаботаСИБ.os
@@ -174,10 +174,11 @@
 			УдалитьФайлы(ТекКаталог.ПолноеИмя);
 			Лог.Информация("Удален каталог кэша ""%1"".", ТекКаталог.ПолноеИмя);
 		Исключение
+			ТекстОшибки = ПодробноеПредставлениеОшибки(ИнформацияОбОшибке());
 			Лог.Предупреждение("Ошибка удаления каталога кэша ""%1"": %2%3",
 			                   ТекКаталог.ПолноеИмя,
 			                   Символы.ПС,
-			                   ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+			                   ТекстОшибки);
 		КонецПопытки;
 
 	КонецЦикла;