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

(0) У "Валюта" тип, поди СправочникСсылка.Валюты?

(1) Очевидно, что да.

(0) 47000 обращений к субд потому что.

(0) Уж сколько раз твердили миру, что получение значения реквизита через точку от ссылки - зло. По сути в момент обращения через точку, из БД получается весь объект.

Так нельзя сравнивать. Надо писать:
Если Валюта = ВалютаРуб Тогда
   Молодец();
КонецЕсли

(0) А что по-твоему делает элементарная операция Если Валюта.Код = "643"?
Понаберут по объявлению

Там еще за 250 строчку по ходу нужно морду набить

(3) А как же внутреннее кеширование от 1с?

(1) И в строке 250 небось, НайтиПокоду() :)

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

(7) +100

Там видимо поиск по коду или по наименованию.. Внутри цикла, 47 тыс раз поиск одного и того же элемента.

(8) Кеширование чего, разных валют во всех местах, которые проверяет ТС?

(8) 1с все равно проверяет, не изменился ли обьект, запрашивая version

(8) Код работает так долго, что кэш устаревает :)

А кто сказал, что выше строчкой нет такого?
Валюта = Выборка.Валюта;

(14) да ладно. Вроде секунд 20 должен жить. По идее чтений десять по идеее должно быть реально у ТСа
(13) Вот это интересно. Спасибо.

(13) Не каждый раз же.

(0) из запроса получай сразу поле "ВалютаЕстьРубль"

а что 1с уже не кэширует получение через точку?

ну и разве быстрее будет
Валюта = Справочники.Валюты.НайтиПоКоду
или
Валюта = Константы.Валюта.Получить()

(19) Очевидно, что значение переменной "Валюта" меняется в цикле.

(19) Еще один
Что она должна кешировать?

(22) всегда кэшировала (раньше по крайней мере)

(20) Ты бредишь
Соберись, тряпка и прочти ветку еще раз

(5) Попробовал изменить сравнение, чтобы сравнивать ссылки:
1) Обе переменные теперь ссылочного типа: https://ibb.co/72QTj9f
2) сделал новый замер, который показывает, что это всё равно самая длительная операция: https://ibb.co/MhkqcYb

(20) (22) Вотс! Я думал, вы один и тот же человек...

(23) У ТС охрениард разных валют в цикле
Что именно 1С должна кешировать?

(22) Объекты ссылочных типов.

(24) а как ты еще валюту руб найдешь?

(29) Ты же сейчас шутишь?
Я ее, блеать, один раз вне цикла нахожу

(28) И? Продолжай

(21) Думаешь там 40тыс разных валют?

(25) Код уже показывай.

(27) иди почитай про кэширование вначале, а потом уже приходи какахами кидаться

(25) Тебе сказали же, сделай поле ЭтоРубли в запросе

(34) Упс. С тобой все понятно. Вопросов больше не имею.

(31) https://its.1c.ru/db/metod8dev/content/2717/hdoc

(37) И че и че?

(38) "Соответственно, при многократном обращении к одним и тем же данным будет выполняться многократное считывание. Этого можно избежать, обращаясь к свойствам ссылки. В этом случае используется кэширование объектов и в определенном интервале времени при повторном обращении к данным объекта не будет выполняться повторное считывание. "

(32) Думаю валют там не 40 тыс, но они могут быть разные. Значение переменной "Валюта" заменяется в цикле новым значением, например, из выборки запроса. А значит кэш очищается.

Впрочем, у ТС теперь в (25) "Обе переменные теперь ссылочного типа, но это всё равно самая длительная операция."
И вот это уже непонятно.

(0) и желательно ВалютуРубли определить до цикла в котором запрос создается

(25) Ты где то ошибся. Или внешнюю обработку не сохранил, или она закешировалась. Ненормален такой результат.

(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с возмет их из кеша а не бд а это уже фейл (