Что можно оптимизировать в ниже приведенном коде и почему?
Процедура ОбработкаПроведения(Отказ, РежимПроведения)

лкЗапрос = Новый Запрос;
лкЗапрос.Текст = "ВЫБРАТЬ
                 |    РеализацияАвтомобилей.Автомобиль,
                 |    РеализацияАвтомобилей.Стоимость,
                 |    РеализацияАвтомобилей.СтоимостьУЕ,
                 |    РеализацияАвтомобилей.Ссылка
                 |ИЗ
                 |    Документ.РеализацияАвтомобилей.Автомобили КАК РеализацияАвтомобилей";

лкВыборка = лкЗапрос.Выполнить().Выбрать();

Пока лкВыборка.Следующий() Цикл
    
    лкЗапись = Движения.БУ.Добавить();
    
    лкЗапись.СчетДт         = лкВыборка.Ссылка.ДоговорВзаиморасчетов.СчетРасчетов;
    лкЗапись.СчетКт         = ПланыСчетов.БУ.Счет_90_01_1;
    
    лкЗапись.Период         = лкВыборка.Ссылка.Дата;
    лкЗапись.Регистратор         = лкВыборка.Ссылка;
    лкЗапись.Сумма         = лкВыборка.Стоимость;
    
    Если лкЗапись.СчетДт.Валютный Тогда
        лкЗапись.ВалютаДт = лкВыборка.Ссылка.ДоговорВзаиморасчетов.ВалютаУчета;
        лкЗапись.ВалютнаяСуммаДт = лкВыборка.СтоимостьУЕ;
    КонецЕсли;
    
    лкЗапись.Содержание = "Отражена вырузка по автомобилю " + лкВыборка.Автомобиль.Наименование;
    
    лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Контрагенты] = лкВыборка.Ссылка.Контрагент;
лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ДоговорыВзаиморасчетов] = лкВыборка.Ссылка.ДоговорВзаиморасчетов;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ВидыДеятельности] = лкВыборка.Ссылка.ВидДеятельности;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Брэнды] = лкВыборка.Ссылка.Брэнд;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.СтавкиНДС] = лкВыборка.СтавкаНДС;
        
КонецЦикла;    

КонецПроцедуры

В запросе специально нет отбора по ссылке? Он же все документы из базы вытянет

Запрос к документам конечно так себе.
Постоянное обращение к ссылке через точку - эти поля можно запросом выбрать.

Откуда такая задача?

(1) и если делать отбор по ссылке - то зачем тогда запрос, не проще ли работать сразу с таб частью?

(3) Нет не проще.

(1) Согласен, принимается.
(2) Тоже сразу на это обратил внимание, принимается.
(4) Согласен.
Что-то ещё заметили?

(0) лк в контексте явно излишне, потому как нет большого перемешивания кода оазной принадлежности/видимости.

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

Движения.БУ.Загрузить(Запрос.Выполнить().Выгрузить());

+(7) выбрать все поля и дать им соответствующие алиасы

(7)Абсолютно верно, тоже рассматривал этот вариант!

Дичь какаято.
Это даже на задачи с собеседования не тянет.

+(7) Хотя с субконто вроде не получится

(11) получится. С субконто как раз Загрузить набор с субконто из ТЗ самое простое, не надо заморачиваться. Просто пишешь Загрузить()

(12) Можно подробнее?

(0)
Получить в запросе все необходимые поля через ВЫРАЗИТЬ, особенно вот это:

Если лкЗапись.СчетДт.Валютный Тогда
        лкЗапись.ВалютаДт = лкВыборка.Ссылка.ДоговорВзаиморасчетов.ВалютаУчета;
        лкЗапись.ВалютнаяСуммаДт = лкВыборка.СтоимостьУЕ;
    КонецЕсли;

лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Контрагенты]  - это тоже через запрос.

Вощем все это запихнуть в запрос..

(14) Может через ВЫБОР ... КОГДА ... ТОГДА?

Ну у тебя же все сущности предопределенные, попробуй через эту конструкцию

Я бы через точку ничего не делал)
Временных таблиц в запросе бы наделал, где все составные поля через ВЫРАЗИТЬ вывел.
А потом соединениями вывел бы выборку.

Ну и эта, почему запрос к документу?
Может быть есть какой-то регистр?

(0) В чем прикол- то? ))

лкЗапись.Регистратор         = лкВыборка.Ссылка;


это надо ?

(18)Контекст неизвестен.

(20) Под вопросом, задача вырвана из контекста.

лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Контрагенты] = лкВыборка.Ссылка.Контрагент;
лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ДоговорыВзаиморасчетов] = лкВыборка.Ссылка.ДоговорВзаиморасчетов;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ВидыДеятельности] = лкВыборка.Ссылка.ВидДеятельности;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Брэнды] = лкВыборка.Ссылка.Брэнд;
    лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.СтавкиНДС] = лкВыборка.СтавкаНДС;

В цикле, каждую итерацию делать запрос к таблице пвх? Может есть смысл иицициализировать эти 4 переменные?
Ну и там верно сказали, подготовить таблицу в запросе, и загрузить ее в движения...

(23) Как это будет выглядеть?

(23)(24) Вижу так,
Эти строки загоняются в ТЗ, затем запросом получаем выборку из ТЗ, присваяивая нужные псевдонимы полям и загружаем в движения документа:
лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Контрагенты] = лкВыборка.Ссылка.Контрагент;
лкЗапись.СубконтоДт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ДоговорыВзаиморасчетов] = лкВыборка.Ссылка.ДоговорВзаиморасчетов;
лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.ВидыДеятельности] = лкВыборка.Ссылка.ВидДеятельности;
лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.Брэнды] = лкВыборка.Ссылка.Брэнд;
лкЗапись.СубконтоКт[ПланыВидовХарактеристик.ВидыСубконтоБУ.СтавкиНДС] = лкВыборка.СтавкаНДС;

(25) Читаем как: "... присваивая нужные псевдонимы полям, и загружаем в движения документа:".

(26) нет. так

ВЫБРАТЬ
                 |    РеализацияАвтомобилей.Ссылка.Контрагент КАК СубконтоДт1,
                 |    РеализацияАвтомобилей.Ссылка.ДоговорВзаиморасчетов КАК СубконтоДт2,
                 |    РеализацияАвтомобилей.Ссылка.ВидДеятельности КАК СубконтоКт1,
                 |    РеализацияАвтомобилей.Ссылка.Брэнд КАК СубконтоКт2,
                 |    РеализацияАвтомобилей.СтавкаНДС КАК СубконтоКт3,
                  |    РеализацияАвтомобилей.Автомобиль,
                 |    РеализацияАвтомобилей.Стоимость,
                 |    РеализацияАвтомобилей.СтоимостьУЕ,
                 |    РеализацияАвтомобилей.Ссылка
                 |ИЗ
                 |    Документ.РеализацияАвтомобилей.Автомобили КАК РеализацияАвтомобилей";

и потом делаешь (7)

а эту хрень из (25) выкинь

Всем спасибо (особенно (27))!!!

(27) В первых четырёх полях запроса следует убрать "Ссылка".

(29) конечно убери "Ссылка" и не забудь сюда пожаловаться )

(29) Это же запрос к табличной части. Поле Ссылка позволяет обратиться к шапке документа.

(31) Точно, спасибо, не заметил.

(32): а еще, наверное, не заметил, что каждое(!) ".Ссылка." - это неявное выполнение запроса... лучче бы соединить и потом тянуть.

(33) Это было сразу замечено, плюс ещё упоминали об этом в (17)?

(34): в (17) глупость, которая совсем не про то. (там - про точку (без которой. строго говоря, никак), я - про две точки в варианте, ведущем к неявному запросу). ну и независимо от - ч упомянул о том, что в варианте, декларируемом в качестве решения, есть конкретное упущение (вне зависимости от то. что там и где было "сразу замечено").

(35) Так и воспринималась информация, что речь в (17) идёт о двух точках и более. Понятное ведь дело, что без точки никак!