Телеграмм чат группы ssl1c страница 4076

Size: a a a

1С, БСП, DevOps и Архитектура

2307 membersпожаловаться на группу

2021 July 07

АЛ

Артем Лазаренко... in 1С, БСП, DevOps и Архитектура

источник

15:39пожаловаться #1

Andrey Ovsiankin in 1С, БСП, DevOps и Архитектура

@vbondarevsky @infactum и другие любители неявных возвратов. Как вам такое?

Функция СуммаСкидки(Знач КорзинаЗаказа)
    Если КорзинаЗаказа.Строки.Количество() > 10 Тогда
        Возврат Скидки.СкидкаНаКрупнуюКорзину(КорзинаЗаказа);
    ИначеЕсли КорзинаЗаказа.ЕстьКартаЛояльности Тогда
        Скидки.СкидкаПоКартеЛояльности(КорзинаЗаказа);
    КонецЕсли;
КонецФункции

источник

15:40пожаловаться #2

Konstantin Kozyrev in 1С, БСП, DevOps и Архитектура

> bsl
Мило)

источник

15:40пожаловаться #3

Alexander Strizhachu... in 1С, БСП, DevOps и Архитектура

а вдруг)

источник

15:41пожаловаться #4

Konstantin Kozyrev in 1С, БСП, DevOps и Архитектура

Так и нету ведь: всё есть fun, fn, void, func

источник

15:41пожаловаться #5

Ramis Bakhtiev in 1С, БСП, DevOps и Архитектура

ну ясно же что в одном случае человек хочет вернуть результат работы, а в другом нужно просто запустить расчет скидки )))))

источник

15:42пожаловаться #6

gosn1ck in 1С, БСП, DevOps и Архитектура

тут хватает возвратов?

источник

15:42пожаловаться #7

КЧ

Кирилл Черненко... in 1С, БСП, DevOps и Архитектура

Скидка по карте лояльности процедура?

источник

15:42пожаловаться #8

Andrey Ovsiankin in 1С, БСП, DevOps и Архитектура

Нет, функция, возвращающая сумму скидки

источник

15:42пожаловаться #9

John Doe in 1С, БСП, DevOps и Архитектура

Смахивает на последствия рефакторинга, когда после блока условия был еще какой-то код, который потом стал нежелательным для выполнения при срабатывании первой ветки блока условия и поэтому в нее влепили возврат, а потом после блока условия код вообще под нож пошел.
Ну это если попытаться как-то оправдать чуть дальше чем простое "забыл".

источник

15:42пожаловаться #10

КЧ

Кирилл Черненко... in 1С, БСП, DevOps и Архитектура

А, а то я уж подумал что тут функция с сайд эффектом

источник

15:43пожаловаться #11

John Doe in 1С, БСП, DevOps и Архитектура

Я просто не в теме за другие языки

источник

15:43пожаловаться #12

АЛ

Артем Лазаренко... in 1С, БСП, DevOps и Архитектура

между прочим по законам шариата за такой код будут судить

источник

15:43пожаловаться #13

Vladimir Bondarevski... in 1С, БСП, DevOps и Архитектура

Функция СуммаСкидки(Знач КорзинаЗаказа)
    Если КорзинаЗаказа.Строки.Количество() > 10 Тогда
        Возврат Скидки.СкидкаНаКрупнуюКорзину(КорзинаЗаказа);
    ИначеЕсли КорзинаЗаказа.ЕстьКартаЛояльности Тогда
        Скидки.СкидкаПоКартеЛояльности(КорзинаЗаказа);
    КонецЕсли;

 Возврат Неопределено;
КонецФункции

Что поменялось?

источник

15:43пожаловаться #14

КЧ

Кирилл Черненко... in 1С, БСП, DevOps и Архитектура

Можно диагностику напилить, типа результат функции отправляется в /dev/null

источник

15:43пожаловаться #15

Andrey Ovsiankin in 1С, БСП, DevOps и Архитектура

Появилась явная декларация намерений автора

источник

15:43пожаловаться #16

Vladimir Bondarevski... in 1С, БСП, DevOps и Архитектура

Автор говнарь. Забыл где надо возврат или забыл поменять возвращаемое значение

источник

15:44пожаловаться #17

John Doe in 1С, БСП, DevOps и Архитектура

Стало нагляднее что обратно может прилетать неопределено

источник

15:44пожаловаться #18

Alexander Strizhachu... in 1С, БСП, DevOps и Архитектура

вместо суммы (число)

источник

15:44пожаловаться #19

Alexander Strizhachu... in 1С, БСП, DevOps и Архитектура

говнокод

источник

15:44пожаловаться #20