есть регистр сведений
Измерения:
Номенклатура
СерияНоменклатуры
Ресурсы:
Остальный пар-ры

МассивУзлов=ПроцедурыОбменаДаннымиПоМагазину.ВернутьМассивУзловПоОдномуМагазину(Магазин, СоответствиеУиМ);
Если МассивУзлов <> Неопределено И МассивУзлов.Количество() > 0 Тогда
НаборЗаписей=РегистрыСведений.бгсЗначенияСвойствХарактеристик.СоздатьНаборЗаписе();
   НаборЗаписей.Отбор.Номенклатура.Установить(Номенклатура,ИСТИНА);
   НаборЗаписей.Отбор.СерияНоменклатуры.Установить(Справочники.бгсСерииНоменклатуры.ПустаяСсылка(),ИСТИНА);
   НаборЗаписей.Прочитать();
   для каждого запись из НаборЗаписей цикл
       запись.Скидка=Скидка;
   конеццикла;
   НаборЗаписей.Записать();//при мониторинге производ. это срока
//ест 60%
   ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов,НаборЗаписей);
КонецЕсли;
Помогите оптимизировать код.

И что удивительного?

"СоздатьНаборЗаписе()" - это работает?

60% от 1 часа или 1 микросекунды?

этот код полностью рабочий,
но медленный при большом объеме

60% от 7 часов

(0) так будет быстрее

//НаборЗаписей.Записать();//при мониторинге производ. это срока

(4) брешишь, см (2)

Может у тебя вместо винчестера секретарь-стенографист...

точнее сказать от всего затраченного времени на процедуру

Или у тебя эта строка повторяется миллиард раз

(6) так еще быстрее:

//МассивУзлов=ПроцедурыОбменаДаннымиПоМагазину.ВернутьМассивУзловПоОдномуМагазину(Магазин, СоответствиеУиМ);
//Если МассивУзлов <> Неопределено И МассивУзлов.Количество() > 0 Тогда
//НаборЗаписей=РегистрыСведений.бгсЗначенияСвойствХарактеристик.СоздатьНаборЗаписе();
//    НаборЗаписей.Отбор.Номенклатура.Установить(Номенклатура,ИСТИНА);
//    НаборЗаписей.Отбор.СерияНоменклатуры.Установить(Справочники.бгсСерииНоменклатуры.ПустаяСсылка(),ИСТИНА);
//    НаборЗаписей.Прочитать();
//    для каждого запись из НаборЗаписей цикл
//        запись.Скидка=Скидка;
//    конеццикла;
//    НаборЗаписей.Записать();//при мониторинге производ. это срока
//
////ест 60%
//
//    ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов,НаборЗаписей);
//КонецЕсли;

8 ядер 32ГБ ОЗУ и винты по 10000

самый оптимальный вариант:

строка повторяется 250 000 раз

(0) поменяй бгсСерииНоменклатуры на бггСерииНоменклатуры

Это оптимально)))

СоздатьНаборЗаписей() - Й не влез

Сделай набор общий и запиши его один раз

угу и все записи регистра похерить?
сам регистр состоит из 3 млн записей

Используй транзакции блоками.

а если читать наборы по номенклатуре, а потом менять только необходимые серии?

НаборЗаписей.Отбор.СерияНоменклатуры.Установить(Справочники.бгсСерииНоменклатуры.ПустаяСсылка(),ИСТИНА); - это исключить

Была, была подобная ветка...

*** так зло, с прищуром....

(19) а каков глубинный смысл этого регистра

Юннат тоже утверждал, что запись тормозит.
Мы уж и индексы отключали, и измерения местами меняли и вывод статуса на экран отключали...

А все оказалось дико проще.

Жаль, архив отключен, найти не могу...

Я бы предложил взять в руки ассемблер... Но кто из нынешних 1Сников знает, что это такое...?

если % записей больше определенного от общего количества, то быстрее прочитать все без отбора, поменять нужные, записать все бехз отбора. но 2гб на клиенте может не хватить, надо на сервере делать, причем 64битном, если не хватает

% определяется опытным путем, обычно 10-20

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

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

(32) там может быть только вид сравнения "равно"

(19) а зачем 2 раза регистрировать изменения? в ПриЗаписи() регистра они и так регистрируются.

я тоже думаю, что решение есть!!!!
отбор по списку не работает

изменить нужно ресурс согласно отбора

H A D G E H O G s - вспомнил решение?

(37) может транзакции спасут гиганта мысли? По 100-200 номенкклатур в каждой транзакции.

(34) а если авторегистрации нет?

(39) наоборот, если авторегистрация, то на уровне платформы регистрируется, а если авторегистрации нет, то в подписках ПередЗаписью и ПриЗаписи. Наверняка у него в подписках всё это есть.

авторегистрации нет
при транзакции Записать(); - будет делать столько же раз с тойже скоростью?

Код в транзакции выполняется?

Если попробовать кусками писать?

СоздатьНаборЗаписе - это опечатка ?

НаборЗаписей.Прочитать(); -сколько времени жрется на это?

Попробуй данные запросом вытащь.. чтение набора записей не юзать

кстати, "прочитать"  действительно здесь нафиг не сдалось

(45) он скидку меняет

43 - да
45 - нужно ... если есть др вариант напишите

(47) зачем?
два измерения
по обоим отбор на равенство
запись только одна

тут вообще через менеджер можно

... единственное - если не нашли запись, то ничего не создается - но это решается запросом и отбором только необходимых записей

в общем код не оптимален - у тебя пустых наборов сколько записывается?

(41) причем тут записать()? у тебя сейчас допустим 100000 транзакций, а если сделаешь их порциями, то будет например 1000 транзакций. Ощущаешь разницу?

(48)(49) Регистр может быть периодическим/подчинен регистратору.. возможно автор не указал это

(50) ЗначенияСвойствХарактеристик ?? подчинен регистратору?

(50) тогда в отборе должен быть период и регистратор, не так?

52 - нет

НаборЗаписей.Прочитать();
   для каждого запись из НаборЗаписей цикл
       запись.Скидка=Скидка;
   конеццикла;
   НаборЗаписей.Записать();//при мониторинге производ. это срока

как минимум так:

 НаборЗаписей.Прочитать();
   для каждого запись из НаборЗаписей цикл
       запись.Скидка=Скидка;
       ЗаписатьНабор = Истина;
   конеццикла;
   Если ЗаписатьНабор Тогда
       НаборЗаписей.Записать();//при мониторинге производ. это срока

(51) ЗначенияСвойствХарактеристик в типовых нет..
а вот бгсЗначенияСвойствХарактеристик вполне возможно

(54) Да запросом надо данные считать.. там же и проверить менялась ли скидка

48 при одном прочтении - 1 строчка
54 - у меня нет пустых наборов
55- регистр не типовой
56- это изменения их точно надо применять в проверке не нуждается

(57) ставь номер поста, на который отвечаешь в скобочки

(48) при одном прочтении - 1 строчка
(54) - у меня нет пустых наборов
(55)- регистр не типовой
(56)- это изменения их точно надо применять в проверке не нуждается

(0) Попробуйте выяснить почему НаборЗаписей.Записать() жрет столько времени. Может быть там в модуле регистра процедура ПриЗаписи() вызывающая десятка полтора других процедур из разных модулей? Ситуация абсолютно реальная, приходилось сталкиваться.

(60) пусто

Форумчанне дайте совет по оптимизации

(62) а совет писать либо все сразу, либо в транзакции по 100-1000 штук чем не угодил?

(62) Кучу советов дали же:
1) Попробовать обработать через менеджер (самому приходилось сталкиваться с тем что он работает в разы быстрее)
2) Сделать дополнительные проверки, меняется ли скидка
3) Порезать набор на Н-частей
4) Индексирование
5) Получение данных из запроса (на получение же данных тратиться 2-3 часа)

Это все дело теста, универсальную панацею никто не даст

(64)
1. пробую
2. сделал, еще до того как на форум написал
3. производительность ухудш. на 10%
4. скулем?
5. если я будут получать данные запрос, то как мне менять их ?

(65)
4. индексирование измерений у регистра
5. менеджером + при этом постараться избавиться от "трех точек" в цикле:
Справочники.бгсСерииНоменклатуры.ПустаяСсылка()
такие вещи сильно посаживают производительность

НаборЗаписей=РегистрыСведений.бгсЗначенияСвойствХарактеристик.СоздатьМенеджерЗаписи();
НаборЗаписей.Номенклатура=Номенклатура;
НаборЗаписей.СерияНоменклатуры=Серия;
НаборЗаписей.Скидка=Скидка;
НаборЗаписей.Записать();
НаборЗаписей=РегистрыСведений.бгсЗначенияСвойствХарактеристик.СоздатьНаборЗаписей();
НаборЗаписей.Отбор.Номенклатура.Установить(Номенклатура,ИСТИНА);
НаборЗаписей.Отбор.СерияНоменклатуры.Установить(Серия,ИСТИНА);
НаборЗаписей.Прочитать();

ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов,НаборЗаписей);

Скорость увеличил на 5%

Подскажите, как заменить этот кусок кода
НаборЗаписей.Отбор.Номенклатура.Установить(Номенклатура,ИСТИНА);
НаборЗаписей.Отбор.СерияНоменклатуры.Установить(Серия,ИСТИНА);
НаборЗаписей.Прочитать();