349 / добавлен метод генерации комбинаций вариантов #454
Conversation
WalkthroughВ данном пулл-запросе внесены изменения в файл Changes
Assessment against linked issues
Possibly related PRs
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
SonarQube Quality Gate failed! 
No data Coverage
| Варианты.КоличествоРеквизитов = Вариант.Количество(); | ||
| Попытка | ||
| //@skip-check module-unused-local-variable | ||
| Проверка = Новый Структура(Реквизиты); |
There was a problem hiding this comment.
🔗Удалите неиспользуемую переменную Проверка
badpractice brainoverload unused Why is this an issue?
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (2)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl (1)
74-86: Рефакторинг функцииДобавитьдля устранения дублирования кодаВ условных операторах функции
Добавитьприсутствует дублирование кода при создании структурыВариант. Рекомендуется упростить логику, используя массив параметров и цикл для создания структуры с переменным количеством реквизитов.Предлагаемый рефакторинг:
Параметры = Новый Массив(); Параметры.Добавить(Параметр1); Параметры.Добавить(Параметр2); Параметры.Добавить(Параметр3); Параметры.Добавить(Параметр4); Параметры.Добавить(Параметр5); Параметры.Добавить(Параметр6); Параметры.Добавить(Параметр7); Параметры = Параметры[0:Варианты.КоличествоРеквизитов - 1]; Вариант = Новый Структура(Варианты.СтрокаРеквизиты, Параметры);tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl (1)
56-82: Проверка генерации комбинаций вДобавитьКомбинации_2х3Тест проверяет корректность генерации комбинаций для двух реквизитов с тремя значениями каждого. Логика теста верна, ожидаемый результат соответствует заданным параметрам.
Рекомендуется добавить проверку на случай, если массивы значений реквизитов пусты или содержат
Неопределено.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (5)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl(4 hunks)tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl(1 hunks)tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/ОМ_ЮТКонструкторВариантов.mdo(1 hunks)tests/src/CommonModules/ОМ_ЮТЧитательСлужебный/Module.bsl(1 hunks)tests/src/Configuration/Configuration.mdo(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/ОМ_ЮТКонструкторВариантов.mdo
🔇 Additional comments (8)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl (2)
30-35: Улучшение обработки ошибок при инициализации структуры
Добавление блока Попытка...Исключение при инициализации структуры Проверка = Новый Структура(Реквизиты); обеспечивает более надежную обработку некорректных имен реквизитов, улучшая устойчивость функции Варианты.
160-179: Проверка корректности рекурсии в процедуре ДобавитьВарианты
В процедуре ДобавитьВарианты используется рекурсия для генерации комбинаций вариантов. Рекомендуется убедиться, что базовый случай и условия рекурсии определены корректно, чтобы избежать возможных ошибок переполнения стека или неверного вычисления комбинаций.
tests/src/Configuration/Configuration.mdo (1)
48-48: Добавление нового общего модуля ОМ_ЮТКонструкторВариантов
Включение нового модуля в конфигурацию корректно и синхронизировано с остальными изменениями в коде.
tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl (3)
21-33: Корректная регистрация исполняемых сценариев в процедуре ИсполняемыеСценарии
Процедура ИсполняемыеСценарии правильно добавляет тесты для исполнения, имена тестов и их представления установлены корректно.
37-54: Проверка процедуры Варианты на корректность инициализации контекста
Процедура Варианты корректно инициализирует контекст вариантов и проверяет его параметры. Ассерты в тесте подтверждают правильность работы функции.
85-110: Тестирование комбинаций с тремя реквизитами в ДобавитьКомбинации_3х2
Тест успешно проверяет генерацию комбинаций для трех реквизитов с двумя значениями. Код теста корректен.
tests/src/CommonModules/ОМ_ЮТЧитательСлужебный/Module.bsl (2)
93-93: Обновление ожидаемого количества наборов тестов
В процедуре ЗагрузитьТесты изменено ожидаемое количество наборов тестов с 39 на 40. Это корректно, если в системе действительно добавлен новый набор тестов.
Line range hint 123-135: Добавлена процедура ИсполняемыеСценарииМодуляСОшибкойЧтения для обработки ошибок чтения
Добавление новой процедуры для тестирования сценариев с ошибкой чтения повышает надежность и полноту тестирования.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (3)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl (1)
184-184: Неочевидное присваивание результата сравнения переменнойВ строке 184 выражение
ПоследняяИтерация = Инд = Ключи.ВГраница();может быть неправильно истолковано и усложняет понимание кода. Рекомендуется явно выполнить сравнение и присвоение для повышения читаемости.Примените следующий дифф для исправления:
- ПоследняяИтерация = Инд = Ключи.ВГраница(); + ПоследняяИтерация = (Инд = Ключи.ВГраница());documentation/docs/features/variants.md (2)
29-30: Повторяющиеся вызовы.СПараметрамис одинаковыми значениямиВ строках 29 и 30 метод
.СПараметрами("1", 1, Ложь)вызывается дважды с идентичными параметрами. Возможно, это опечатка. Рекомендуется проверить необходимость дублирования и при необходимости удалить один из вызовов или изменить параметры.
155-155: Орфографическая ошибка в текстеВ строке 155 обнаружена ошибка: следует использовать слово в единственном числе.
Примените следующий дифф для исправления:
- * Падения теста на одном из вариантов не влияет на другие. + * Падение теста на одном из вариантов не влияет на другие.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
documentation/docs/features/variants.md(1 hunks)exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl(5 hunks)
🔇 Additional comments (2)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl (2)
134-134: 🛠️ Refactor suggestion
Использование магического числа "7"
В строке 134 используется магическое число 7 для задания количества параметров. Рекомендуется создать константу с понятным названием, например МаксимальноеКоличествоПараметров, и использовать её вместо числа 7. Это улучшит читаемость и поддерживаемость кода.
Примените следующий дифф для исправления:
+ Константа МаксимальноеКоличествоПараметров = 7;
...
- КоличествоПараметров = 7;
+ КоличествоПараметров = МаксимальноеКоличествоПараметров;126-132: 🛠️ Refactor suggestion
Слишком много необязательных параметров в функции ДобавитьКомбинации
Функция ДобавитьКомбинации принимает 7 необязательных параметров, что превышает рекомендуемое количество. Рекомендуется уменьшить количество необязательных параметров до 3 или менее и передавать параметры в виде массива или структуры. Это упростит вызов функции и дальнейшую поддержку кода.
Пример исправления:
Функция ДобавитьКомбинации(МассивПараметров) Экспорт
// МассивПараметров - Массив из массивов значений параметров
// Дальнейший код адаптировать для работы с массивом параметров
КонецФункции
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (2)
tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl (2)
28-28: Удалите лишний пробел в названии тестаВ строке 28 в представлении теста имеется лишний пробел в конце строки "Комбинация из одного реквизита ". Это может привести к некорректному отображению названия теста.
Предлагаемая правка:
- .ДобавитьТест("ДобавитьКомбинации_1х1").Представление("Комбинация из одного реквизита ") + .ДобавитьТест("ДобавитьКомбинации_1х1").Представление("Комбинация из одного реквизита")
57-156: Устраните дублирование кода в тестовых процедурахТестовые процедуры
ДобавитьКомбинации_2х3,ДобавитьКомбинации_3х2,ДобавитьКомбинации_1х1иДобавитьКомбинации_РазнаяРазмерностьсодержат повторяющийся код для подготовки ожидаемых результатов и вызова методов. Рекомендуется обобщить их, используя параметризованные тесты или вспомогательные функции, чтобы повысить читаемость и упростить поддержку кода.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl(5 hunks)tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl(1 hunks)
🔇 Additional comments (1)
exts/yaxunit/src/CommonModules/ЮТКонструкторВариантов/Module.bsl (1)
32-32: Удалите неиспользуемую переменную Проверка
Переменная Проверка в строке 32 не используется после инициализации. Рекомендуется удалить переменную или переписать код без ее использования.
| Процедура Добавить() Экспорт | ||
|
|
||
| Варианты = ЮТКонструкторВариантов.Варианты("Р1, Р2, Р3") | ||
| .Добавить(1, , 3, 4) |
There was a problem hiding this comment.
Исправьте синтаксис вызова метода с пропущенным параметром
В строке 161 при вызове метода Добавить пропущен второй параметр без указания значения: .Добавить(1, , 3, 4). В языке 1С следует явно указывать Неопределено для пропущенных параметров, чтобы избежать ошибок во время выполнения.
Предлагаемая правка:
- .Добавить(1, , 3, 4)
+ .Добавить(1, Неопределено, 3, 4)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| .Добавить(1, , 3, 4) | |
| .Добавить(1, Неопределено, 3, 4) |
| Функция ДобавитьКомбинации(ЗначенияПараметра1, // BSLLS:NumberOfOptionalParams-off | ||
| ЗначенияПараметра2 = Неопределено, | ||
| ЗначенияПараметра3 = Неопределено, | ||
| ЗначенияПараметра4 = Неопределено, | ||
| ЗначенияПараметра5 = Неопределено, | ||
| ЗначенияПараметра6 = Неопределено, | ||
| ЗначенияПараметра7 = Неопределено) Экспорт | ||
|
|
||
| КоличествоПараметров = 7; | ||
|
|
||
| ВсеПараметры = Новый Массив(КоличествоПараметров); | ||
| ВсеПараметры[0] = ЗначенияПараметра1; | ||
| ВсеПараметры[1] = ЗначенияПараметра2; | ||
| ВсеПараметры[2] = ЗначенияПараметра3; | ||
| ВсеПараметры[3] = ЗначенияПараметра4; | ||
| ВсеПараметры[4] = ЗначенияПараметра5; | ||
| ВсеПараметры[5] = ЗначенияПараметра6; | ||
| ВсеПараметры[6] = ЗначенияПараметра7; | ||
|
|
||
| ПараметрыВариантов = ЮТКонтекстСлужебный.ЗначениеКонтекста(КлючКонтекста()); | ||
| ЗначенияПараметров = Новый Массив(ПараметрыВариантов.КоличествоРеквизитов); | ||
|
|
||
| Для Инд = 0 По ПараметрыВариантов.КоличествоРеквизитов - 1 Цикл | ||
| ЮТПроверкиСлужебный.ПроверитьТипПараметра(ВсеПараметры[Инд], Тип("Массив"), "ДобавитьКомбинации", "ЗначенияПараметра" + (Инд + 1)); | ||
| ЗначенияПараметров[Инд] = ВсеПараметры[Инд]; | ||
| КонецЦикла; | ||
|
|
||
| ДобавитьВарианты(ПараметрыВариантов.Варианты, | ||
| ЗначенияПараметров, | ||
| ПараметрыВариантов.Реквизиты, | ||
| 0, | ||
| Новый Структура(ПараметрыВариантов.СтрокаРеквизиты)); | ||
|
|
||
| Возврат ЮТКонструкторВариантов; | ||
|
|
||
| КонецФункции |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Используйте массив параметров вместо множества опциональных
Метод ДобавитьКомбинации принимает до семи опциональных параметров, что усложняет его расширение и поддержку. Рекомендуется изменить сигнатуру метода так, чтобы он принимал массив или коллекцию параметров. Это позволит работать с произвольным количеством параметров и упростит логику функции.
Предлагаемая сигнатура метода:
Функция ДобавитьКомбинации(МассивЗначенийПараметров) Экспорт
// Реализация метода с использованием МассивЗначенийПараметров
КонецФункции349 / добавлен метод генерации комбинаций вариантов
Описание изменений
Close #349
Summary by CodeRabbit
Новые функции
ДобавитьКомбинациидля обработки массивов параметров.ИсполняемыеСценарии,Варианты,ДобавитьКомбинации_2х3,ДобавитьКомбинации_3х2,ДобавитьКомбинации_1х1,ДобавитьКомбинации_РазнаяРазмерность,Добавить, иСписокВариантов.Исправления ошибок
Вариантыс более информативными сообщениями.Документация
Общие изменения