Функция запуска проверки конфигурации #137

Golovanoff · 2022-02-17T08:50:02Z

Имена проверок сделал такие же, как их видно в конфигураторе при ручном запуске, но без пробелов и скобочек.
Работает аналогично процедуре ВыполнитьСинтаксическийКонтроль, но можно указать все доступные ключи проверки конфигурации.

Пример вызова проверки:

Конфигуратор = Новый УправлениеКонфигуратором;
Конфигуратор.УстановитьКонтекст(СтрокаСоединения, Пользователь, Пароль);

Проверки = Новый Массив();
Проверки.Добавить("Сервер");
Проверки.Добавить("ТолстыйКлиентОбычноеПриложение");
Проверки.Добавить("ПроверкаСуществованияНазначенныхОбработчиков");
Проверки.Добавить("ПоискПустыхОбработчиков");
Проверки.Добавить("РасширеннаяПроверка");

Конфигуратор.ПроверкаКонфигурации(Проверки);

EvilBeaver · 2022-02-17T09:13:04Z

src/v8runner.os

+	ДопустимыеКлючи.Вставить("ПоискИспользованияМодальности", "-CheckUseModality");
+	ДопустимыеКлючи.Вставить("ПоискИспользованияСинхронныхВызовов", "-CheckUseSynchronousCalls");
+	ДопустимыеКлючи.Вставить("ПоискНеподдерживаемойФункциональности", "-UnsupportedFunctional");
+	ДопустимыеКлючи.Вставить("ВыполнитьЗаданныеПроверкиДляВсехРасширений", "-AllExtensions");


Может покороче назвать как-то? ПроверкаВсехРасширений/ПроверкаРасширений?

можно наделать в соответствии сколько угодно записей. я добавил так, как это в конфигураторе у меня называется.

Ну а зачем нам неиспользуемые ключи и синонимы?

EvilBeaver · 2022-02-17T09:15:44Z

src/v8runner.os

@@ -1459,6 +1459,50 @@

 КонецПроцедуры

+Функция ПроверкаКонфигурации(Проверки, ИмяРасширения = "") Экспорт


Мне кажется, было бы удобнее, чтобы пользователь метода не запоминал все возможные настройки и получал приемлемый результат. Например, есть какой-то метод, который возвращает предзаполненные "Проверки" наиболее употребимым составом проверок.

ну а какие предзаполненные? у меня, например, тослтый клиент, у меня свои предзаполненные. у остальных - тонкие клиенты и там свои. ну и есть же старая ВыполнитьСинтаксическийКонтроль, там как раз предзаполненность для всех.

да и мы же не для пользователей пишем, а для суровых бородатых прогеров.

можно, наверное, при пустом массиве ставить проверки такие же, как в конфигураторе по умолчанию ставятся - проверку лог. целостности, поиск некорректных ссылок, тонкий клиент, сервер и расширенную проверку.
но это будет долгая проверка - у меня при установленной расширенной проверке джоба работает около часа.

я также хотел предложить, чтобы был публичный метод, возвращающий соответствие с возможными ключами

исправлено. @EvilBeaver закрываем обсуждение?

Как именно исправлено? где?

Как именно исправлено? где?

@EvilBeaver исправлено в c268631

src/v8runner.os

artbear

еще немного замечаний

artbear · 2022-02-17T11:53:20Z

src/v8runner.os

+	Возврат Истина;
+КонецФункции
+
+Функция КлючиПроверкиКонфигурации() Экспорт


а где описание публичной функции? )

artbear · 2022-02-17T11:54:02Z

src/v8runner.os

+		КлючПроверки = ДопустимыеКлючи.Получить(СокрЛП(Проверка)); 
+
+		Если КлючПроверки <> Неопределено Тогда
+			КлючЗапуска = ?(НРег(КлючПроверки) = "-extension", СтрШаблон("%1 %2", КлючПроверки, ИмяРасширения), КлючПроверки);


Сонара на тебя нет )
тернарные операторы не юзаем.

2.1 регистр важен ИМХО

1С-ка в каких-то версиях ругалась, если ключ в другом регистре.
могли и исправить, конечно.

2 предлагаю явно написать СтрШаблон("-Extension %1", ИмяРасширения)

Артур, чего это мы ?() вдруг не юзаем? Вложенные - да, а тут обычный

сонар бсл и на обычный тернарник ругается. и я с ним согласен, код гораздо менее читабелен.

Запрет тернарника по умолчанию выключен.

artbear · 2022-02-17T11:57:05Z

src/v8runner.os

@@ -1459,6 +1459,102 @@

 КонецПроцедуры

+Функция ПроверкаКонфигурации(Проверки, ИмяРасширения = "") Экспорт


нет описания публичной функции

artbear · 2022-03-06T16:10:06Z

Эх, ну почему в мастер-то?? теперь девелоп отстал от мастера (

накидал функцию

953df59

EvilBeaver requested changes Feb 17, 2022

View reviewed changes

artbear requested changes Feb 17, 2022

View reviewed changes

src/v8runner.os Outdated Show resolved Hide resolved

src/v8runner.os Show resolved Hide resolved

вынес допустимые ключи в отдельную функцию

c268631

artbear requested changes Feb 17, 2022

View reviewed changes

EvilBeaver approved these changes Feb 22, 2022

View reviewed changes

EvilBeaver merged commit 3e5e1fc into oscript-library:master Feb 22, 2022

artbear mentioned this pull request Jun 19, 2022

v1.9.0 - Поддержка параллельной загрузки из dt, ПроверкаКонфигурации и т.д. #144

Merged

artbear added this to the v1.9.0 milestone Jun 19, 2022

		@@ -1459,6 +1459,50 @@

		КонецПроцедуры

		Функция ПроверкаКонфигурации(Проверки, ИмяРасширения = "") Экспорт

		@@ -1459,6 +1459,102 @@

		КонецПроцедуры

		Функция ПроверкаКонфигурации(Проверки, ИмяРасширения = "") Экспорт

Функция запуска проверки конфигурации #137

Функция запуска проверки конфигурации #137

Uh oh!

Conversation

Golovanoff commented Feb 17, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

artbear commented Mar 6, 2022

Uh oh!

Uh oh!