Небольшой рефакторинг

Author: otymko

packagedef

@@ -7,7 +7,7 @@
 	.Автор("otymko")
 	.АдресАвтора("olegtymko@yandex.ru")
 	.Описание("Версионирование правил обмена 1С с помощью git")
-	.ВерсияСреды("1.0.11")
+	.ВерсияСреды("1.0.21")
 	.ВключитьФайл("src")
 	.ВключитьФайл("docs")
 	.ВключитьФайл("tests")
@@ -20,7 +20,6 @@
 	.ОпределяетКласс("ОбработкаПравил", "src/core/Классы/ОбработкаПравил.os")
 	.ОпределяетМодуль("РазобратьПравилаОбмена", "src/core/Модули/РазобратьПравилаОбмена.os")
 	.ОпределяетМодуль("СобратьПравилаОбмена", "src/core/Модули/СобратьПравилаОбмена.os")
-	.ОпределяетМодуль("ОбщиеУтилиты", "src/core/Модули/ОбщиеУтилиты.os")
 	.ОпределяетМодуль("ОбщийФункционал", "src/core/Модули/ОбщийФункционал.os")
 	.ИсполняемыйФайл("src/cmd/gitrules.os", ИмяПродукта)
 ;

src/cmd/gitrules.os

@@ -23,11 +23,30 @@
     Приложение = Новый КонсольноеПриложение(ИмяПродукта, "Версионирование правил обмена 1С с помощью git");
     Приложение.Версия("v version", ПараметрыПриложения.ВерсияПродукта());
 
-    Приложение.ДобавитьКоманду("i install", СтрШаблон("Установить %1 в текущий репозиторий", ИмяПродукта), Новый КомандаInstall);
-    Приложение.ДобавитьКоманду("r remove", СтрШаблон("Удалить %1 в текущем репозитории", ИмяПродукта), Новый КомандаRemove);
-	Приложение.ДобавитьКоманду("e export", "Распаковка правил обмена xml на файлы и каталоги", Новый КомандаExport);
-    Приложение.ДобавитьКоманду("a assembly", "Сборка из каталогов и файлов правил xml", Новый КомандаAssembly);
-    Приложение.ДобавитьКоманду("p precommit", "Разборка правил обмена xml без git commit", Новый КомандаPrecommit);
+    Приложение.ДобавитьКоманду(
+        "i install", 
+        СтрШаблон("Установить %1 в текущий репозиторий", ИмяПродукта), 
+        Новый КомандаInstall);
+
+    Приложение.ДобавитьКоманду(
+        "r remove", 
+        СтрШаблон("Удалить %1 в текущем репозитории", ИмяПродукта), 
+        Новый КомандаRemove);
+
+	Приложение.ДобавитьКоманду(
+        "e export", 
+        "Распаковка правил обмена xml на файлы и каталоги", 
+        Новый КомандаExport);
+
+    Приложение.ДобавитьКоманду(
+        "a assembly", 
+        "Сборка из каталогов и файлов правил xml", 
+        Новый КомандаAssembly);
+
+    Приложение.ДобавитьКоманду(
+        "p precommit", 
+        "Разборка правил обмена xml без git commit",
+        Новый КомандаPrecommit);
 
 	Приложение.УстановитьОсновноеДействие(ЭтотОбъект);
 

src/core/Классы/ОбработкаПравил.os

@@ -56,6 +56,7 @@
 		Попытка
 			УдалитьФайлы(КаталогПроекта);
 		Исключение
+			Сообщить("Не удалось удалить каталог " + КаталогПроекта);
 		КонецПопытки;
 	КонецЕсли;
 	СоздатьКаталог(КаталогПроекта);
@@ -83,9 +84,9 @@
 	Возврат Массив;
 КонецФункции
 
-Функция РазложитьФайлПравила(ПутьКФайлу) Экспорт
+Процедура РазложитьФайлПравила(ПутьКФайлу) Экспорт
 	РазобратьПравилаОбмена.ВыполнитьРазбор(ОсновнойКаталогПравил, ПутьКФайлу);
-КонецФункции
+КонецПроцедуры
 
 Процедура ДобавитьПравилаВИндексGIT() Экспорт
 	Команда = Новый Команда;

src/core/Модули/ОбщийФункционал.os

@@ -45,7 +45,7 @@
 
 	ИменаРеквизитовКонвертации = Новый Массив;
 
-	//Реквизиты конвертации
+	// Реквизиты конвертации
 	ИменаРеквизитовКонвертации.Добавить("ВерсияФормата");
 	ИменаРеквизитовКонвертации.Добавить("Ид");
 	ИменаРеквизитовКонвертации.Добавить("Наименование");
@@ -73,7 +73,7 @@
 
 	МассивОбработчиков = Новый Массив;
 
-	//Конвертация
+	// Конвертация
 	МассивОбработчиков.Добавить("ПослеЗагрузкиПравилОбмена");
 	МассивОбработчиков.Добавить("ПередВыгрузкойДанных");
 	МассивОбработчиков.Добавить("ПередПолучениемИзмененныхОбъектов");
@@ -90,25 +90,25 @@
 	МассивОбработчиков.Добавить("ПослеЗагрузкиПараметров");
 	МассивОбработчиков.Добавить("ПослеПолученияИнформацииОбУзлахОбмена");
 
-	//ГруппаСвойств
+	// ГруппаСвойств
 	МассивОбработчиков.Добавить("ПередОбработкойВыгрузки");
 	МассивОбработчиков.Добавить("ПередВыгрузкойСвойства");
 	МассивОбработчиков.Добавить("ПриВыгрузкеСвойства");
 	МассивОбработчиков.Добавить("ПослеВыгрузкиСвойства");
 	МассивОбработчиков.Добавить("ПослеОбработкиВыгрузки");
 
-	//Свойства
+	// Свойства
 	МассивОбработчиков.Добавить("ПередВыгрузкойСвойства");
 	МассивОбработчиков.Добавить("ПриВыгрузкеСвойства");
 	МассивОбработчиков.Добавить("ПослеВыгрузкиСвойства");
 
-	//ПравилаВыгрузкиДанных
+	// ПравилаВыгрузкиДанных
 	МассивОбработчиков.Добавить("ПередОбработкойПравила");
 	МассивОбработчиков.Добавить("ПослеОбработкиПравила");
 	МассивОбработчиков.Добавить("ПередВыгрузкойОбъекта");
 	МассивОбработчиков.Добавить("ПослеВыгрузкиОбъекта");
 
-	//ПравилаКонвертацииОбъектов
+	// ПравилаКонвертацииОбъектов
 	МассивОбработчиков.Добавить("ПередВыгрузкой");
 	МассивОбработчиков.Добавить("ПриВыгрузке");
 	МассивОбработчиков.Добавить("ПослеВыгрузки");
@@ -123,23 +123,23 @@
 	МассивОбработчиков.Добавить("ПослеЗагрузкиОбъекта");
 	МассивОбработчиков.Добавить("ПоследовательностьПолейПоиска");
 
-	//ПравилаРегистрацииОбъектов
+	// ПравилаРегистрацииОбъектов
 	МассивОбработчиков.Добавить("ПередОбработкой");
 	МассивОбработчиков.Добавить("ПриОбработке");
 	МассивОбработчиков.Добавить("ПриОбработкеДополнительный");
 	МассивОбработчиков.Добавить("ПослеОбработки");
 
-	//ПравилаОчисткиОбъектов
+	// ПравилаОчисткиОбъектов
 	МассивОбработчиков.Добавить("ПередОбработкойПравила");
 	МассивОбработчиков.Добавить("ПередУдалениемОбъекта");
 	МассивОбработчиков.Добавить("ПослеОбработкиПравила");
 
-	//Параметры
+	// Параметры
 	МассивОбработчиков.Добавить("ПослеЗагрузкиПараметра");
 
 	СтруктураНастроек.Вставить("ИменаСобытийКонвертации", МассивОбработчиков);
 
-	//Исключения для выгрузки в XML
+	// Исключения для выгрузки в XML
 	ИменаИсключений = Новый Массив;
 	ИменаИсключений.Добавить("Правило");
 	ИменаИсключений.Добавить("Запрос");
@@ -158,48 +158,6 @@
 
 КонецФункции
 
-Функция РазложитьСтрокуВМассивПодстрок(Знач Строка, Знач Разделитель = ",", Знач ПропускатьПустыеСтроки = Неопределено, СокращатьНепечатаемыеСимволы = Ложь) Экспорт
-	
-	Результат = Новый Массив;
-	
-	// Для обеспечения обратной совместимости.
-	Если ПропускатьПустыеСтроки = Неопределено Тогда
-		ПропускатьПустыеСтроки = ?(Разделитель = " ", Истина, Ложь);
-		Если ПустаяСтрока(Строка) Тогда 
-			Если Разделитель = " " Тогда
-				Результат.Добавить("");
-			КонецЕсли;
-			Возврат Результат;
-		КонецЕсли;
-	КонецЕсли;
-	//
-	
-	Позиция = СтрНайти(Строка, Разделитель);
-	Пока Позиция > 0 Цикл
-		Подстрока = Лев(Строка, Позиция - 1);
-		Если Не ПропускатьПустыеСтроки Или Не ПустаяСтрока(Подстрока) Тогда
-			Если СокращатьНепечатаемыеСимволы Тогда
-				Результат.Добавить(СокрЛП(Подстрока));
-			Иначе
-				Результат.Добавить(Подстрока);
-			КонецЕсли;
-		КонецЕсли;
-		Строка = Сред(Строка, Позиция + СтрДлина(Разделитель));
-		Позиция = СтрНайти(Строка, Разделитель);
-	КонецЦикла;
-	
-	Если Не ПропускатьПустыеСтроки Или Не ПустаяСтрока(Строка) Тогда
-		Если СокращатьНепечатаемыеСимволы Тогда
-			Результат.Добавить(СокрЛП(Строка));
-		Иначе
-			Результат.Добавить(Строка);
-		КонецЕсли;
-	КонецЕсли;
-	
-	Возврат Результат;
-	
-КонецФункции
-
 Процедура СоздатьКаталогРасширенный(ИмяКаталога) Экспорт
 
 	Файл = Новый Файл(ИмяКаталога);
@@ -212,7 +170,7 @@
 
 Функция ЭтотКаталогСуществует(Путь) Экспорт
 	Файл = Новый Файл(Путь);
-	Возврат Файл.Существует() и Файл.ЭтоКаталог();
+	Возврат Файл.Существует() И Файл.ЭтоКаталог();
 КонецФункции
 
 Процедура ЗаписатьЗначениеВТекстовыйДокумент(ИмяФайла, Содержимое) Экспорт