Тест скомпилирован на базе последних исходников icpp?

Arta писал(а) 25. Августа 2008 :: 21:34:

Есессно.
Если не использовать добавленный метод контекстов - это обычный trunk icpp.

artbear писал(а) 26. Августа 2008 :: 03:10:

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

P.S.
Там очень простая реализация.
Нам сейчас надо оценить безопасность модели.

artbear писал(а) 26. Августа 2008 :: 06:15:

Т.е. у тебя в прямом редактировании ТП будет вызвано CGroupContext::Accept, ТурбоБЛ обновит данные контекста, а твои добавленные контексты пропадут
Т.е. ИМХО код из вызова метода

std::vector<CBLPtr<CBLContext> > vc;
std::swap(vc, m_AddedCtx);
pGroupCont->Accept();
std::swap(vc, m_AddedCtx); 

нужно встраивать непосредственно в перехваченный в ТурбоБЛ CGroupContext::Accept !

ИМХО при добавлении контекста было бы удобно передавать некий флаг, регулирующий проверку совпадающих свойств и методов у исходного контекста и добавляемого.
Типа если Флаг = 1 (разработка), то происходит анализ совпадения и выдаются ошибки в окно сообщений
Если Флаг = 0 (боевая база), то нет никаких проверок, все тупо и быстро.

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

artbear писал(а) 26. Августа 2008 :: 06:15:

http://www.1cpp.ru/forum/YaBB.pl?num=1219421843/19#19

Артур, надо учиться выпускать релизы.

artbear писал(а) 26. Августа 2008 :: 11:10:

Потом расскажу.
...
Лана, щас расскажу.

Ну, вкратце - я могу сделать точно такую же безопасную коллекцию, но из-за реализации контекстов в 1С доступ по имени свойства будет иметь сложность O(N).
Потому что в итоге приходится находить по имени не просто пару "ключ, значение", а ее номер в коллекции, а это - O(N).
Поэтому использован некий хак, который позволяет решить эту задачу со сложностью const.
Но чем-то приходится жертвовать - и в данном случае - это жестко регламентированный протокол использования методов контекста.

Для ассоциативного вектора нельзя кешировать номера свойств.
Можно только искать свойства через FindProp, а после получать значения через возвращенный дескриптор (а не номер) свойства.
Это прекрасно работает в родном окружении 1С.
Но вот добавить такой контекст через TurboBL увы, не получится.

Да и не велика потеря.
Я только сейчас заметил, что и добавление структуры у меня в Тест2() работает неверно (ну или "весьма своеобразно").
Свойства-то структуры к контексту добавляются, но только те, которые были добавлены к контексту до добавления контекста структуры.
Но вот дальнейшая вставка свойств уже в контексте не отображается.
Например, нельзя получить свойство структуры "Десять" через контекст.
По крайней мере, до тех пор, пока не будет вызван очередной Accept().

Т.е. добавление полиморфных контекстов к ГК не имеет смысла.
Все равно TurboBL не заточена под поддержку таких контекстов в соединении с ГК.

А вот добавлять что-то статическое, типа первого теста - это должно работать.
Можно даже придумать, когда это действительно будет нужно.

P.S.
Я так и знал, что добавление контекстов - это очередной ящик Пандоры...

artbear писал(а) 27. Августа 2008 :: 03:47:

Исправил.