Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Оптимизация механизмов движка #392

Merged
merged 4 commits into from
Aug 5, 2024

Conversation

alkoleft
Copy link
Member

@alkoleft alkoleft commented Aug 5, 2024

Описание изменений

  • Оптимизация метода "ЗначениеВМассиве"
  • Оптимизация запуска тестов из предприятия.
  • Оптимизация запуска предприятия без тестов

  • Добавлены тесты
  • Изменения отражены в документации (при необходимости)

Close #<Укажите номер issue для автоматического закрытия после принятия реквеста>

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SonarQube Code Analytics

🔗Quality Gate failed

failed

Additional information

The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.

Issues

bug 0 Bugs

vulnerability 0 Vulnerabilities

code_smell 3 Code Smells

Coverage and Duplications

No data Coverage

duplication_lt_3 0.00% Duplication


Если Значения = Неопределено Тогда
Значения = Новый Массив(- Инд);
Значения = Новый Массив(10);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔗Создайте константу с понятным названием, присвойте ей значение "10" и используйте эту константу вместо магического числа.

CODE_SMELL Codesmell  MINOR Minor   ⌛ 1min effort

badpractice parameters  Why is this an issue?

Значения[9] = Значение10;

Для Инд = -9 По 0 Цикл
Если Значения[-Инд] <> ФлагПустогоЗначения Тогда
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔗Слева от '-' не хватает пробела

CODE_SMELL Codesmell  INFO Info   ⌛ 1min effort

badpractice parameters  Why is this an issue?

@@ -21,6 +21,9 @@
&НаКлиенте
Перем ИсполняемыеТестовыеМодули;

&НаКлиенте
Перем ДанныеПоИдентификаторам;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔗Добавьте описание переменной

CODE_SMELL Codesmell  MINOR Minor   ⌛ 1min effort

standard  Why is this an issue?

@alkoleft alkoleft merged commit a70cd68 into develop Aug 5, 2024
2 of 5 checks passed
@alkoleft alkoleft deleted the feature/perfomance branch August 5, 2024 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant