Мертвые ветки логики в логике 1С

Хочу показать Вам на одном примере, как 1С пишет логику своих решений. Встречаюсь с таким не впервые. Лично я считаю это позором для компании, открытый код которой видит вся Россия, но 1С, видимо думает, что никто не заметит. А может думает, что и так сойдет.

Речь пойдет о формировании отчета о розничных продажах (ОРП) из чеков.

Вообще, 1С может сформировать не один, а несколько ОРП, если в чеках за смену были чеки с разной системой налогообложения (в процедуре РозничныеПродажи.ЗаполнитьОтчетыОРозничныхПродажахИВозвратах):

Но если мы спустимся на уровень чуть ниже и посмотрим функцию ЗаполнитьОтчетОРозничныхПродажах, которая заполняет ОРП, увидим странное.

Формируется выборка по аналитике, но используется только первое значение аналитики, то есть если в выборке несколько различных значений, то будет использовано только первое (используется Если, а не Пока):

Выборка по аналитике строится по таблице ТаблицаИндексовПоАналитике. Так вот, если туда передать разные склады, то 1С не оставит один склад, она просто возьмет товары только одного склада.

Понятно, что внутри функции ЗаполнитьОтчетОРозничныхПродажах 1С может заполнить только один ОРП, но из-за странной логики она теряет товары.

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

Хотя в таблице ТаблицаИндексовПоАналитике есть колонка склад, но использовать ее нельзя в произвольном порядке, там может быть только один склад. Но зачем тогда она нужна? Так было удобнее разработчикам, чтобы не передавать склад отдельной таблицей.

Любой стороний разработчик видит колонку и подозревает, что если туда занести разные значения, будут созданы ОРП по разным складам. Но не тут-то было. 1С не для того эту колонку заводило!

К сожалению, 1с мне не заплатит за этот бесплатный code-review и, что важнее — не сделает из него выводов.

Среда: УТ 11.5.24.68