Народ, вчера обдумывал проблему медленного создания объектов в 1С
и у меня родился вопрос - почему мы для создания объектов юзаем CreateObject/Instance(ИмяКласса), а не напрямую создаем объект через new CStruct, new CVTExtended и т.д.


В результате проверки этой идеи в 1С++ был реализован подобный подход, пока для примера ускоренное создание ИТЗ и Структуры.

Вот результаты сравнения времени создания объектов:
Цитата:


Цитата:


Т.е. для ИТЗ ускорение 68%-80% и выше при больших объемах
для Структуры 60%
Соответственно для других объектов 1С++ будет аналогичное ускорение.

Это просто здорово.

Проблем с объектами при подобном создании вроде бы не обнаружено, 1С работает с этими объектами нормально, без всяких проблем.

Что скажете? Какие подводные проблемы я пропустил?

artbear писал(а) 04. Июля 2008 :: 04:40:


+10

Для реализации ускорения для создания различных объектов из ВК, написанных на С++, я планирую следующее:
1. Будет использоваться техника создания объектов по прототипу или клонирование, т.е. каждый объект сам знает, как создавать себе подобных. Этим будет достигнута универсальность.
2. Технически все просто
а) Создается интерфейс типа ICloneCreated c методом
  CBLContext* Clone() const = 0;
б) все объекты, которые хотят быстро создаваться, должны реализовывать этот интерфейс
в) в 1С++ в моем, уже существующем классе ускорения создания объектов CCreateInstanceBoost, будет реализован механизм кеширования -
при создании объекта по имени указанное имя ищется в карте методов,
если в карте не найдено соответствия, объект создается, как обычно, по имени, далее проверяется наличие интерфейса ICloneCreated, если он есть, то создается второй объект, который и заносится в карту методов;
Если в карте найдено соответствия, то создается новый объект путем вызова метода Clone()
В итоге все быстро и удобно.

ЗЫ кстати, подозреваю, что быстрое создание ТЗ и СЗ через сериализованное представление как раз и делается через прямое создание путем new

ЗЫ подобное будет только для объектов, созданных на С++ !!

Зарегил хотелку http://www.1cpp.ru/bugs/show_bug.cgi?id=3962

Кстати, вот еще вопрос:
Цитата:

Как раз выложенное мной сравнение и есть сравнение через new и через RTC, c обычным 1C, без загруженной 1С++, разница еще больше

При группировании индексированной таблицы(по группам) - там кажется создается нехилое
количество подтаблиц - поможет?
Или там уже все оптимизировано?

vandalsvq писал(а) 04. Июля 2008 :: 08:26:


Ок, спасибо, просмотрел
Но по любому
+10

kms писал(а) 04. Июля 2008 :: 08:45:

Само собой, предварительно я посмотрел реализацию CreateInstance()
Она делает следующее
1. Идет тупой перебор в цикле имен всех зарегистрированных классов, сравнение через lstrcmpiA - вот основной тормоз
Если объект найден, то по полученному индексу получается Рантайм-класс, после чего создается объект pCont = CRuntimeClass::CreateObject(), далее вызывается pCont = CBLContext::InitObject(ИмяКласса)
2. Если объект не найден в цикле, идет попытка создания КОМ, ОЛЕ-объекта

В принципе уже существующее ускорение создание объектов (ДмитрО + я) - основано на схеме из п.1, только вместо тупого, тормозного перебора идет очень быстрый поиск в CIStringMap,  а далее также создание из Рантайм-класса, и опять вызов ИнитОбъект.

Так что в общем, для объектов, чье поведение и создание известно, можно ускорить процесс создания, для чего и будет задействован интерфейс клонирования.

Да и результаты говорят сами за себя.

ЗЫ обычная беда 1С - такое впечатление, что авторы кода 1С не знают, ничего кроме CArray и тупого перебора в цикле

Могу кинуть тестовую сборку, сам убедишься

Обалдеть - тесты для ИТЗ стали проскакивать просто моментально, почти незаметно - нажал и тут же результаты для 176 тестов, а раньше все-таки приходилось ждать их завершения.

Ура!

kms писал(а) 08. Июля 2008 :: 13:26:

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

А в сборке никогда не будет тестового метода ФабрикаОбъектов::ТестОтключитьБыстроеСоздание_Артур(Парам) - т.к. он фактически нужен только для сравнения, а в реальных базах не нужен.

ЗЫ хотя могу и залить, если все-таки нужно,
все равно метод у меня через переменную препроцессора реализован, т.е. я ее просто отключаю, и метода нету

artbear писал(а) 08. Июля 2008 :: 13:29:

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

Alister писал(а) 08. Июля 2008 :: 14:09:

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

Есть простое предложение.

Избавиться от CBLContext::InitObject(LPCSTR).
Огульно делать это нельзя, ибо InitObject может нести полезную нагрузку.

Но чистый CBLContext::InitObject() - это просто HashMethods(), для CMyContextBase и подобных это не нужно.

P.S.
Никогда бы не подумал, что нужно ТАК хешировать методы.
Зачем это делать при каждом создании объекта?

P.P.S.
Корни проблемы тормозов создания ТЗ и СЗ, полагаю, также уходят в InitObject.

P.P.P.S.
Ппц. ;)

kms писал(а) 08. Июля 2008 :: 19:31:


Нет, все-таки не настолько безполезна
1. Все-таки моя схема немного, но выигрывает у схемы без InitObject()
2. Самое главное - моя схема позволяет много большее - она добавляет к классу технику созданию объекта из прототипа объекта. А т.к. класс намного лучше знает о своем создании, чем его окружение, он может и намного более эффективно создать объект.
Например, для пользовательского класса (а он ведь очень медленно создается даже в режиме оптимизации) можно не выполнять довольно медленный поиск статических данных класса из системных данных, а просто скопировать их из прототипа, и будет достигнуто ускорение.

Поэтому предлагаю компромиссное решение -
оставляем мою схему + избавляемся от InitObject для некоторых классов.

Например, в CMyContextBase в InitObject не делаем вызов CBLContext::InitObject и т.д.

artbear писал(а) 09. Июля 2008 :: 03:39:

Где ж ты его нашел?
В свое время CIStringMap это не умел, а CMap так и не умеет.
Но это давно уже не актуально.

Цитата:

В точности.

Цитата:

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

В принципе, не вопрос пусть остается.
Но для ИТ и для большинства классов лучше убрать этот интерфейс из списка базовых.
И просто объявить пустые InitObject().

P.S.
Хорошо, что наконец-то разобрались с этим вопросом.

kms писал(а) 09. Июля 2008 :: 06:34:

Согласен.
ИМХО подобное нужно сделать и для базового класса CMyContextBase

kms писал(а) 09. Июля 2008 :: 07:40:

А смысл заливать для Групповых Контекстов ?
ИМХО их создание и так тормозит по куче других причин, так что здесь выигрыша и не увидишь.

Сейчас скомпилируется, запущу тесты.

Это не для групповых, а для чистых CBLContext зашунтировано сейчас.
--
Да, нужно избавляться от HashMethods/HashProps.
Но с ним еще связаны FindMethod/FindProp.
Если их не изменить, будет опять тормозить поиск номера метода для ряда контекстов.

А еще контексты могут быть динамическими (как контекст ТЗ, например).
Наверное, поэтому, Саша и не сделал универсальным механизм поиска номера метода.

У тебя в методе погрешность измерений приличная.
Лучше стандартно в цикле создавать N объектов и замерять время.

А вот почему результаты выдаются таймером в разном порядке - это я не знаю.

kms писал(а) 09. Июля 2008 :: 10:32:

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

ИМХО нужно перехват InitObject перенести из ТурбоБЛ в мой класс CCreationBoost, все-таки ускорение создания - это задача это класса.

Михаил, что скажешь?

kms писал(а) 10. Июля 2008 :: 01:57:

Согласен, я также очень доволен, пусть в итоге и не мой метод победил - главное, нашли отличный вариант
Очередное ура !!

[quote author=artbear link=1215146216/30#43 date=1215665213]
Михаил, будем выпускать релиз ?
Пусть народ пощупает сборку и потестирует?[/quote]
Конечно, выпускай.
Я уж думал, пока пару часиков посплю, ты и выпустишь. ;)

P.S.
Закрываю номера: 3851, 3916, 3969.
Развернутые описания потом добавлю прямо в wiki.

По вектору можно пока не спешить - в следующий раз, наверное, войдет.

[quote author=kms link=1215146216/30#44 date=1215677167]
Конечно, выпускай.[/quote]
Мне чё-т первый раз страшновато качать новую сборку. ИМХО, это очень существенные изменения.
Ну если гуру скажут, что всё пучком - скачаю и даже протестирую на своих!  ;)

С удовольствием потестим  

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

[quote author=artbear link=1215146216/45#47 date=1215685460]У меня все тесты проходят, т.е. куча различных объектов, как 1С++, так и 1С, создается и уничтожается прекрасно, в т.ч. и справочники, и документы.[/quote]
Артур, подскажи, плиз как можно протестировать корректное уничтожение объекта.

artbear писал(а) 10. Июля 2008 :: 10:18:

Выпустил.
Ты тогда по необходимости дополни описание по закрытым тобой заявкам.