Исправления по замечания coderabbit

Author: Stivo182

src/BenchmarkOneScript/cmd/Классы/КомандаЗапуска.os

@@ -115,7 +115,7 @@
 Функция ПодключитьТипы()
 
 	ПодключенныеТипы = Новый Массив();
-	Для Каждого ИмяФайла Из ФайлыБенчмарков() Цикл
+	Для Каждого ИмяФайла Из НайтиФайлыБенчмарков() Цикл
 		Тип = Бенчмаркинг.ПодключитьКласс(ИмяФайла);
 		Если Не Тип = Неопределено Тогда
 			ПодключенныеТипы.Добавить(Тип);
@@ -126,18 +126,23 @@
 
 КонецФункции
 
-Функция ФайлыБенчмарков()
+Функция НайтиФайлыБенчмарков()
 
 	Файлы = Новый Массив();
 
 	Для Каждого ИмяФайла Из _МассивФайлов Цикл
 		Если ФС.КаталогСуществует(ИмяФайла) Тогда
-			Файлы = НайтиФайлы(ИмяФайла, , _Рекурсивно);
-			Для Каждого ИмяФайлаВКаталоге Из Файлы Цикл
-				Файлы.Добавить(ИмяФайлаВКаталоге);
+			НайденныеФайлы = НайтиФайлы(ИмяФайла, , _Рекурсивно);
+			Для Каждого ИмяФайлаВКаталоге Из НайденныеФайлы Цикл
+				Если Файлы.Найти(ИмяФайлаВКаталоге) = Неопределено Тогда
+					Файлы.Добавить(ИмяФайлаВКаталоге);
+				КонецЕсли;
 			КонецЦикла;
 		Иначе
-			Файлы.Добавить(ПолноеИмяФайлаСценария(ИмяФайла));
+			ПолноеИмяФайла = ПолноеИмяФайлаСценария(ИмяФайла);
+			Если Файлы.Найти(ПолноеИмяФайла) = Неопределено Тогда
+				Файлы.Добавить(ПолноеИмяФайла);
+			КонецЕсли;
 		КонецЕсли;
 	КонецЦикла;
 

src/BenchmarkOneScript/core/Классы/СериализаторНастроекБенчмарков.os

@@ -44,7 +44,7 @@
 //   ИмяФайла - Строка - Имя файла, из которого будет прочитана конфигурация.
 //
 // Возвращаемое значение:
-//   БенчмаркиСКонфигурацией
+//   НастройкиБенчмарковДто
 Функция ПрочитатьИзJson(ИмяФайла) Экспорт
 
 	_Лог.Отладка("Начало чтения файла: %1", ИмяФайла);

src/BenchmarkOneScript/core/Модули/ИзвлекательКонфигурацииБенчмарков.os

@@ -34,24 +34,24 @@
 		ПрочитатьАннотациюМинимальноеКоличествоВызововЗаИтерацию(СвойстваКонструктора, КонфигурацияБенчмарков);
 		ПрочитатьАннотациюСортировка(СвойстваКонструктора, КонфигурацияБенчмарков);
 		ПрочитатьАннотациюМониторингПамяти(СвойстваКонструктора, КонфигурацияБенчмарков);	
-		ПрочитатьАннотацииКолонок(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюКаталогАртефактов(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюЭкспортMarkdown(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюЭкспортJson(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюЭкспортXml(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюЭкспортHtml(СвойстваКонструктора, Конфигурация);
-		ПрочитатьАннотациюИсполняющаяСреда(СвойстваКонструктора, Конфигурация);
+		ПрочитатьАннотацииКолонок(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюКаталогАртефактов(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюЭкспортMarkdown(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюЭкспортJson(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюЭкспортXml(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюЭкспортHtml(СвойстваКонструктора, КонфигурацияБенчмарков);
+		ПрочитатьАннотациюИсполняющаяСреда(СвойстваКонструктора, КонфигурацияБенчмарков);
 	КонецЕсли;
 
 	Для Каждого СвойстваПоля Из ТаблицаСвойств Цикл
 		ПрочитатьАннотациюПараметры(СвойстваПоля, КонфигурацияБенчмарков);
 		ПрочитатьАннотациюИсточникПараметров(СвойстваПоля, КонфигурацияБенчмарков);
 	КонецЦикла;
 
-	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПередВсеми, РефлекторОбъекта, Конфигурация);
-	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПослеВсех, РефлекторОбъекта, Конфигурация);
-	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПередКаждым, РефлекторОбъекта, Конфигурация);
-	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПослеКаждого, РефлекторОбъекта, Конфигурация);
+	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПередВсеми, РефлекторОбъекта, КонфигурацияБенчмарков);
+	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПослеВсех, РефлекторОбъекта, КонфигурацияБенчмарков);
+	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПередКаждым, РефлекторОбъекта, КонфигурацияБенчмарков);
+	ПрочитатьОбработчикиСобытия(СобытияБенчмарков.ПослеКаждого, РефлекторОбъекта, КонфигурацияБенчмарков);
 
 	Возврат КонфигурацияБенчмарков;
 

src/BenchmarkOneScript/core/Модули/Компаратор.os

@@ -3,8 +3,8 @@
 // Определяет, совпадают ли значения двух объектов по их внутреннему содержанию.
 //
 // Параметры:
-//   Значение1 - Проивзольный - Первый объект для сравнения
-//   Значение2 - Проивзольный - Второй объект для сравнения
+//   Значение1 - Произвольный - Первый объект для сравнения
+//   Значение2 - Произвольный - Второй объект для сравнения
 //
 // Возвращаемое значение:
 //   Булево

src/BenchmarkOneScript/dto/Классы/НастройкиБенчмарковДто.os

@@ -6,4 +6,9 @@
 
 &Сериализуемое
 &Тип("КонфигурацияБенчмарков")
-Перем Конфигурация Экспорт; // КонфигурацияБенчмарков
+Перем Конфигурация Экспорт; // КонфигурацияБенчмарков
+
+Процедура ПриСозданииОбъекта()
+	ДескрипторыБенчмарков = Новый КоллекцияДескрипторовБенчмарков();
+	Конфигурация = Новый КонфигурацияБенчмарков();
+КонецПроцедуры

tests/ТестыСериализаторБенчмарков.os

@@ -39,13 +39,13 @@
 
 	Ожидаем.Что(ДескрипторыБенчмарков.Количество()).Равно(1);
 
-	Дексриптор = ДескрипторыБенчмарков.ПолучитьПервый();
-	НаборыПараметров = Дексриптор.НаборыПараметров();
-	ИсточникиПараметров = Дексриптор.ИсточникиПараметров();
+	Дескриптор = ДескрипторыБенчмарков.ПолучитьПервый();
+	НаборыПараметров = Дескриптор.НаборыПараметров();
+	ИсточникиПараметров = Дескриптор.ИсточникиПараметров();
 
-	Ожидаем.Что(Дексриптор.ТипОбъекта()).Равно(Тип("БенчмаркСАннотациямиКонфигурации"));
-	Ожидаем.Что(Дексриптор.Метод()).Равно("Бенчмарк");
-	Ожидаем.Что(Дексриптор.ЭтоЭталон()).ЭтоЛожь();
+	Ожидаем.Что(Дескриптор.ТипОбъекта()).Равно(Тип("БенчмаркСАннотациямиКонфигурации"));
+	Ожидаем.Что(Дескриптор.Метод()).Равно("Бенчмарк");
+	Ожидаем.Что(Дескриптор.ЭтоЭталон()).ЭтоЛожь();
 
 	Ожидаем.Что(НаборыПараметров, "Наборы параметров").ИмеетДлину(2);
 	Ожидаем.Что(НаборыПараметров[0].Получить(0).Имя, "Наборы параметров [0].Имя").Равно("Парам");