Приветствую всех.

Некоторое время назад довелось заниматься code review команды разработчиков.
Наткнулся на то, что в коллективе была принята методология использования вызова функций вместо процедур независимо от того, возвращает ли что-то данная функция или нет.

То есть код

РассчитатьЗадолженность(); //где РассчитатьЗадолженность() - функция

Повсеместно.

У кого какой опыт, кто какой методологии придерживается? Может быть я консервативен, но зачем использовать не по назначению конструкции?

нужно следовать гайдлайнам языка

Да ващще, пишут как хотят. Никакого уважения.

функция [в 1с] всегда что-то возвращает, хотя бы Неопределено, т.е. можно к ней обратиться как к функции, а к процедуре уже нельзя

(11) "Процедура может поменять" - процедура ничего не может, если ты там не наговнокодишь. Посмотри в любой типовой обработку проведения, например.

Процедура ОбработкаПроведения(Отказ, РежимПроведения)

    // Инициализация дополнительных свойств для проведения документа
    ПроведениеСервер.ИнициализироватьДополнительныеСвойстваДляПроведения(Ссылка, ДополнительныеСвойства, РежимПроведения);
    
    
    // Движения по денежным средствам.
    ДенежныеСредстваСервер.ОтразитьДенежныеСредстваУПодотчетныхЛиц(ДополнительныеСвойства, Движения, Отказ);
    ДенежныеСредстваСервер.ОтразитьДенежныеСредстваКСписаниюСПодотчетныхЛиц(ДополнительныеСвойства, Движения, Отказ);

...

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





И это вполне себе логично.

В каждой "внутренней" процедура в начале добавляешь проверку на отказ и всё.

Напиши мне этот же код только на функциях. Я даже за попкорном схожу.

Если используют функции, там где это не обязательно, можно закрыть на это глаза. В каких то случаях это допустимо или это частный случай. Но если везде вместо процедур пишут исключительно функции, то это говнокод.

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

Тупейшая хрень. "А вот чо место простаивает, давайте его используем как возврат для ошибок". Тот кто мыслит строками может такое выдумать. Кто мыслит объектами и взаимодействиями, четко понимает чем одно отличается от другого и никогда на эту херню не подпишется.

функция которая ничего не возвращает это обман.
Ненаписание слова "Знач" это тоже обман.
Не надо никого обманывать.

бывает так, что нужен результат выполнения какой-либо операции, тогда вроде функция ведет себя как процедура, но возвращает какие данные о статусе выполнения операции. я делаю так

Функция СделатьДело()

МассивОшибок = Новый Массив;

Попытка

РезультатОперации = Сделать1();

Исключение

РезультатОперации = Неопределено;

МассивОшибок.Добавить("Не шмогла " + КраткоеОписаниеОшибки(ИнформацияОбОшибке()));

КонецПопытки

Результат = Новый Структура;

Результат.Вставить("ОперацияУспешна", МассивОшибок.Количество() = 0);
Результат.Вставить("МассивОшибок", МассивОшибок);
Результат.Вставить("РезультатОперации", РезультатОперации);

Возврат Результат;

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

Самое адекватное объяснение ИМХО в (51)

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

*