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

Локальные переменные не должны инициализироваться и тут же возвращаться из функции

Объявление локальной переменной только для того, чтобы немедленно вернуть из функции, является плохой практикой.

Часто использование подобного кода аргументируется тем, что для повышения читаемости полезно показать имя переменной, которая будет расшифровывать возвращаемый результат.

При это нужно учесть, что возвращаемый результат должен описываться именем самой функции. И нет необходимости добавлять описание дополнительно.

Пример подозрительного кода

Функция СложитьДваПараметра(Парам1, Парам2)
    A = Парам1 + Парам2;// лишняя переменная
    Возврат A;
КонецФункции

Более правильный и читабельный код

Функция СложитьДваПараметра(Парам1, Парам2)
    Возврат Парам1 + Парам2;
КонецФункции

См. также

Экранирование кода

// sonar:ImmediatelyReturnedVariable:off
// sonar:ImmediatelyReturnedVariable:on