Спасибо!

varelchik писал(а) 08. Ноября 2007 :: 07:00:

Просьба - оформлять свой код в рамки кода, как положено на форуме
Иначе очень неудобно его читать

varelchik писал(а) 08. Ноября 2007 :: 07:00:

Циклическая ссылка при использовании Перем _Перехватчик;
деструктор данного объекта никогда не будет вызван , т.е. объект никогда не будет удален
Большой УПС - попутал свой же встроенный класс с твоим пользовательским классом
Все ок, ничего не нужно менять

varelchik писал(а) 08. Ноября 2007 :: 07:00:

Лучше вместо этого кода использовать спец. фабричный метод или еще лучше фабричный класс типа ФабрикаСозданияПерехватчиков.
Причины:
1) Наверняка для различных форм (отчеты, справочники, документы, журналы и т.д.) будут использоваться различные события, необходимо различать обработчики этих событий, т.е. плодить
различные классы
2) А также если для различных форм одного вида будут использоваться различные обработчики
И соответственно, в различных конфигурациях будут различные классы-перехватчики

Поэтому, чтобы не менять каждый раз класс ПерехватГМ для каждой конфигурации или конфы, лучше сделать так:
1) Создаем базовый фабричный класс БазоваяФабрикаСозданияПерехватчиков
2) Определяем у него открытый метод ПерехватитьКонтекст(КонтФормы) Экспорт
Метод в классе ничего не делает или выдает исключение или создает пустой перехватчик. Типа этот класс является абстрактным, только для наследования
Основное действие будет происходить в конкретных классах-наследниках
3) для каждой различной конфы пишем код в ГМ

	ФабрикаСозданияПерехватчиков = СоздатьОбъект("ФабрикаСозданияПерехватчиковДляДаннойКонфы"); //твой класс перехвата всех форм
	НовыйПерехватчикГМ = ФабрикаСозданияПерехватчиков.ПерехватитьГМ(Конт); 

[code]
4) а в наследнике ФабрикаСозданияПерехватчиковДляДаннойКонфы в методе ПерехватитьКонтекст пишем код по созданию точных обработчиков для нужных форм в данной конфе

Т.е. ООП в действии
Этот метод хорош тем, что происходит минимальное изменение существующего клиентского кода.

artbear писал(а) 08. Ноября 2007 :: 12:50:


Точно. Просто не следовало мне, в Деструкторе перехватчика ГМ, вызывать Сообщить(), 1С ведь уже закрылась почти, и сообщать ей не куда. Видимо.

varelchik писал(а) 08. Ноября 2007 :: 12:43:

Класс делал из болванки, просто чтобы начал работать. Переменная осталась от поиска причины падежа. Удаление лишнего делаю, когда уже знаю, что не лишнее. Деструктор здесь и в самом деле ни к чему. Если на последок в SQL-базу ни чего записать не захочу...

artbear писал(а) 08. Ноября 2007 :: 13:11:


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

При открытии формы документа вываливает следующее:

Перехватчик::ВыполнитьОригинальноеСобытиеГК - Неверное число параметров у обработчика события <ПриОткрытии> 

Во всех документах - без параметров, а в этом есть параметр "Флаг".
Т.е. для текущего дока надо писать свой перехват событий, с учетом параметров, или можно создать универсальный класс, который бы обрабатывал события как с параметрами, так и без них?
Поискал по форуму. Нашел про конструкторы и ФабрикуОбъектов http://www.1cpp.ru/forum/YaBB.pl?num=1193554475/20#20
Попробую этот метод.

Выложи код, что вызываешь и как вызываешь.