BSL194 - Функция всегда возвращает одно и то же примитивное значение¶

Кратко¶

Функция всегда возвращает одно и то же примитивное значение

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

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

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

Поведение¶

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

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

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

[tool.onec-hbk-bsl]
select = ["BSL194"]
ignore = ["FunctionReturnsSamePrimitive"]

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

noqa:

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

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

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

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

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

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

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

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

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

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

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

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

Функция не должна возвращать одно и тоже примитивное значение. Если результат функции не учитывается - то нужно функцию переписать на процедуру.

Примеры¶

Плохо:

Функция ПроверитьСтроку(Знач СтрокаТаблицы)

    Если ЭтоХорошаяСтрока(СтрокаТаблицы) Тогда
        ДелаемЧтоТоЧтобыНеРугаласьПроверкаНаПовтор();
        Возврат Истина;
    ИначеЕсли ЭтоТожеНеплохаяСтрока(СтрокаТаблицы) Тогда
        ДелаемДругоеЧтоТоЧтобыНеРугаласьПроверкаНаПовтор();
        Возврат Истина;
     Иначе
        Возврат Истина;
    КонецЕсли;

КонецФункции

Хорошо:

Процедура ПроверитьСтроку(Знач СтрокаТаблицы)

    Если ЭтоХорошаяСтрока(СтрокаТаблицы) Тогда
        ДелаемЧтоТоЧтобыНеРугаласьПроверкаНаПовтор();
    ИначеЕсли ЭтоТожеНеплохаяСтрока(СтрокаТаблицы) Тогда
        ДелаемДругоеЧтоТоЧтобыНеРугаласьПроверкаНаПовтор();
    Иначе
        ДелаемЧтотоДругоеИзВеткиИначе();
    КонецЕсли;

КонецПроцедуры

Особенности¶

Из проверки исключаются подключаемые Функции. Например:

Функция Подключаемый_КакаяТоКоманда(Команда)

    Если ЗначениеЗаполнено(ТекущаяДата) Тогда
        Возврат Неопределено;
    КонецЕсли;

    Возврат Неопределено;

КонецФункции