Перейти к содержанию

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

Кратко

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

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

Поле Значение
Код правила BSL240
Совместимый псевдоним RewriteMethodParameter
Серьёзность WARNING
Включено по умолчанию Да
Реализовано Да
Теги suspicious, correctness

Поведение

  • Публичный идентификатор 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""" + КаталогБазы + """";
  Иначе
   НоваяСтрокаПодключения = СтрокаПодключения;
  КонецЕсли;
...
КонецФункции

Источники