BSL240 - Перезапись параметров метода¶

Кратко¶

Перезапись параметров метода

Идентификаторы¶

Поле	Значение
Код правила	`BSL240`
Совместимый псевдоним	`RewriteMethodParameter`
Серьёзность	`WARNING`
Без конфигурации	Да
BSLLS JSON, режим `ON` без переопределений	Да
Флаг реестра `implemented`	Да
Теги	`suspicious`, `correctness`

Флаг implemented отражает реестр и не подтверждает проверенное совпадение с BSLLS.

Поведение¶

Публичный идентификатор BSL240 и псевдоним RewriteMethodParameter стабильны.
Правило сообщает о случаях, описанных на этой странице.
Подавления и проектная конфигурация применяются до публикации результата.
Для выполнения правила не требуется внешний анализатор или сетевой доступ.

Настройка и подавление¶

Код BSL### - основной стабильный идентификатор. Совместимый псевдоним принимается в select, ignore и совместимых блоковых комментариях.

[tool.onec-hbk-bsl]
select = ["BSL240"]
ignore = ["RewriteMethodParameter"]

Все три семейства подавлений работают для текущей строки и диапазона. Если открывающий комментарий стоит после кода, он действует только на эту строку. Используйте любой один вариант:

noqa:

Значение = "пример";  // noqa: BSL240

bsl-disable (совместимый вариант):

Значение = "пример";  // bsl-disable: BSL240

совместимый BSLLS-вариант:

Значение = "пример";  // BSLLS:RewriteMethodParameter-off

Если тот же открывающий комментарий стоит на отдельной строке, он начинает диапазон. Закройте его парным маркером того же семейства:

// noqa: BSL240
// код без этой диагностики
// noqa-enable: BSL240

// bsl-disable: BSL240
// код без этой диагностики
// bsl-enable: BSL240

// BSLLS:RewriteMethodParameter-off
// код без этой диагностики
// BSLLS:RewriteMethodParameter-on

Чтобы отключить правило до конца файла, не добавляйте закрывающий noqa-enable, bsl-enable или BSLLS:…-on.

Открывающий и закрывающий маркеры должны принадлежать одному семейству.

Описание диагностики¶

Иногда разработчик пишут функции таким образом, когда аргументы функции перезаписываются сразу на входе в функцию/процедуру.

Такое поведение вводит в заблуждение других разработчиков, которые вызывают подобные функции/процедуры. Эти функции нужно исправить. Например, убрать параметры, преобразовав их в локальные переменные.

Примеры¶

Подозрительный код

Функция Конфигуратор(Знач СтрокаПодключения, Знач Пользователь = "", Знач Пароль = "") Экспорт
  СтрокаПодключения = "/F""" + КаталогБазы + """"; // Здесь
...
КонецФункции

Исправленный код

Функция Конфигуратор(Знач КаталогБазы, Знач Пользователь = "", Знач Пароль = "") Экспорт
  СтрокаПодключения = "/F""" + КаталогБазы + """"; // Здесь
...
КонецФункции

или

Функция Конфигуратор(Знач КаталогБазы, Знач Пользователь = "", Знач Пароль = "") Экспорт
  Если Не ПустаяСтрока(КаталогБазы) Тогда
   НоваяСтрокаПодключения = "/F""" + КаталогБазы + """";
  Иначе
   НоваяСтрокаПодключения = СтрокаПодключения;
  КонецЕсли;
...
КонецФункции