Какие проблемы у SysOperationProgress?

[mazzy]

Здесь aEremenko: Update_RecordSet
я советовал пользоваться классом SysOperationProgress для отображения текущего статуса выполнения.

Было очень много намеков, что этот класс работает плохо.

Цитата:

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

Особенно чудные результаты дает этот класс при работе с несколькими компаниями.

Для обсуждения создал новую ветку.

[Recoilme]

Смотри ветку:
Тормозит ли градусник
30 минут из 40 тратить на перерисовку текста - это ненормально, притом что кода там - 2 строчки поправить

[mazzy]

Цитата:

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

Смотри ветку:
Тормозит ли градусник
30 минут из 40 тратить на перерисовку текста - это ненормально, притом что кода там - 2 строчки поправить

Вам несколько людей пытались объяснить, что не тратит прогресс-бар столько времени на прорисовку. Но вам "надоело ковыряться".

Я могу только повторить - разберитесь со стандартным кодом.
Разберитесь с tutorial_progress как работает прогресс-бар в двухуровневой и трехуровневой системе.

Ок. Recoilme, большое спасибо за ваш ценный вклад.

Еще проблемы есть?
Я, например, очень хотел бы понять что за проблемы с переключением компании. Прогрусс-бар вообще не должен зависеть от компаний.
А еще?

[ivas]

код:

PHP код:

#Define.LoopCount(100000)
static void Job1(Args _args)
{
    SysOperationProgress pBar = new SysOperationProgress();
    int time = timeNow();
    int i, j;
    ;
    pBar.setCaption("Обработка");
    for (i = 0; i < #LoopCount; i++)
    {
        pBar.setText(strfmt("Операция № %1", i));
        pBar.incCount();
        j++;
    }
    info(strfmt("Прошло: %1 сек.", timeNow() - time));
} 


c прогресс баром работает минуту
без < 1сек
а если у нас 10 000 000 циклов (например бежим по InventTrans)
получается прогресс бар отожрет почти 2 часа

[mazzy]

Цитата:

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

код:

PHP код:

#Define.LoopCount(100000)
static void Job1(Args _args)
{
    SysOperationProgress pBar = new SysOperationProgress();
    int time = timeNow();
    int i, j;
    ;
    pBar.setCaption("Обработка");
    pBar.setCount(#LoopCount);
    for (i = 0; i < #LoopCount; i++)
    {
        pBar.setText(strfmt("Операция № %1", i));
        pBar.incCount();
        j++;
    }
    info(strfmt("Прошло: %1 сек.", timeNow() - time));
} 


c прогресс баром работает минуту
без < 1сек
а если у нас 10 000 000 циклов (например бежим по InventTrans)
получается прогресс бар отожрет почти 2 часа

Еще бы. Этот цикл ничего кроме показа прогресс бара и не делает.

Представьте нормальную работу... Представили?

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

Как вы думаете, сколько часов займет в Аксапте выполнение 10млн циклов при НОРМАЛЬНОЙ работе? Вы предлагаете пользователю ждать все это время, не имея никакой информации о ходе работ? Я правильно понимаю, что вы предлагаете пользователю сидеть несколько часов-суток перед пустым экраном? Я не ошибся?

Да, прогресс-бар сам занимает некоторое количество ресурсов.
Но его польза при больших нормальных обработках гораздо больше, нежели какие-то жалкие несколько процентов дополнительного времени...

Возвращаясь к вашему коду.
Вы просто не умеете его готовить.
Во-первых, вы не установили setTotal.
Во-вторых, если отображение прогресс-бара выполняет дольше чем сама работа, то надо пользоваться методом UpdateInterval(), чтобы установить большее время между обновлениями.

Посмотрите же в метод update(), люди!

[ivas]

согласен, setCount по запарке вставил
вот по вашему совету

#Define.LoopCount(100000)
static void Job80(Args _args)
{
    SysOperationProgress pBar = new SysOperationProgress();
    int time = timeNow();
    int i, j;
    ;
    pBar.setCaption("Обработка");
    pBar.updateInterval(10);  // типа ускорение?
    for (i = 0; i < #LoopCount; i++)
    {
        pBar.setText(strfmt("Операция № %1", i));
        pBar.incCount();
        j++;
    }
    info(strfmt("Прошло: %1 сек.", timeNow() - time));
}

по времени +10 сек
дело в том что действительно сталкивался с этим в реальной обработке:
предварительная выгрузка inventTrans (15000000 строк) в мапу
с прогресс баром 4часа без 2часа
решилось добавлением пары строк

PHP код:

 for (i = 0; i < #LoopCount; i++)
    {
        if (i mod 1000 == 0)
        {
              pBarar.setText(strfmt("Операция № %1", i));
              pBar.incCount(1000);
        }
        j++;
    } 


[Recoilme]

Хоть кто то вменяемый тут есть..Спасибо ivas)
Я решил проблему точно так же, только код вставил в sysoperationprogress, чтоб не возвращаться к этому вопросу в дальнейшем

            //VKUL -- Quick Progress --
            countIncrement[_idx] +=_progressCountIncrement;
            if (countIncrement[_idx] == updateCount[_idx])
            {
                this.setCount(progressCount[_idx] + countIncrement[_idx],_idx);
                countIncrement[_idx]  =   0;
            }
            //VKUL -- Quick Progress --
            //this.setCount(progressCount[_idx] + _progressCountIncrement,_idx);

[AndyD]

Сергей, при таком большом количестве вызовов уже вступает в действие общая тормознутость пользовательского кода. На фоне сотни тысяч вызовов методов setText() и incCount() несколько десятков раз обновление формы просто теряется.
updateInterval() тут уже не поможет.

Можно написать дополнительный метод, который объединяет в себе изменение текста и кол-ва шагов - получим выигрышь около 50%.
А можно перенести функциональность метода update() в свой код - т.е. вызывать обновление только по истечение интервала времени - в этом случае торможения практически не будет

[mazzy]

Цитата:

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

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

Понял. Спасибо.

Так каков вывод? этим классом нужно/можно/нельзя пользоваться?

[mazzy]

Запустил у себя тест:

#Define.LoopCount(100000)
static void Job6(Args _args)
{
    SysOperationProgress pBar = new SysOperationProgress();
    int time = timeNow();
    int i, j;
    ;
    pBar.setCaption("Обработка");
    pBar.setTotal(#LoopCount);
    //pBar.updateInterval(100);
    for (i = 0; i < #LoopCount; i++)
    {
        pBar.setText(strfmt("Операция № %1", i));
        //pBar.incCount();
        j++;
    }
    info(strfmt("Прошло: %1 сек.", timeNow() - time));
}

Со всеми методами: 14 сек
если выключить один из методов: 6 сек.
если выключить все методы: 0 сек.

В данном случае длительность updateInterval(100) не влияет.
Просто если его устанавливать, то окошко не показывается.

[ivas]

Цитата:

Так каков вывод? этим классом нужно/можно/нельзя пользоваться?

конечно нужно
а есть варианты?

[ivas]

Цитата:

В данном случае длительность updateInterval(100) не влияет

имхо она вообще не влияет на скорость работы ну может +-1%

[mazzy]

Цитата:

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

имхо она вообще не влияет на скорость работы ну может +-1%

Блин, сделайте нормальную обработку: Изменяйте что нибудь в базе и сравните.
Или сделайте вместо куцего ++j паузу хотя бы на 200 милисекунд sleep(200);
Сравните.

[mazzy]

Цитата:

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

конечно нужно
а есть варианты?

Варианты конечно есть.
Хорошо. Нужно. Идем дальше.

Какие у этого класса проблемы (помимо "общей тормознутости пользовательского кода")?

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

Какие еще проблемы есть?

[slava]

Цитата:

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

здесь прозвучало очень интересное сообщение о проблемах при переключении между компаниями?

Мое предположение, у меня и было. Ситуация: создание нескольких сотен заказов, создание закупки по ним (это все в компании 1), и создание продажи по ним (в компании 2). Обработка по продажам и закупкам счетов на оплату. Для перехода между компаниями - changecompany. Дык вот через раз Аксапта "путалась" в какой компании она разносит. Причем путалась на ровном месте (то есть в различных частях кода). Пока не "отключил" прогресс в классах обработки именно для этой операции. После этого - ни одной ошибки. Так что претензии не к самому прогрессу, а к связке его работы в рамках нескольких компаний.

[AndyD]

Конечно, можно
Простейшее

    int tm = timeNow();
    int interval = 1;
...

        if (timeNow()-tm >= interval)
        {
            pBar.incCountText(strfmt("Операция ? %1", i), i-pBar.getCount_RU());
            tm = timeNow();
        }

практически полность уберет негативное влияние вызова методов прогрессбара
incCountText() - добавил метод для одновременного измения текста и прогресса
PS миднайт не учитываю для простоты

[mazzy]

Цитата:

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

Конечно, можно
Простейшее

Да, но заманаешься такое в код вставлять. Но еще раз спасибо.

[ivas]

Цитата:

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

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

Цитата:

Или сделайте вместо куцего ++j паузу хотя бы на 200 милисекунд sleep(200);
Сравните.

j++
это не задержка просто проверял не выкинул ли оптимизатор мой цикл
кстати он вообще есть в аксапте? всмысле оптимизатор

[mazzy]

Цитата:

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

всмысле оптимизатор

оптимизатора кода в аксапте нет

[kashperuk]

Да-да. В тему вот эта тема: (простите тавтологию )
Глюки при обработке отборочной накладной