Да, тоже, читать не приятно. Скорее стоило побить про процедурам, но, чтобы точно ответить, надо погрузиться и понять, какую проблему решал автор, чтобы вот тут все в одно место написать. Но как минимум кучу выносов в локальные переменные или заносов в кэш явно выделяемы в кусок кода (пусть даже не в процедуру отдельную).
И такого кода много. Это разработчики от 1С, мне хочется верить, что уж у кого-кого, а у них с архитектурой всё продумано. А потом в процессе отладки натыкаешься на такое...