Добрый день, коллеги. Подскажите, как ускорить. Надо добавить колонку со свойством номенклатуры в документ. В процедуре "При получении данных" сделал так:

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

Разумеется, дико тормозит. Какие будут идеи, как сделать хорошо?

|    ЗначенияСвойствОбъектов.Значение.Представление
       |ИЗ
       |    РегистрСведений.ЗначенияСвойствОбъектов КАК ЗначенияСвойствОбъектов
       |ГДЕ
       |    ЗначенияСвойствОбъектов.Объект.Ссылка = &СсылкаНом
       |    И ЗначенияСвойствОбъектов.Свойство.Ссылка = &СсылкаСв";

заменить на

       |    ЗначенияСвойствОбъектов.Значение.Представление
       |ИЗ
       |    РегистрСведений.ЗначенияСвойствОбъектов КАК ЗначенияСвойствОбъектов
       |ГДЕ
       |    ЗначенияСвойствОбъектов.Объект= &СсылкаНом
       |    И ЗначенияСвойствОбъектов.Свойство= &СсылкаСв";

Цикл
       ЗапросКСвойствам = Новый Запрос;

Убрать запрос из циула

1. Запрос в цикле - даже при сдаче спеца за это снизят. Формируй одним запросом а потом поиск в полученной тз в цикле.
2. Если уж совсем свойства - дико большая таблица, то можно при открытии ее получить в переменную. Вероятность изменения свойства "крайне мала")) Ну или каждую минуту ее обновлять.

Привести к типу результат через Выразить()

да еще и НайтиПоКоду в цикле

(4)
Вероятность изменения свойства "крайне мала")) Ну или каждую минуту ее обновлять.

Бред

Делать при открытии - заполняеш по всем в ТЧ.
И при изменениии номенклатурины в ТЧ - вытаскиваеш свойства.

>> Элемент.ТекущаяСтрока.Номенклатура.Ссылка
заменить на Элемент.ТекущаяСтрока.Номенклатура

За такие вопросы авторам нужно обрезать радугу до 2мм

Короче код - пример того, как вообще не надо писать код.
Образчик маразма и ошибок.

ПланыВидовХарактеристик.СвойстваОбъектов.НайтиПоКоду("000000013")

"ВЫБРАТЬ РАЗЛИЧНЫЕ ПЕРВЫЕ 1"

нормальная тема! :)

налетели то

СамаЯчейка = ОформлениеСтроки.Ячейки.Производитель;
       СамаЯчейка.ОтображатьТекст = Истина;
       СамаЯчейка.Текст = Строка(Выборка.ЗначениеПредставление);

заменить на

ОформлениеСтроки.Ячейки.Производитель.УстановитьТекст(трока(Выборка.ЗначениеПредставление));

(7) Мотивируй или предложи быструю альтернативу.

мой вывод адресов в списке контрагентов, вроде не тормозит:

Процедура СправочникСписокПриПолученииДанных(Элемент, ОформленияСтрок)
   Контрагенты = Новый Массив();
   Для Каждого Оформление Из ОформленияСтрок Цикл
       ДанныеСтроки = Оформление.ДанныеСтроки;
       Если ДанныеСтроки.ЭтоГруппа Тогда
           Продолжить;
       КонецЕсли;    
       Контрагенты.Добавить(ДанныеСтроки.Ссылка);
   КонецЦикла;    
   Запрос = Новый Запрос;
   Запрос.Текст = "ВЫБРАТЬ
   |    КонтактнаяИнформация.Объект,
   |    КонтактнаяИнформация.Представление
   |ИЗ
   |    РегистрСведений.КонтактнаяИнформация КАК КонтактнаяИнформация
   |ГДЕ
   |КонтактнаяИнформация.Тип = &Тип
   |И КонтактнаяИнформация.Вид = &Вид
   |И КонтактнаяИнформация.Объект в(&Контрагенты)";
   Запрос.УстановитьПараметр("Тип",Перечисления.ТипыКонтактнойИнформации.Адрес);
   Запрос.УстановитьПараметр("Вид",Справочники.ВидыКонтактнойИнформации.ЮрАдресКонтрагента);
   Запрос.УстановитьПараметр("Контрагенты",Контрагенты);    
   
   Выборка = Запрос.Выполнить().Выбрать();
   
   Для Каждого Оформление Из ОформленияСтрок Цикл
       ДанныеСтроки = Оформление.ДанныеСтроки;
       Если ДанныеСтроки.ЭтоГруппа Тогда
           Продолжить;
       КонецЕсли;
       Выборка.Сбросить();
       Если Выборка.НайтиСледующий(ДанныеСтроки.Ссылка,"Объект") Тогда
           Оформление.Ячейки.Адрес.УстановитьТекст(Выборка.Представление);
       КонецЕсли;    
   КонецЦикла;    
       
КонецПроцедуры

(14) заклевали , а Темный наверно вбросил чтобы посмеяться над советчиками

(16) все правильно - это бред. Свойства разные бывают.

(18) это наверное вообще НеТемный

(16) Быстрая альтернатива - написать все правильно.

(17) Херово

Ставлю бутылку кефира - Темный - семерошник.

(21) "Идеальный мир - это мир без косяков" (С)

:-) Спасибо за быстрые ответы, коллеги!
Теперь по пунктам.

(1) Сделаю. Не думаю, правда, что повысит быстродействие на порядок;

(2). Точно. Знаю же об этом. Вынесу текст из цикла;

(26) +1

вообще не заметно будет

(23) они все такие?

(3) Убрать запрос из цикла? То есть, сделать как в (17)?

(28) Бугага

был хороший вопрос - а что вообще за свойство и как часто оно меняется?

(29) Почти.
Опасные аж жуть.

(6) Там 20 строк... Думаю, миллисекунды на это уходят. Но, вынесу, оф коз.

(5) В запросе? Выразить КАК Строка()?

(8) Непонятно, как быть с новым, еще не записанным документом

(13) Ну это можно убрать :-)

(15) Ускорит в 10 раз?

(35) ниче не надо выражать.
      |    ЗначенияСвойствОбъектов.Объект.Ссылка = &СсылкаНом
       |    И ЗначенияСвойствОбъектов.Свойство.Ссылка = &СсылкаСв
за это уволил бы.
убрать ".Ссылка". и будет все летать. с выразить все еще дольше.

(39)+ и это
ЗапросКСвойствам.УстановитьПараметр("СсылкаНом",Элемент.ТекущаяСтрока.Номенклатура.Ссылка);
на
ЗапросКСвойствам.УстановитьПараметр("СсылкаНом",Элемент.ТекущаяСтрока.Номенклатура);
в общем уволил бы уже.

(17) Склоняюсь к такому варианту, но    H A D G E H O G s говорит, что плохо будет... Может, он пояснит свою позицию?

(39) а я бы тебя уволил за "ниче не надо выражать"

(42) если нет обращение через точку к Объект. то выражать не надо. все только тормозит. поэтому увольняй себя сам.

(41) вобще сделано с типового справочника "Сотрудники"

(39) Почему ты не так?

"ВЫБРАТЬ

       |    ЗначенияСвойствОбъектов.Значение.Представление
       |ИЗ
       |    РегистрСведений.ЗначенияСвойствОбъектов КАК ЗначенияСвойствОбъектов
       |ГДЕ
       |    ВЫРАЗИТЬ(ЗначенияСвойствОбъектов.Объект как Справочник.Контрагенты) В (&СписокКонтрагентов)
       |    И ЗначенияСвойствОбъектов.Свойство= &СсылкаСв";

(43) ))) тут разве нет обращения "Значение.Представление"?

(32) Свойство меняется редко.

(41)
|И КонтактнаяИнформация.Объект в(&Контрагенты)"

заменить на

|И ВЫРАЗИТЬ (КонтактнаяИнформация.Объект как Справочник.Контрагенты) в (&Контрагенты)"

(43) Уволь себя.

(43) посмотри во что превращается запрос а потом делай такие выводы

(43) Чем более подробно опишешь составные типы - тем меньше запрос будет болтаться по БД.

ЗначенияСвойствОбъектов.Значение составной тип
и обращение через точку к полям от "Значение" приводит сам знаешь к чем

(45) проверял я это. время ухудшается только. по замеру. вот если после ВЫРАЗИТЬ(ЗначенияСвойствОбъектов.Объект как Справочник.Контрагенты) ставить .Наименование тогда да, желательно выразить.
(52) знаю. тока там не настолько много строк чтобы все было так плохо. но в целом для оптимизации можно.

ВЫРАЗИТЬ(ЗначенияСвойствОбъектов.Объект как Справочник.Контрагенты) В (&СписокКонтрагентов)

"Выразить" лишнее будет.. там тип и так проверяется.

ЗначенияСвойствОбъектов.Значение.Представление
1 .Здесь именно тоже надо выразить.
2. Если точно знаешь основное представление объекта то получай конкретно его (код или наименование)

(48) >> ВЫРАЗИТЬ (КонтактнаяИнформация.Объект как Справочник.Контрагенты) в (&Контрагенты)
это не нужно делать

(51) если нет обращения через точку к реквизиту, то это ничего не изменит. только лишнее условие добавит никому не нужное.

(36) Для новых элементов свойтсво подтягивай 1 раз, при изменении номенклатуры в ТЧ

(54) тип везде проверяется.

вы лучше автора запрос оптимизируйте, че до контрагентов докопались то?

Давайте так, коллеги. Поэкспериментируем, так сказать.
1 этап. Уберу "ссылки", вынесу "НайтиПоКоду", и текст запроса из цикла. Посмотрим, будет ли "летать" (сомневаюсь, есличесно)
2 этап. Сделаю как в (17), с рекомендациями. Думаю, эффект будет более явным :-)

(54) 1С настолько умная, что проверяет каждый элемент массива?
Счаст протестим.

та не, просто в случае проверки ссылок нафиг вообще что-то проверять?

За все увольнения бы уволил. При обращении Строчка.Номенклатура автоматом на кэшируется представление и ССЫЛКА объекта. Так что хоть Номенклатура.Ссылка что Номенклатура.Ссылка.Ссылка.Ссылка.Ссылка = побую.

(61) она разве не по одному полю условие делает?

(62) Доп. фильтр по TRef?

(65) а зачем ссылка разве не уникальная?

(66) Только в разрезе одной таблицы

Внезапно!

Без ВЫРАЗИТЬ
exec sp_executesql N'SELECT
T1._Fld14443
FROM _InfoRg14439 T1 WITH(NOLOCK)
WHERE (T1._Fld14440_TYPE = P1 AND T1._Fld14440_RTRef = @P2 AND T1._Fld14440_RRRef IN (@P3))',N'P1 varbinary(1),@P2 varbinary(4),@P3 varbinary(16)',0x08,0x00000061,0xBF180018F304900E11DE46C5C97947FF

С ВЫРАЗИТЬ

exec sp_executesql N'SELECT
T1._Fld14443
FROM _InfoRg14439 T1 WITH(NOLOCK)
WHERE (CASE WHEN T1._Fld14440_TYPE = P1 THEN T1._Fld14440_RRRef WHEN T1._Fld14440_TYPE IN (P1) THEN @P2 END IN (@P3))', N'P1 varbinary(1),@P2 varbinary(16),@P3 varbinary(16)', 0x08, 0x00000000000000000000000000000000, 0xBF180018F304900E11DE46C5C97947FF

(68) т.е. без выразить круче?

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

(63) сам придумал?

зато мы теперь знаем, что на мисте есть чувак с ником P1

Господа, я в шоке.
Привел процедуру к такому виду:

   СвЭл = ПланыВидовХарактеристик.СвойстваОбъектов.НайтиПоКоду("000000013");
   ЗапросКСвойствам = Новый Запрос;
   ЗапросКСвойствам.Текст =
   "ВЫБРАТЬ ПЕРВЫЕ 1
   |    ЗначенияСвойствОбъектов.Значение.Представление
   |ИЗ
   |    РегистрСведений.ЗначенияСвойствОбъектов КАК ЗначенияСвойствОбъектов
   |ГДЕ
   |    ЗначенияСвойствОбъектов.Объект = &СсылкаНом
   |    И ЗначенияСвойствОбъектов.Свойство = &СсылкаСв";
   
   Для каждого ОформлениеСтроки Из ОформленияСтрок
   Цикл
       
       ЗапросКСвойствам.УстановитьПараметр("СсылкаСв",СвЭл);
       ЗапросКСвойствам.УстановитьПараметр("СсылкаНом",ОформлениеСтроки.ДанныеСтроки.Номенклатура);
       
       Выборка = ЗапросКСвойствам.Выполнить().Выбрать();
       Выборка.Следующий();
       
       СамаЯчейка = ОформлениеСтроки.Ячейки.Производитель;
       СамаЯчейка.ОтображатьТекст = Истина;
       СамаЯчейка.Текст = Строка(Выборка.ЗначениеПредставление);
   КонецЦикла;

Стало работать на порядок быстрее! Собственно, достаточно быстро для комфортной работы. Этап № 2 пожалуй не буду делать, и так все нормально.

(73) давай, будущее поколения тебя вспомнит )

(73) Ну и балбес.
1. Запрос в цикле  
2. ЗначенияСвойствОбъектов.Значение.Представление
-не юзаешь ВЫРАЗИТЬ

(68) кстати, интересно, что при несовпадении типов сравнивает с пустой ссылкой

Ахренеть. Я недооценивал 1С.

exec sp_executesql N'SELECT
T1._Fld14443
FROM _InfoRg14439 T1 WITH(NOLOCK)
WHERE (T1._Fld14440_TYPE = P1 AND T1._Fld14440_RTRef + T1._Fld14440_RRRef IN (@P2 + @P3, @P4 + @P5))',N'P1 varbinary(1),@P2 varbinary(4),@P3 varbinary(16),@P4 varbinary(4),@P5 varbinary(16)',0x08,0x00000061,0xBF180018F304900E11DE46C5C97947FF,0x00004A98,0xBF180018F304900E11DE46C5C97947F2

Это когда в массиве отбора - 2 разнотипных ссылки.

(70) Где ты увидел что без индекса?

Я, разумеется, знал и про то, что от "ссылок" надо избавляться, и что запрос надо объявлять не в цикле, и что все поиски и вообще вычисления лучше делать вне цикла. Но я всегда считал, что это добавит 10-20, ну максимум 30% производительности. Теперь же вижу, что ускорение может быть на порядок, и даже больше.

(77) Получается - анализирует состав списка отбора поэлементно перед формированием запроса. Респект.

(77) С разнотипными не айс..

(75) Вы настаиваете на том, что бы сделать как в (17)?

(77) нифига не понял... куда пустая ссылка в @P2 из (68) делась?

(81) Но 1С оптимизировала как могла это

T1._Fld14440_RTRef + T1._Fld14440_RRRef IN (@P2 + @P3, @P4 + @P5))

(82) Именно, но с поправками

Что означает это условие
T1._Fld14440_RTRef + T1._Fld14440_RRRef IN (@P2 + @P3, @P4 + @P5)

Что надо сложить _Fld14440_RTRef и _Fld14440_RRRef и проверить вхождение?

(84) Т.е. отказываемся от индекса

(78) (CASE WHEN T1._Fld14440_TYPE = P1 THEN T1._Fld14440_RRRef WHEN T1._Fld14440_TYPE IN (P1) THEN @P2 END IN (@P3))
тут индексы на поиск работать не будут, только на скан.
(77) для больших объемов через объединить все будет работать быстрее..
(83) а зачем она нужна, если мы не сравниваем с неопределено?

(83)а, не, понял. ну тогда косяк - если 1типный набор, то в случае несовпадения типов - сравнивает с пустой ссылкой, а если неоднотипный набор - то пустая ссылка в наборе сравнения уже не выдаст "истина"

(83) @P2 там - тип справочника, порядковый номер таблицы. Что непонятно то?

(84) т.е. чисто теоретически совпасть не должно, но в любом случае байбай индексы

(90)=>(89)

(88) Разве? Конкатенации поля поиска не вижу..
Надо проверить

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

(94) Мне кажется что индекс сработает по одному полю.. а потом скан выборки по второму.. Это я хотел бы проверить

(88) Даа, table scan. Прискорбно.

(95) Проверил. Scan.

(96) статистику обнови и еще раз выполни

(97) Печаль

Надо запомнить и никогда так не делать.

(97) план запроса выбирается SQL, чтобы использовались индексы нужно чтобы в таблице было много записей.

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

кстати, тогда кому не дель - с одинаковым уидом в 2 справочниках создайте элементы, а потом в регистр с составным полем обоих этих справочников запись с одним из них запихните, в запросе поиск по В(&другойэлемент) сделайте - попадет в выборку неправильная штука, или нет?

*лень

(101) ой ли? видимо Вы совсем не в курсе скуль сервера..

(103) Вот ведь делать нечего.

(104) Чего проверять.. типы то разные

(103) там не совсем нужно одинаковый УИД у справочников
например справочник 1 имеет порядковый номер 1, справочник 2 порядковый номер 2
уиды должны различаться на 1 чтобы сумма совпала..

(105) Че не так? Нужно больше 400 записей вроде. Не помню точно, что бы индексы юзал.

(107) тип справочник одинаков. =)

(105) вот тебе и ойли, я знаю что говорю

(103) Будет как в (77) И?

(98) Обновил. Тех же щей.

(108) Там вообще то не сложение а конкатенация строк получается

По итогам -
ВЫРАЗИТЬ - убрать из фильтра, вы были правы.

1С сама отбирает по типу метаданных и номеру таблицы и юзает IndexSeek

Если будет ВЫРАЗИТЬ() - 1С пишет хитрый запрос и юзает IndexScan

Ну и разнотипные элементы лучше размещать в разных массивах и выполнять запросы через Объединить (не проверял правда еще)

Зашел, называется, в веточку (** сходил за хлебушком).

Век живи, век учись.

(109) для кластерных индексов достаточно одной записи, чтобы использовался кластеред индекс сик..