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

BSL260 — Небезопасное использование метода НайтиПоКоду()

← Все правила

Кратко

Небезопасное использование метода НайтиПоКоду()

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

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

Поведение

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

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

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

[tool.onec-hbk-bsl]
select = ["BSL260"]
ignore = ["UnsafeFindByCode"]

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

  • noqa:
Значение = "пример";  // noqa: BSL260
  • bsl-disable (совместимый вариант):
Значение = "пример";  // bsl-disable: BSL260
  • совместимый BSLLS-вариант:
Значение = "пример";  // BSLLS:UnsafeFindByCode-off

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

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

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

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

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

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

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

Диагностика обнаруживает использование метода НайтиПоКоду() (или FindByCode()) для справочников, у которых:

  • отключен контроль уникальности кода (свойство Контроль уникальности установлено в значение Ложь)
  • или включены серии кодов не по всему справочнику (свойство Серии кодов не равно ПоВсемуСправочнику)

В таких случаях использование метода НайтиПоКоду() может привести к неожиданному поведению, так как код может быть не уникальным в пределах всего справочника или может существовать несколько элементов с одинаковым кодом в разных сериях.

Примеры

Неправильно:

// Справочник без контроля уникальности
СправочникБезКонтроля = Справочники.СправочникБезКонтроляУникальности.НайтиПоКоду("001");

// Справочник с сериями кодов "В пределах подчинения"
СправочникССериями = Справочники.СправочникССериямиПодчинения.НайтиПоКоду("001");

Правильно:

// Использование метода НайтиПоКоду() для справочника с контролем уникальности
// и сериями кодов по всему справочнику
СправочникСКонтролем = Справочники.СправочникСКонтролемУникальности.НайтиПоКоду("001");

// Альтернативный вариант - использовать метод НайтиПоНаименованию() или другие методы поиска
Справочник = Справочники.СправочникБезКонтроляУникальности.НайтиПоНаименованию("Элемент");

Источники