Помогите начинающему оптимизировать код

1. uDav 29.11.23 13:31 Сейчас в теме
Создал внешнюю печатную форду для документа "Маршрутный лист" в конфигурации УНФ . Так как только начинаю разбираться в программировании 1С. Сам код работает и печатную форму выводит. Но меня интересует на сколько оптимально и эффективно я написал док. Помогите плиз.
Прикрепленные файлы:
ПечатьОтгрузкиДляТеста.epf
По теме из базы знаний
Ответы
Подписаться на ответы Инфостарт бот Сортировка: Древо развёрнутое
Свернуть все
8. user856012 14 29.11.23 14:23 Сейчас в теме
(1)
Так как только начинаю разбираться в программировании 1С.
Ну да, попытка четыре года назад - не считается: ведь перед ней было 11 лет молчания на форуме!
13. starik-2005 3075 29.11.23 14:56 Сейчас в теме
(1)
Сам код работает и печатную форму выводит.
Малацца! Ну если сам написал. Остальное про оптимальность - это никого не интересует, особенно бизнес. Когда появится над головой карающая рука тимлида, палящего сонар, то все неоптимальности быстро разрюхаются и уйдут. А если не появится - печаль-огорчение.
Хошь кодревью? Сонар поставь и играйся.
2. soft_wind 29.11.23 13:46 Сейчас в теме
вот с ходу
у вас запрос в цикле!!!

и вообще, так себе!
сделайте себе привычку, во ВПФ вставлять форму обработки! это вам поможет в других печатях при отладке.
сможете запускать такие ВПФ под отладчиком)
user1619761; +1 Ответить
3. uDav 29.11.23 13:55 Сейчас в теме
(2) В этот цикл он попадает один раз я так понял и поэтому на него не обратил внимание, что лучше будет обращаться к документу через МассивОбъектов[0] ? Можно дальше расшифровать "и вообще, так себе" . Я понимаю, что не хватает опыта.
5. Sashares 35 29.11.23 14:09 Сейчас в теме
(3)Нет.
Если в форме списка документов будут выделены 2 (или более) документа, в процедуре печати в массиве объектов будут 2 (или более) ссылки.
Процедура печати должна быть адаптирована для печати произвольного количества документов.

До цикла выполнить запрос по массиву объектов.
В запрос добавить итоги по МаршрутныйЛистЗаказы.Ссылка.

Доработать обход выборки с учетом этого.
Перенести вывод шапки в цикл.

Добавить разрыв страницы перед выводом следующего маршрутного листа.

Примерно такие замечания.
4. uDav 29.11.23 13:58 Сейчас в теме
(2) Про отладчик согласен с вами . На данный момент отлаживаю через Сообщить (). ;-) !
6. soft_wind 29.11.23 14:17 Сейчас в теме
(4) уже сколько раз здесь (на форуме) писалось
ВОЗЬМИТЕ ГОТОВЫЙ ХОРОШИЙ ПРИМЕР ВПФ!!!!
и на его примере штампуйте свои,
вам останется только менять: имена ПФ, назначение, макеты и запрос(ы)
остальное все унифицировано
7. uDav 29.11.23 14:22 Сейчас в теме
(6) Можно ссылку где скачать ВФП и еще меня интересует момент объединения колонок. Точнее правильность реализации данной части кода.
9. Sashares 35 29.11.23 14:27 Сейчас в теме
(7)Если объединяются, значит правильно. В чем сомнения?
10. uDav 29.11.23 14:30 Сейчас в теме
(9) в рациональности кода.
11. lefthander 29.11.23 14:34 Сейчас в теме
(10)иногда рациональность может завести в дебри. Это просто ВПФ, главное что бы работало.... Рациональность и оптимизация важны когда вы обрабатываете и считаете объемы за большой период.
12. Sashares 35 29.11.23 14:37 Сейчас в теме
(10)Объединять можно, если это нужно.
Если вам в таком виде нужна печатная форма, других вариантов особо нет.
14. soft_wind 29.11.23 16:29 Сейчас в теме
(7) у нас базы разные, и разные версии БСП,
проверьте в своей базе еще другие ВПФ наверняка найдется правильная, оптимальная, с формой и т.д.
вот ее и возьмите за пример.
15. uDav 29.11.23 17:25 Сейчас в теме
(14) Увы и ах. Других там нет. Так как брал чистую базу.
Оставьте свое сообщение

Для получения уведомлений об ответах подключите телеграм бот:
Инфостарт бот