Делаю небольшую шабашку - наткнулся на код который меня прямо в ступор ввел
УПП 1.2.16.1, Документ "ФормированиеПотребностей"
Фрагмент кода по заполнению табличной части потребностей

Для каждого Строка из ЗаказыНаПроизводство Цикл
       ЗапросНовыеПотребности.УстановитьПараметр("ЗаказНаПроизводствоДатаПотребности" + Формат(Индекс, "ЧГ=0"), НачалоДня(Строка.ДатаПотребности));
       
       ЗапросНовыеПотребности.Текст = ЗапросНовыеПотребности.Текст + "
       |ОБЪЕДИНИТЬ ВСЕ
       |
       |ВЫБРАТЬ
       |    &ДатаДокумента КАК Период,
       |    ЗаказыНаПроизводствоОстатки.Номенклатура КАК Номенклатура,
       |    ЗаказыНаПроизводствоОстатки.ХарактеристикаНоменклатуры КАК ХарактеристикаНоменклатуры,
       |    ЗаказыНаПроизводствоОстатки.Номенклатура.ЕдиницаХраненияОстатков КАК ЕдиницаИзмерения,
       |    &Товар КАК ТоварТара,
       |    Ложь КАК Тара,
       |    &ЗаказНаПроизводствоДатаПотребности" + Формат(Индекс, "ЧГ=0") + " КАК ДатаПотребности,
       |    ЗаказыНаПроизводствоОстатки.ЗаказНаПроизводство КАК Заказ,
       |    ЗаказыНаПроизводствоОстатки.Номенклатура КАК КонечнаяПродукция,
       |    &ПроектПустаяСсылка КАК Проект,
       |    &СценарийПустаяСсылка КАК Сценарий,
       |    ЗаказыНаПроизводствоОстатки.Спецификация КАК Спецификация,
       |    ЗаказыНаПроизводствоОстатки.ВидВоспроизводства КАК ВидВоспроизводства,
       |    ЗаказыНаПроизводствоОстатки.КоличествоОстаток КАК Количество
       |ИЗ
       |    РегистрНакопления.ЗаказыНаПроизводство.Остатки(КОНЕЦПЕРИОДА(&ДатаДокумента, ДЕНЬ), Номенклатура.Услуга = Ложь";
       
       Если ЗначениеЗаполнено(Строка.Заказ) Тогда
           
           ЗапросНовыеПотребности.УстановитьПараметр("ЗаказНаПроизводство" + Формат(Индекс, "ЧГ=0"), Строка.Заказ);
           ЗапросНовыеПотребности.Текст = ЗапросНовыеПотребности.Текст + " И ЗаказНаПроизводство = &ЗаказНаПроизводство" + Формат(Индекс, "ЧГ=0");
           
       КонецЕсли;
       
       ЗапросНовыеПотребности.Текст = ЗапросНовыеПотребности.Текст +
       ") КАК ЗаказыНаПроизводствоОстатки
       |
       |ГДЕ ЗаказыНаПроизводствоОстатки.КоличествоОстаток > 0
       |";
       
       Индекс = Индекс + 1;
       
   КонецЦикла;
   
   Если ЗапросНовыеПотребности.Текст <> "" Тогда
       
       ЗапросНовыеПотребности.Текст = Сред(ЗапросНовыеПотребности.Текст, 16);
       Результат = ЗапросНовыеПотребности.Выполнить().Выгрузить(ОбходРезультатаЗапроса.Прямой);
       РаспределитьПоНоменклатуре(НовыеПотребности, Результат);
       
КонецЕсли;

Блин.. ну и как после этого конфа может не тормозить??

А в чем проблема. Остатки по датам из ТЧ

Запрос выполняется ввне цикла

(2) Внимательно посмотри. Идет склейка запроса на каждый заказ строки

А как еще помжно получить остатки по разным датам?

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

и склейка идет не по каждой строке а по каждому конкретному заказу.

Еще один мнящий себя гуру )) автар вроде не пятница .

Т.е. запрос такого типа в порядке вещей?

"ВЫБРАТЬ
|    &ДатаДокумента КАК Период,
|    ЗаказыНаПроизводствоОстатки.Номенклатура КАК Номенклатура,
|    ЗаказыНаПроизводствоОстатки.ХарактеристикаНоменклатуры КАК ХарактеристикаНоменклатуры,
|    ЗаказыНаПроизводствоОстатки.Номенклатура.ЕдиницаХраненияОстатков КАК ЕдиницаИзмерения,
|    &Товар КАК ТоварТара,
|    ЛОЖЬ КАК Тара,
|    &ЗаказНаПроизводствоДатаПотребности КАК ДатаПотребности,
|    ЗаказыНаПроизводствоОстатки.ЗаказНаПроизводство КАК Заказ,
|    ЗаказыНаПроизводствоОстатки.Номенклатура КАК КонечнаяПродукция,
|    &ПроектПустаяСсылка КАК Проект,
|    &СценарийПустаяСсылка КАК Сценарий,
|    ЗаказыНаПроизводствоОстатки.Спецификация КАК Спецификация,
|    ЗаказыНаПроизводствоОстатки.ВидВоспроизводства КАК ВидВоспроизводства,
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток КАК Количество
|ИЗ
|    РегистрНакопления.ЗаказыНаПроизводство.Остатки(
|            КОНЕЦПЕРИОДА(&ДатаДокумента, ДЕНЬ),
|            Номенклатура.Услуга = ЛОЖЬ
|                И ЗаказНаПроизводство = &ЗаказНаПроизводство) КАК ЗаказыНаПроизводствоОстатки
|ГДЕ
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток > 0
|
|ОБЪЕДИНИТЬ ВСЕ
|
|ВЫБРАТЬ
|    &ДатаДокумента,
|    ЗаказыНаПроизводствоОстатки.Номенклатура,
|    ЗаказыНаПроизводствоОстатки.ХарактеристикаНоменклатуры,
|    ЗаказыНаПроизводствоОстатки.Номенклатура.ЕдиницаХраненияОстатков,
|    &Товар,
|    ЛОЖЬ,
|    &ЗаказНаПроизводствоДатаПотребности1,
|    ЗаказыНаПроизводствоОстатки.ЗаказНаПроизводство,
|    ЗаказыНаПроизводствоОстатки.Номенклатура,
|    &ПроектПустаяСсылка,
|    &СценарийПустаяСсылка,
|    ЗаказыНаПроизводствоОстатки.Спецификация,
|    ЗаказыНаПроизводствоОстатки.ВидВоспроизводства,
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток
|ИЗ
|    РегистрНакопления.ЗаказыНаПроизводство.Остатки(
|            КОНЕЦПЕРИОДА(&ДатаДокумента, ДЕНЬ),
|            Номенклатура.Услуга = ЛОЖЬ
|                И ЗаказНаПроизводство = &ЗаказНаПроизводство1) КАК ЗаказыНаПроизводствоОстатки
|ГДЕ
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток > 0
|
|ОБЪЕДИНИТЬ ВСЕ
|
|ВЫБРАТЬ
|    &ДатаДокумента,
|    ЗаказыНаПроизводствоОстатки.Номенклатура,
|    ЗаказыНаПроизводствоОстатки.ХарактеристикаНоменклатуры,
|    ЗаказыНаПроизводствоОстатки.Номенклатура.ЕдиницаХраненияОстатков,
|    &Товар,
|    ЛОЖЬ,
|    &ЗаказНаПроизводствоДатаПотребности2,
|    ЗаказыНаПроизводствоОстатки.ЗаказНаПроизводство,
|    ЗаказыНаПроизводствоОстатки.Номенклатура,
|    &ПроектПустаяСсылка,
|    &СценарийПустаяСсылка,
|    ЗаказыНаПроизводствоОстатки.Спецификация,
|    ЗаказыНаПроизводствоОстатки.ВидВоспроизводства,
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток
|ИЗ
|    РегистрНакопления.ЗаказыНаПроизводство.Остатки(
|            КОНЕЦПЕРИОДА(&ДатаДокумента, ДЕНЬ),
|            Номенклатура.Услуга = ЛОЖЬ
|                И ЗаказНаПроизводство = &ЗаказНаПроизводство2) КАК ЗаказыНаПроизводствоОстатки
|ГДЕ
|    ЗаказыНаПроизводствоОстатки.КоличествоОстаток > 0
|
|ОБЪЕДИНИТЬ ВСЕ
|
...

и так 200 раз по числу строк

Я валяюсь с тех, кто валяется с кода УПП.
Покажи как ты бы сделал

Мужики я не одинЭснег где смеятся скажите

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

А типа передать во времянку ТаблицуДокумента и слева соединить не кошерно?

(12) Ты не заюзаешь виртуальные таблицы , то твой вариант будет тормозить больше

(13) А точнее виртуальные параметры . Выход вижу только в другой организации регистра , а запрос корректный.

(13) Временную таблицу я не смогу использовать в параметрах виртальной таблицы??
Не смеши меня

(15) Имел виду параметры виртуальной таблицы . Смеятся не чему.

(15) Ну ты ТУПОЙ. Остатки на РАЗНЫЕ, Р-А-З-Н-Ы-Е даты

(17) +1

Можно конечно взять обороты и там вертеть ее , но это будет тормозить ооочень .

(8) ты прeдложи по другому

Можно конечно одинаковые даты группировать, но смыла нет - сиквел и так закэширует

(17) Дебил.. глянь запрос
КОНЕЦПЕРИОДА(&ДатаДокумента, ДЕНЬ) -остатки на дату документа ФормированияПотребностей

(22) бугага не позорся.

(22)ЗапросНовыеПотребности.УстановитьПараметр("ЗаказНаПроизводствоДатаПотребности" + Формат(Индекс, "ЧГ=0"), НачалоДня(Строка.ДатаПотребности));

На личные оскорбления не переходим...

где строка это каждый документ из ЗаказыНаПроизводство

(26) Ну.. и она благополучно выводится как реквизит запроса

+27 |    &ЗаказНаПроизводствоДатаПотребности1,

С пятницей .. программеры млин..
Признавайтесь, кто из вас такую лажу написал?

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

(30) Запрос в (8)
Там понятно или мне полностью выложить?

можно было б и общий сделать но значит там где то подвох. ты же написал кусок только.
(31) пля а до этого где код?
вот это ведь не даром стоит ЗапросНовыеПотребности.УстановитьПараметр("ЗаказНаПроизводствоДатаПотребности" + Формат(Индекс, "ЧГ=0"), НачалоДня(Строка.ДатаПотребности));

ЗапросНовыеПотребности = Новый Запрос;
   ЗапросНовыеПотребности.УстановитьПараметр("ДатаДокумента", Дата);
   ЗапросНовыеПотребности.УстановитьПараметр("ПроектПустаяСсылка", Справочники.Проекты.ПустаяСсылка());
   ЗапросНовыеПотребности.УстановитьПараметр("СценарийПустаяСсылка", Справочники.СценарииПланирования.ПустаяСсылка());
   ЗапросНовыеПотребности.УстановитьПараметр("СпецификацияПустаяСсылка", Справочники.СпецификацииНоменклатуры.ПустаяСсылка());
   ЗапросНовыеПотребности.УстановитьПараметр("ПустаяДата", Дата('00010101000000'));
   ЗапросНовыеПотребности.УстановитьПараметр("Товар", Перечисления.ТоварТара.Товар);
   ЗапросНовыеПотребности.УстановитьПараметр("Тара", Перечисления.ТоварТара.Тара);
   ЗапросНовыеПотребности.УстановитьПараметр("ВозвратнаяТара", Перечисления.СтатусыПартийТоваров.ВозвратнаяТара);

и особенно еще вот это
  |    &ЗаказНаПроизводствоДатаПотребности" + Формат(Индекс, "ЧГ=0") + " КАК ДатаПотребности,

(33) да ты тупишь. точно тебе сказали. в 34 строка говорящая о том что каждый запрос на разную дату потребности строится.

Хотя может ты и прав. Извиняюсь

в каждом запросе у всех строк относящихся к конкретному запросу устанавдивается определенная дата потребности по параметру. этот параметр определяется в цикле как и каждый заказ.

(35) Е-мое.. ты запрос видешь или нет? Фрагмент кода тоже есть.. где млин остатки на разную дату вылазят?

Так, Маня успакаиваемся!

Сейчас я популярно для маньяков разъясню как надо писать запросы.

Широкий абсолютно прав - запрос г@вно.
Достаточно сунуть таблицу документа в временную и левым соединением прекрасно можно получить эту самую дату потребности из ТЗ и все прочие поля из регистра. Маньяк - сначала подучи SQL, прежде чем возражать. Собственно отбор по заказам на производство сделать элементарно выбрав их из вложенной таблицы. Да даже если бы остатки требовались на дату потребности - то решается это запросом на раз-два.

Так что, с коллегой из (0) АБСОЛЮТНО СОГЛАСЕН.

(40) Меня смущает левое соединение.

Напиши запрос и сравни по времени выполнения , твой будет уступать.

200 объединений ВСЕГДА будет медленнее одного левого соединения.

(42) 200 возможно, надо тестить .

Стандартный глюк у начинающих программистов, которые не сталкивались с SQL - использовать циклы где попало. УПП большая, есть хороший код, есть тупой. Всякое бывает. У меня сейчас перед глазами стоит код, который получает сальдо по плану счетов в Галактике - вот это космос!

(44) Лей, любопытно..

(45) смеяца не будете? :-)

(46) Ты что ли написал его ? )

(40)
УПП 1.2.16.1 вышла с 8.1.9.57  или 8.1.10.50

временных таблиц еще не было

(47) упаси господи! Просто пришлось остатки перетаскивать.
Вот ссылочка:
http://www.tyumbit.ru/gal_forum/viewtopic.php?t=8815&sid=3074a14417054a5554b7dcce540b8cc8

Посмотрел в последней (27) версии, возможно все не так однозначно. Из всех табличных частей данные выбираются по такому же принципу. Код понятный, читается хорошо. И не факт что особо тормозит именно из за кучи объединений. Так что предложение Широкому - предложи свой альтернативный вариант более подробно.

(0)
Вообще то надо разделять красоту запроса и его эффективность
Запрос конечно некрасивый.
Но компиляция занимает относительно немного времени по сравнению
с извлечением данных.

Если ЗаказНаПроизводство входит в индекс таблиц , то существенного
прироста не получишь.
Даже наоборот - бывают короткие запросы , но которые приводят к
сканированию всей таблицы регистра.

Короче приведи свой и время выполнения.

да все бывает. вот например в 27 релизе есть тупо пропущенные переменные и по два раза объявленные функции в одном модуле. т.е. элементарно не проходит синтаксис контроль.
ну что поделать - торопились люди.
а в бюджетировании есть места где в цикле читается константа ВалютаРегламентированногоУчета (по количеству строк документа) кажется в УчетФактическихДанныхПоБюджетам - просто молча исправляем и думаем: "а если бы я такую сложную конфу писал? сколько бы еще более тупых ошибок сделал?"

(48) Я не в курсе откуда у тебя такая инфа, но в коде Временные таблицы встречаются достаточно часто.

(50) "И не факт что особо тормозит именно из за кучи объединений" - ты сам то понял что сказал?

(51) Я свой вариант я уже описывал: скинуть таблицу документа во временную таблицу, потом левое соединение с остатками. В параметры виртуальной таблицы остатков также добавить фильтр по времянке.

Подобный запрос использовался в ЗУП 8.0 при расчете НДФЛ. Количество равнялось количеству месяцев, прошедших с начала года. Иногда захватывался прошлый год. В итоге при выполнении такого запроса в декабре или январе SQL-сервер давал ошибку. В версии 8.1 в запросе стали использовать временную таблицу. Это хороший пример, как не нужно программировать. Уже из-за этакого стиля разработчиков 1С можно отнести к вредоносному ПО. :)

(54) Количество объединений равнялось количеству месяцев ...

(54) Это лано еще количеству месяцев .. а тут, блин, соответствие количеству строк.. а их может быть их как одна, так и пару штук..

(53) а ты напиши свой вариант и сравни время выполнения, если не лень =)

(57) Да я и так тебе могу сказать .. время выполнения сократиться как минимум раза в 3.. если не больше

(58) а я вот возьму и не поверю =) если уж речь заходит о производительности, то без конкретных тестов на рабочих данных нельзя говорить о каком либо варианте...

Автор,а ты поновее релизы не смотрел? Иногда там здорово перекраиваются запросы. Может быть там и сабж здорово исправлен?

(58) А можно ваш код? Проверим.)

(59) Пиво подгонишь?
(60) Это возможно.. Но вот только это шабашка.. а задание состоит не в оптимизации

(62) неа

Для чего так пишут в типовой УПП:


СуммаВключаетНДС = (СуммаВключаетНДС = Истина);

А вы что думаете?

http://1csoft.com/node/861

(64) я так делаю, когда в СуммаВключаетНДС может лежать еще и Неопределено

(64) Это когда СуммаВключаетНДС не обязательно булево

(65,66)а почему бы не написать?

СуммаВключаетНДС = Булево(СуммаВключаетНДС)

(67) Потому что: {(1)}: Преобразование значения к типу Булево не может быть выполнено

(0) "ЗапросНовыеПотребности.УстановитьПараметр("ЗаказНаПроизводство" + Формат(Индекс, "ЧГ=0"), Строка.Заказ);" я так понял что все объединения ради этой строчки

наследие от 8.0