Оптимизирую самописную внешнюю обработку. Она на основании огромного количества документов за определенный период создаёт новые документы.
В примере на скрине мы создаём новые документы на основании 21000 поступлений и 26000 реализаций, поэтому в некоторые строки мы попадаем по 47000 раз.
Скрин: https://ibb.co/8MY5DN3
Почему замер производительности показывает, что элементарная операция сравнения "Если Валюта.Код = "643" Тогда" выполняется в разы дольше, чем например какие-нибудь сложные операции типа запросов, которые вызываются примерно такое же количество раз?
Запускал замер производительности уже не один раз и всегда сравнение со строкой самое длительное.

(38) Первый раз слышишь про объектный кэш?

(40) >> А значит кэш очищается.
Вот это я так понимаю теперь очень спорный вопрос. Часть людей считают (я в том числе), что данные кэшируются по ссылке и не умирают в кэше при обновлении переменной. Вторая часть (минимум ты и PR) считают что кэш живёт и умирает вместе с переменной.

(40) Значение переменной замещается, а кэш остается в памяти. Если объект не менялся.

Вот весь код:

Функция ПересчитатьВалютнуюСумму(ВалютнаяСумма, Валюта, Дата)
    
    Если Валюта = ВалютаРуб Тогда
        
        Возврат ВалютнаяСумма;
        
    Иначе
        
        
        Возврат ПересчетСуммыПоКурсу(ВалютнаяСумма, Валюта, Дата);
        
    КонецЕсли;
    
КонецФункции

Этот кэш нужен именно для таких случаев. Обращение к часто используемым объектам из разных мест кода, в разных переменных.

(44) Из (37)
"Разумеется, если необходимо многократно обращаться к одному и тому же полю в пределах одного и того же алгоритма, то эффективнее считать данные, запомнить их в переменную и обращаться к уже считанным данным. Но получение данных через ссылку может выдавать ранее считанные данные, даже если они были считаны в другом вызове встроенного языка, так как кэширование выполняется в рамках всей клиентской сессии."

(39) А где ты увидел одни и те же данные? Валюта же меняется в цикле

На самом деле, PR тут поднял интересную тему. Которую Печкин не может вкурить.

Запрос.Текст=
"Выбрать
Номенклатура.Ссылка как Ссылка1,
Номенклатура.Ссылка как Ссылка2
Из...."

Выборка=Запрос.Выполнить().Выбрать();
Выборка.Следующий();

Ссылка1=Выборка.Ссылка1;
Ссылка2=Выборка.Ссылка2;
Наименование1=Ссылка1.Наименование;
// Будет ли тут использоваться кеш?
Наименование2=Ссылка2.Наименование;

(50) я от тебя ответа на (44) ожидал, а для тебя это тоже интересная тема))

(50) Везде будет.

(47) "в разных переменных."

Инфа - 100% ?

(40) Это непонятно, да

(49) Через N итераций валюта повторяется.

Ща проверим

(45) (44) Проведите простой эксперимент.
Сформируйте запрос 40 тыс раз получающий пару валют из справочника, упорядочив по ссылке.

И далее, замерьте 2 варианта кода

Пока Выборка.Следующий() Цикл
Валюта = Выборка.Валюта;
Сообщить(Валюта.Код);
КонецЦикла;


Пока Выборка.Следующий() Цикл
Если НЕ Валюта = Выборка.Валюта ТОгда
   Валюта = Выборка.Валюта;
КонецЕсли;
Сообщить(Валюта.Код);
КонецЦикла;

(50) Да, будет.
В https://its.1c.ru/db/metod8dev/content/2717/hdoc есть про правильное получение представления.

(56) и про version дай пруф, пожалуйста, если есть под рукой

(57) Сам не хочешь? :)

(46) Это не может быть всем кодом, потому что ты обращаешься к Валюта, которой нет ни в параметрах ни присвоения раньше
Это же, надеюсь, не реквизит формы?

(59) https://its.1c.ru/db/metod8dev/content/2700/hdoc/

(50) Хороший вопрос
И проверяется легко

(62) что-то не то.

(55) Не уверен, что если ты в переменную записал валюту, которая уже раньше где-то как-то закешировалась, то реквизиты теперь возьмутся из кеша
Опять же, это легко проверяется

(64) То самое.

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

Да, кешируется, несмотря на разные переменные.

(67) Ну хоть тут интуиция (и не 1С) не подвела))

Что-то ТС темнит

(67) А если валюты одинаковые, но взяты из разных мест, например из разных документов, то есть путь разный?

Про повторное считывание - оно считывает Версию раз в 20 секунд. И получает весь пул реквизитов, по условию, отличному от версии:

Запрос=Новый Запрос;
    Запрос.Текст=
    "ВЫБРАТЬ
    |    Справочник1.Ссылка КАК Ссылка1,
    |    Справочник1.Ссылка КАК Ссылка2
    |ИЗ
    |    Справочник.Справочник1 КАК Справочник1";
    Выборка=Запрос.Выполнить().Выбрать();
    Выборка.Следующий();
    
    Ссылка1=Выборка.Ссылка1;
    Ссылка2=Выборка.Ссылка2;
    Наименование1=Ссылка1.Наименование;
    Пока Истина Цикл
        Наименование2=Ссылка2.Наименование;
    КонецЦикла;

(66) Спасибо. Я "верс", "vers" искал

(61) через ПЕРЕМ объявил переменную ВалютаРуб, записал в неё валюту один раз и потом в цикле к ней обращаюсь

(70) кэш по ссылке, по фигу откуда тыее берешь

(73) https://prnt.sc/12vjmd7

Подозреваю, что когда у сервера 1С занятая память подходит к 85% физической, картинка меняется.

Сеансовые данные в такой период активно сбрасываются на диск, например.

(76) Спасибо

Кстати, а транзакций в секретном коде ТС нет ли?

(80) нет, транзакций нет

(81) Покажите код, где вызывается ваша функция ПересчитатьВалютнуюСумму()

(82) Ага. прикольно будет, если она с клиента контекстным серверным вызовом происходит) За то сразу всё станет на место, даже замер в (25).

Выше уже подтвердили, но и я отпишусь.
(60) Я проверил. Вы правы. Данные объекта кэшируются.

(74) Толстый клиент? Формы не управляемые?

(0) Потому что потому (с) Сами-знаете-кто.

(83) В замере (0) видно, что все на сервере.

(81) всё на сервере вызывается. Сначала запросом формирую таблицу документов, а потом итерируюсь по этой таблице и выполняю различные действия, в том числе пересчитываю сумму в рубли согласно курсу
(85) управляемые формы

Откуда в вашей функции берется значение переменной ВалютаРуб?
ВалютнаяСумма, Валюта, Дата - вы передали в качестве параметров функции. А ВалютаРуб - это что?


Функция ПересчитатьВалютнуюСумму(ВалютнаяСумма, Валюта, Дата)
    
    Если Валюта = ВалютаРуб Тогда
        
        Возврат ВалютнаяСумма;
        
    Иначе
        
        
        Возврат ПересчетСуммыПоКурсу(ВалютнаяСумма, Валюта, Дата);
        
    КонецЕсли;
    
КонецФункции

(89) Меня вот тоже смущает этот момент. Но в (74) сказано что объявил через Перем. Вопрос только, где это объявление и присвоение значения произведено?

(74) Тип переменной какой?

(91) Судя по (25) оба справочник ссылка. Но где в коде эти значения получены, неясно.

(25) у тебя это сравнение 23тыщи раз выполняется - раздели время на количество и будет < 0,0022 на одну операцию

Перем ВалютаРуб;
(92)


Процедура ЗаполнитьТаблицуУслуг(ТаблицаДок)

    ВалютаРуб = Справочники.Валюты.НайтиПоКоду("643");

    Для каждого строка из ТаблицаДок Цикл
                
        НоваяСтрокаУслуги = ТаблицаУслуг.Добавить();

        НоваяСтрокаДокументы.СуммаПоУслуге = (-1)*ПересчитатьВалютнуюСумму(строка.СуммаУслуги,
                                                строка.Документ.ВалютаДокумента,
                                                строка.Документ.Дата);
    КонецЦикла;
КонецПроцедуры;

(94) строка.Документ.ВалютаДокумента - это ппц

(87) строго говоря, видно что эта строка выполняется на сервере. Откуда вызвана функция мы не видим.
Но проверил, расходы на передачу контекста в отладке показываются в строке вызова функции (а не на её первую строку) - моё предположение в (83) не могло быть верным при таком замере

(93) но в любом случае сравнение двух переменных должно выполняться в разы быстрее, чем например запрос.
В строке 521 запрос вызывается 11826 раз (в 2 раза реже. чем сравнение двух переменных). А выполняется он в 7 раз быстрее.
В строке 267 я 23 тыс. раз ищу элементы справочника номенклатурные группы по наименованию. Очевидно, что процедура поиска по значению в разы медленнее, чем процедура сравнения двух ссылок. Но однако она тоже выполняется примерно в 7 раз быстрее.

(97) Ты где то ошибся в замере.

(97) а теперь замените строку "строка.Документ.ВалютаДокумента" на "ВалютаДокумента", а переменной ВалютаДокумента присвойте значение ВалютаДокумента = строка.Документ.ВалютаДокумента чуть выше.
И запустить замер снова.

(94) всё равно говнокод)

(0) Либо ошибка в замере, либо вы в справочник Валюты запихнули реквзит типа ХранилищеЗначений для картинок.

(97) а что там за запрос?

(99) про качество кода, вам уже написали.
Получайте всё, что можно В ЗАПРОСЕ, а не через извраты вида Строка.Документ.ИмяРеквизита.

Явно проблема в "строка.Документ.ВалютаДокумента"
Замер просто не правильно отображает похоже.

(97) Теоретически, если ты в ПересчитатьВалютнуюСумму()
Поставишь в начале
А = 1;
То замер укажет на эту строку, как на наиболее жручую.

(99) и правда причина была в строка.Документ.ВалютаДокумента
Я думал, что если на вход в функцию ПересчитатьВалютнуюСумму передать две ссылки и их сравнить, то нет разницы каким образом эти ссылки были получены. А оказывается разница есть

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

(106) еще и дата получается

(108) получается да. Она не попадала в замер производительности, так как валютных операций не много

(107) Он не на неё грешил, а на Строка.Документ.ВалютаДокумента. Нам же значение параметра ВалютаДокумента требуется только в момент проверки условия, вот он и получает это значение в момент проверки условия, а не при вызове функции. Вдруг, например, нам это значение в функции нигде и не пригодится. Зачем лишний раз что-то считать? Ленивые вычисления

Сделайте замер цикла от 1 по 1000000 с 1 условием:


Если ИСТИНА ИЛИ Строка.Документ.ВалютаДокумента = Рубли Тогда

КонецЕсли;

А потом тоже самое с таким кодом:

Если ЛОЖЬ ИЛИ Строка.Документ.ВалютаДокумента = Рубли Тогда
КонецЕсли;

Результаты замеров будут разные

(111) :-D ну да, ИСТИНА выполняется быстрее чем ЛОЖЬ.

(107) Если действительно хочешь понять, показывай полный код.

(112) потому что зачем выполнять вторую половину условия, если результат будет и так и так ИСТИНА.
При оптимизации кода можно всякие "тяжелые" вычисления в "длинных" условиях стараться отодвигать как можно дальше, чтобы системе их считать приходилось как можно реже.
Но кто будет с таким заморачиваться🤷‍♂️

(114) это же вроде в одних из первых занятиях по 1с рассказывают...

Во всех конфах на БСП есть функция типа ЗначенияРеквизитовОбъекта.
Каждый раз, когда возникает желание получать реквизит ссылки, нужно вспоминать об этой процедуре.

Т.е. к примеру делать так:

      СтруктураРеквизитовДокумента = ОбщегоНазначения.ЗначенияРеквизитовОбъекта(строка.Документ, "ВалютаДокумента, Дата")


      НоваяСтрокаДокументы.СуммаПоУслуге = (-1)*ПересчитатьВалютнуюСумму(строка.СуммаУслуги,
                                                СтруктураРеквизитовДокумента.ВалютаДокумента,
                                                СтруктураРеквизитовДокумента.Дата);

(110) Похоже на правду

(115) Это даже в справке, встроенной написано.

(116) Этот позор архитектуры давно пора спрятать в платформу.

(119) Кто ж спорит, но пока есть, что есть.

(116) в данном случае глобально это картину не сильно изменит - всё равно 40к запросов к базе (только они чуть полегче). Эти данные безусловно должны быть в самой таблице

(119) Тогда неучи его будут использовать как в (116)
По всем нужным документам одним запросом надо получить данные и потом использовать в своем цикле.

27 тысяч запросов в цикле.. еще бы было быстро

ну если нужно 27 тыщ документов записать, то и поболее запросов будет

(122) Да, но платформа вполне способна проанализировать их цикл, собрать все используемые реквизиты и получить их неявно.

(125) платформа с телепатией это круто, но пока только боты.

(126) https://ibb.co/3vfvRRx

(119), (120), вы такие милые со своими авторитетными доморощенными мнениями, прямо ми-ми-ми
Прямо фузиной немножечко пахнуло

(125) Это слишком сложно и не нужно.

(111) >> Результаты замеров будут разные.

Это очевидно. Для этого даже сам замер делать не нужно.
Только каким боком данный пример относится к обсуждению?

(97) Кстати, а почему столько запросов в цикле. Судя по всему, по каждому документу запрос. Может сделать выборку одним запросом?

(111) Прошу прощения, но подозреваю что в условии :

Если ИСТИНА ИЛИ Строка.Документ.ВалютаДокумента = Рубли Тогда

КонецЕсли;

Выражение "Строка.Документ.ВалютаДокумента = Рубли" вообще вычисляться не будет.

Самое простое - пусть второе выражение после "ИЛИ" содержит ошибку (типа, ЗначениеЗаполнено на мутабельное).
Если впереди ИСТИНА - пройдёт со свистом. ЛОЖЬ - вылетит.

Кстати, если хочется получить значение через ссылку но лень писать запрос то есть же:

Код_ = ОбщегоНазначения.ЗначениеРеквизитаОбъекта(МояСсыль, "Код");
Структура_ = ОбщегоНазначения.ЗначенияРеквизитовОбъекта(МояСсыль, "Код, Наименование");

(133) А там типа не запросы...

(134) Но их не надо писать... :)

Не, если можно получить заранее один раз - то надо сохранить в переменную, без базара.
"ЗначениеРеквизитаОбъекта" спасает от загрузки лишней информации при получении данных через ссылку.

(132) Конечно. Просто этот пример в рамках обсуждаемого контекста, так сказать "в моменте", не более того. Если абстрактно "в вакууме" такие примеры показывать, то можно было и так: "ИСТИНА ИЛИ 1/0 = 1" vs "ЛОЖЬ ИЛИ 1/0 = 1"

(37) вот никогда не понимал а почему в теч 20сек 1с не проверяет версию объекта а выдает из кеша ?
https://its.1c.ru/db/pubdevguide83#content:303:hdoc

(5) На больших данных это долго работает в циклах. Оптимизировал условие через булево, условно:
"Если ЭтоВалютаРуб Тогда"
с помощью запроса, скорость проведения стала 7 сек вместо 11.

(137) Чтобы кэширование имело хоть какой-то смысл.

(137) так версию нужно же в бд получать

(137)(139) <Если окажется, что версии данных не совпадают (т. е. произошло изменение данных в базе данных), данные, находящиеся в кеше, будут удалены из него, и будет выполнено повторное считывание данных из базы данных. Начиная с этого момента, идет отсчет следующего 20-секундного интервала валидности этих данных.>

вы точно читаете всё что цитируете?

(141) ага, если данные изменились в 19ую секунду то 1с возмет их из кеша а не бд а это уже фейл (