Свой progressBar

[Андре]

Цитата:

Вот так всегда...
На самом интересном месте выясняется, что не Иван Иванович, а Петр Петрович
не в среду, а в понедельник,
и не продал, а купил...

mazzy, а я где-то писал, что это отчет? или это только твои предположения?

Цитата:

Какого черта беспокоится о юзабилити, если "Пользователи вообще ничего не увидят"?

Пользователи видят прогресс операции. Больше ничего. Если бы последовательность действий можно было бы свести к одному прогресс-бару - с удовольствием бы сделал именно так.

[mazzy]

Цитата:

Сообщение от Андре

mazzy, а я где-то писал, что это отчет? или это только твои предположения?

Да, действительно, только предположения...
В коде был использован ReportId.

[Андре]

Цитата:

Угу. И в пакетном режиме тоже не работает

Ты меня почти убедил, повесил на настройку.

Цитата:

А он еще и не работает?
А как же 20 минут?

Работает у меня на ноуте и работает в промышленной эксплуатации - это очень разные вещи

[Андре]

Цитата:

В коде был использован ReportId.

Идентификатор поступившего на вход документа (excel-файла). Сам он был сформирован не в Аксапте. Моя задача его сохранить в системе, распарсить, на основе него породить ряд новых excel-документов.
Короче, один reportId обрабатывается только один раз. Второй раз будет уже другой reportId.

[mazzy]

Цитата:

Сообщение от Андре

Короче, один reportId обрабатывается только один раз. Второй раз будет уже другой reportId.

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

[Андре]

Цитата:

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

Либо я потерял нить рассуждения, либо ты.... Данный код не относится к прогресс-бару. Если заменить его на sleep(1000) ничего не изменится.
Для отображения прогресс бара ничего генерить не надо. Если это требуется для логики операции - то пож-та.

[Recoilme]

Камешек в стандартный пргресс-бар:
Тормозит ли градусник?

Цитата:

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

Можно не устанавливать итог. А в тексте иногда можно написать что-то типа

Удаление записей в любимой таблице. Запись 39284 из 5847847.

Цитата:

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

2. А есть ли updateInterval? или каждый раз при incCount() выполняется перерисовка формы?

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

[glibs]

В общем это... о вкусах не спорят. Главное, чтобы мы не поссорились из-за пустяков.

Цитата:

Сообщение от Андре

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

По теме вот чего я еще скажу напоследок.

Сразу предупреждаю, что мнение у меня нетрадиционное и встречаемое многими с агрессией.

В моем представлении "длительная операция" — это минуты хотябы две-три.

Пользователь, который регулярно или хотябы время от времени сидит тупо перед монитором и смотрит кино под названием "Красивый прогресс-бар"... он либо работает по принципу "работа не волк — в лес не убежит" (т.е. вредитель с т.з. бизнеса), либо пенсионер, который уже все в жизни успел сделать, либо ... (я промолчу).

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

[mazzy]

Цитата:

Сообщение от Андре

Данный код не относится к прогресс-бару.

К стандартному - не относится.
К самодельному - очень даже.

Цитата:

Сообщение от Андре

Для отображения прогресс бара ничего генерить не надо. Если это требуется для логики операции - то пож-та.

Как это не надо, если у тебя внутри запрос по этому идентификатру идет.

Цитата:

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

Камешек в стандартный пргресс-бар:
Тормозит ли градусник?


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

Еще один неразобравшийся, блин... Recoilme, прочитайте ту ветку до конца.
Не хотите использовать - не используйте.

В общем, ясно.
Андре, спасибо за интересное обсуждение.

1. Если делается для бухгалтеров (да и любых других конечных пользователей), то логи и проценты завершения ошибочное завершение исправить не помогут. Все равно программера вызывать. А ему это не надо.
2. Т.к. полезность информационной составляющей сомнительна, остается только "успокаивающая составляющая". А она в стандарте есть.
3. 1% ситуаций, в котором используется новый прогрессбар приведет к постоянному дерганию программиста "сделайте нам красивенькое окошечко сообщений" в остальных 99%... Оно Вам надо?

[Андре]

Цитата:

Как это не надо, если у тебя внутри запрос по этому идентификатру идет.

Код взят из блока "Пример использования". Запрос идет не внутри прогресс бара. Замени на

for (int i=0; i<=1000; i++)
  progress.nextStep();

ничего не изменится.

Цитата:

Андре, спасибо за интересное обсуждение.

Тебе спасибо

[kashperuk]

Вот проект.
Содержит немного доработанные куски кода Андре.
Job для демонстрации возможностей (кстати, для Маззи туда впихнул стандартный прогресс бар еще - именно так и работает все при установке Оракла, или СКЛ сервера)

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

Мои результаты - самописный вариант быстрее.
Правда я использовал неоптимизированные методы стандартного класса. С ними стандартный, думаю, выиграл бы.

И, специально для Сергея - да, класс не работает без GUI, и у него куча недостатков. Но как идея мне нравится. Доразвить его просто нужно
И опять же - это не замена стандартного прогресс бара, а просто его дополнение.
И как видно из демонстрационного примера, они вполне могут сосуществовать.

[mazzy]

Цитата:

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

Мои результаты - самописный вариант быстрее.

Потому что проще.

Код проще в разы.
Эффект на сервере 11.75/13.6 = 86%.
Эффект на клиенте 19.22/26.19 = 73%

Т.е. ускорение на 15%-30%.
За счет неуниверсальности, небезопасности и кучи потраченного времени программиста.

Я чего? Против что-ли?

[mazzy]

Цитата:

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

Мои результаты - самописный вариант быстрее.

Не. Категорически не согласен.
SetText для стандартного делается ВНУТРИ цикла, а для самописного ЗА циклом.
AndyD говорил о тормознутости самого java-кода и работы со строками.
Какие проблемы у SysOperationProgress?
В данном случае начинает работать этот аспект.

[kashperuk]

Цитата:

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

За счет неуниверсальности, небезопасности и кучи потраченного времени программиста.

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

Я сделал интерфейсный класс.
Поэтому использовать даже проще, чем стандартный.
И код метода
doTestStandartProgressbar
на пару строк больше,чем код метода
doTest

так что спорно. Единственное, что требовалось время, чтобы изначально это все написать. Но раз уже написано, почему бы не пользоваться.

Цитата:

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

Я чего? Против что-ли?

Может и нет.
Но выглядит все так, как будто да

[kashperuk]

Вот результаты, если вынести setText за циклы.

Правда, справедливости ради, стоит заметить, что у стандартного прогресс бара интервал обновления больше (то есть реже обновляется)

[mazzy]

Цитата:

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

И код метода
doTestStandartProgressbar
на пару строк больше,чем код метода
doTest

Угу. Внутри цикла, который выполняется 100тыс раз.

Не давай ты своему начальнику втирать про пару строк будешь?
Мне не надо.

[kashperuk]

Цитата:

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

Угу. Внутри цикла, который выполняется 100тыс раз.

Не давай ты своему начальнику втирать про пару строк будешь?
Мне не надо.

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

[mazzy]

Цитата:

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

Вот результаты, если вынести setText за циклы.

Во! Это похоже на правду.

Цитата:

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

Правда, справедливости ради, стоит заметить, что у стандартного прогресс бара интервал обновления больше (то есть реже обновляется)

Ну, дык.
Т.е. программист:
1. затратил время
2. сделал менее универсальный, менее безопасный код
3. сделал менее быстрый код
4. вдобавок не подумал о параметрах по умолчанию.

Опять же ничего не имею против конкретного Андре.
Он действительно сделал интересную штуку. За это ему огромное спасибо.

Я хочу сказать о подходе.
Если есть стандартный функционал, то его НАДО знать, а не выкидывать со словами "ща мы тут за 20 минут на коленке наваяем".

[mazzy]

Цитата:

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

Я хочу сказать о подходе.
Если есть стандартный функционал, то его НАДО знать, а не выкидывать со словами "ща мы тут за 20 минут на коленке наваяем".

Не могу удержаться. Извините.

Но суть проблемы в том, что в данном случае задача повышения юзабилити решается за счет тотального переписывания. Зачем выкидывать ядро стандартного прогресс-бара? Почему не переделать только уровень представления/отображения?

Ведь в стандартном прогресс-баре все представление-отображение СПЕЦИАЛЬНО собрано в одном месте в классе SysOperationProgressForm. СПЕЦИАЛЬНО форма болванка нарисована - SysOperationProgress.

Дык, ведь нет. Выкидывается ВЕСЬ код, чтобы СОЗДАТЬ аналогичный, только хуже.
Грустно