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

BSL002 — Ограничение на размер метода

Кратко

Ограничение на размер метода

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

Поле Значение
Код правила BSL002
Совместимый псевдоним MethodSize
Серьёзность ERROR
Включено по умолчанию Да
Реализовано Да
Теги size, brain-overload

Поведение

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

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

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

[tool.onec-hbk-bsl]
select = ["BSL002"]
ignore = ["MethodSize"]

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

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

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

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

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

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

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

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

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

Существуют громоздкие методы (процедуры и функции), с которыми невозможно эффективно работать именно из-за их огромного размера. Большой метод зачастую возникает, когда разработчик добавляет в метод новый функционал. "Зачем мне выносить проверку параметров в отдельный метод, если я могу написать ее тут?", "Для чего необходимо выделять метод поиска максимального элемента в массиве, оставим его тут. Так код яснее", - и прочие заблуждения.

Есть два правила рефакторинга большого метода:

  • Если при написании метода хочется добавить комментарий в код, необходимо выделить этот функционал в отдельный метод
  • Если метод занимает более 50-100 строк кода, следует определить задачи и подзадачи, которые он выполняет, и попробовать вынести подзадачи в отдельный метод

Источники