Тест скомпилирован на базе последних исходников 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:

Исправил.

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

Опс, совсем забыл про них.
Хорошо, что ты вспомнил.

artbear писал(а) 28. Августа 2008 :: 04:38:

А я что-то сомневаюсь.
Есть ли смысл выносить этот метод из Системы в контекст?

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

Кстати, а зачем он, вообще, нужен?
Это же такой внутренний метод, зачем он нужен пользователям?

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

Мне не хотелось бы давать такую возможность.
Ибо перестроение контекста - операция небыстрая, небезопасная.
Я бы скорее сейчас считал дизайн, где такой метод необходим, неправильным (я про принудительный Accept).

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

Понимаешь, я не то, чтобы именно по поводу обновления сомневаюсь.
Я вообще сомневаюсь по всему этому творчеству по добавлению контекстов.

Там есть подводные камни.
Например: ТП кеширует номера методов формы (для событий).
При добавлении контекстов, номера методов будут смещаться.

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

Ладно, не обращай внимания это рабочие сомнения такие...

trdm писал(а) 29. Августа 2008 :: 08:08:

Не, Дим, это работает только для ГК (групповых контекстов).
И у добавленного контекста нет доступа к оригинальному контексту.

Т.е. это скорее добавка к Артурову Перехватчику.

Перем ДопСвойство;

Функция ВернутьКонтекст(конт)
	Возврат Конт;
КонецФункции


//*******************************************
Процедура Сформировать()

КонтФормы=ВернутьКонтекст(контекст);
ДопСвойство=1;//Это работает.

Структура = СоздатьОбъект("Структура");
Структура.Вставить("ДопСвойство", "ПростоСтрока");

КонтФормы.__ДобавитьКонтекст(Структура);

Сообщить("Новое свойство = "+КонтФормы.ДопСвойство);

ДопСвойство=1;//А ЭТО УЖЕ НЕ РАБОТАЕТ!!!!!!!
КонтФормы.ДопСвойство=1;// теперь только так работает!!
КонецПроцедуры 

Sergey

Цитата:

Так надо - ты же добавил 10 контекстов.
Другое дело, что сможешь пользоваться только одним - остальные 9 будут недоступны.

Цитата:

Верно, не работает.
Чтобы такое работало, необходимо перекомпилировать модуль.
Здесь об этом и речи не идет - поэтому доступ возможен только через групповой контекст (у тебя - КонтФормы).

Цитата:

Я бы не советовал использовать эту возможность таким образом.

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

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

Это просто возможность добавить несколько фиксированных свойств и методов к контексту - и пока все.
Для перехватчика, в основном, это нужно.