Ускорить вычисление дисплей методов

[IKA]

Есть форма с гридом ,
в гриде 3 контрола(Цена1, Цена2, Цена3), привязанных к датасоуру формы и их значения которых получаются из привязанных к датасоурсу (можно перенести на таблицу) дисплей методов Метод1, Метод2, Метод3

Проблема в том. что все три цены можно получить за одно обращение в методу МетодОбщий, который возвращает контейнер [Цена1, Цена2, Цена3]
МетодОбщий достаточно накладный. Это не оч заметно, когда он 1 раз выполняется, но тк в данной ситуации он невольно вызывается три раза вместо одного, то эффект заметней.
То есть хотелось бы, чтобы для каждой строки грида был вызов
[Цена1, Цена2, Цена3] = МетодОбщий()
а потом уже каждый дисплей метод возращал просто переменную с ценой.

Можно ли так сделать и если да, то где нужно прописать вызов МетодОбщий?

[Pustik]

В форме объявите переменные Price1,Price2 и Price3. Присваивайте им значение в методе Метод1, а в методах Метод2 и Метод3 используйте значения переменных Price2 и Price3

[IKA]

где гарантия, что метод1 вызовется первым?

[Pustik]

Цитата:

Сообщение от IKA

где гарантия, что метод1 вызовется первым?

в гриде контрол Цена1 должен быть первым по отношению к Цена2 и Цена3, в противном случае будет "билибирда"

[IKA]

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

[sukhanchik]

Для примера см классы InventOnHand, PriceDisc и т.д.
Суть: Инициализируется класс, у которого имеется несколько (в Вашем случае 3) методов, возвращающих переменные Цена1, Цена2 и Цена3. Дополнительно в классе объявляется переменная-флажок isCalculated. При расчете, который выполняется в методе МетодОбщий рассчитываются переменные Цена1, Цена2, Цена3, а также взводится флажок isCalculated. В методах, которые возвращают соответствующие переменные производится проверка - а не взведен ли флажок? Если флажок не взведен - то вызывается МетодОбщий. Если флажок взведен - то ничего не вызывается - просто возвращается соответствующая переменная.

Т.о. без разницы - какой метод будет вызван раньше - Цена1 или Цена3 - первый же вызов вызовет МетодОбщий и рассчитает все переменные. При этом повторного вызова МетодОбщий не произойдет.

Само собой, при использовании такого подхода в гриде нужно будет позаботиться, чтобы флажок isCalculated сбрасывался в методе active(), чтобы расчет делался для каждой записи.

Ну и конечно дисплей-методы должны ссылаться на методы этого класса (Цена1, Цена2, Цена3).

[IKA]

метод Active срабатывает только при переходе на новую запись. В моем случае эти три поля на закладке Overview То есть, флажок не будет "сбрасываться", если я правильно понимаю...

Спасибо, сейчас классы гляну, может, там кая-то еще идея

[SRF]

А кеширование дисплей методов не рассматривается ?

[raz]

Используйте Map и будет вам счастье.

[IKA]

Цитата:

Сообщение от SRF

А кеширование дисплей методов не рассматривается ?

рассматривается,конечно, если поясните, как оно тут поможет. то есть, просто опишите что кэшируем и где
Я поясню: если кэшировать все три метода. то все рассчеты все равно 3 раза для каждой записи будут произведены, а не один, как описано в постановке задачи. Но, может быть, я не поняла вашу мысль ..

[SRF]

Ну как правило кеширование методов на форме происходит как уже сказал raz при помощи Map ну или как них нибудь других объектов.

Суть что делаем : делаем общий метод на форме например, в качестве параметра курсор, мап - скажем ключ recid, ну а значение в вашем случае контейнер из трех полей. Метод работает по принципу - есть в мапе данные берем из него, нет вызываем методОбщий, записываем в мап и эти же данные берем в качестве расчета.

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

[IKA]

MAP- точно, отличная мысль! Спасибо!

[mazzy]

Цитата:

Сообщение от IKA

MAP- точно, отличная мысль! Спасибо!

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

вычислять один раз вместо трех на каждый показ записи - вполне разумно.
вычислять один раз вместо всех показов записи - уже как-то безрассудно.

[IKA]

2 Mazzy, а вы что предлагаете-то?

[IKA]

2SRF Да, извините, если вы не про cacheAddMethod, а про Map, то, конечно.
Я по умолчанию под кэшированием, встроенные механизмы подразумеваю.
Спасибо

[mazzy]

Цитата:

Сообщение от IKA

2 Mazzy, а вы что предлагаете-то?

а уже предложено sukhanchik'ом:

Цитата:

Сообщение от sukhanchik

Для примера см классы InventOnHand, PriceDisc и т.д.
Суть: Инициализируется класс, у которого имеется несколько (в Вашем случае 3) методов, возвращающих переменные Цена1, Цена2 и Цена3. Дополнительно в классе объявляется переменная-флажок isCalculated. При расчете, который выполняется в методе МетодОбщий рассчитываются переменные Цена1, Цена2, Цена3, а также взводится флажок isCalculated. В методах, которые возвращают соответствующие переменные производится проверка - а не взведен ли флажок? Если флажок не взведен - то вызывается МетодОбщий. Если флажок взведен - то ничего не вызывается - просто возвращается соответствующая переменная.

[dech]

Вы можете общий метод запихнуть в active() вашего ds, и использовать его результаты (которые будут находиться в classDeclaration) вашими тремя другими методами. Единственное, что эти 3 дисплейных метода будут находиться на форме и их нельзя закэшировать. И не надо никаких флажков.

[Pustik]

Цитата:

Сообщение от dech

Вы можете общий метод запихнуть в active() вашего ds, и использовать его результаты (которые будут находиться в classDeclaration) вашими тремя другими методами. Единственное, что эти 3 дисплейных метода будут находиться на форме и их нельзя закэшировать. И не надо никаких флажков.

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

[Pustik]

Цитата:

Сообщение от IKA

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

Используйте метод displayOption датасорса

public void displayOption(Common _record, FormRowDisplayOption _options)
{
     ;
    conPrice = ['Цена1','Цена2','Цена3'];
    Price1 = conpeek(conPrice,1);
    Price2 = conpeek(conPrice,2);
    Price3 = conpeek(conPrice,3);
    
    Map1.insert(_record.RecId,Price1);
    Map2.insert(_record.RecId,Price2);
    Map3.insert(_record.RecId,Price3);
    
    super(_record, _options);
}

а в дисплей методах тогда :

return (Map1.exists(_Table.RecId) ? Map1.lookup(_Table.RecId) : 0);
...........

[IKA]

Ох, товарищи!

2dech & 2Mazzy.
Там же буквально строкой выше же написано, почему active не подходит для решения задачи
Если вы не согласны. то поясните, как вы считаете можно обойти это ограничение.

2pustik:
Не мое конечно дело, но немного удивительно, что вы "поблагодарили" dech и строкой ниже раскритиковали его подход. За что благодарность-то тогда оставили?

Вопрос закрыт, спасибо raz & srf !