Перебор всех объектов

[Дмитрий Загорулькин]

Моя СУБДшная натура категорически против отказа от транзакций, там где они возможны.

Я так понимаю, что это не те транзакции, что в СУБД. Общее у них только название. А по факту, получился инструмент, который на начальных этапах освоения API сильно упрощает жизнь, а в дальнейшем - может её сильно испортить.

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

В том-то и дело. Я так понимаю, что есть некоторые вспомогательные методы, которые выполняют обработку переданных данных. Пока они вызываются из командных методов, всё работает вроде как надо. Но впоследствии эти же методы возможно понадобится вызывать из различных событий, из методов Overrule и тд и тп, то есть, в тех случаях, когда транзакция может испортить работу AutoCAD, приводя к Fatal Error. Отлавливание таких ошибок - очень непростое дело.
Я для себя пришёл к такому решению: если пишется "библиотечный" метод, лучше, чтобы в нём не было транзакций. Либо передаю уже открытый объект(на чтение, на запись), либо открываю внутри метода объект с помощью ObjectId.Open или OpenCloseTransaction.GetObject.

[avc]

это не те транзакции, что в СУБД

Почему же не те - принцип как раз тот: изоляция изменений до коммита транзакции, откат всех изменений целиком. На ранних этапах (да и теперь) они конечно сильно УСЛОЖНЯЮТ жизнь, удлиняют код. В сравнении со всякими VBA. Но повышают дуракоустойчивость кода. В обработчиках событий транзакции у меня живут прекрасно и им там самое место. ИМХО. Нельзя сказать наверняка, но похоже больше фаталов от смешивания подходов - часть методов безтранзакционные, часть используют toptransaction не явно и влетают на чужие транзакции, которые внезапно комитятся, а открытые объекты остаются в вашем коде... и гуд-бай Автокад... API позволяет намешать такой винегрет, что мама не горюй. А примеры, понадерганные из форумов, этому только способствуют. И, как тут только что выяснилось, примеры с принципиальными ошибками пишут даже гуру. 

[Дмитрий Загорулькин]

Как один из недавних примеров: http://adn-cis.org/forum/index.php?topic=7829.0

Казалось бы, никакой проблемы - используем транзакцию и радуемся. Но! При обработке больших чертежей, в какой-то случайный момент вываливается исключение System.AccessViolationException без указания места, в котором это исключение возникло. Без транзакции исключений не возникает

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

[avc]

Вроде как раз примеры того, как не надо смешивать транзакции и доступ к БД без транзакций. К тем же записям! Уж если где-то получаете объекты без транзакций, то делайте так везде и будет все хорошо. Если, конечно, справитесь с тем, чтоб все вовремя даунгрейдить и диспозить. Другое дело, что никто так не делает, у всех в коде каша. Тотально у всех, у меня тоже. А потом делается вывод, что это транзакции виноваты. Не верю! Если б MS SQL такое позволял, он бы тоже падал когда попало как Автокад.

[Дмитрий Загорулькин]

Не верю!

Ну ок. Я поделился своим опытом. Перенимать его или нет - дело хозяйское. 

[avc]

Не могу не согласиться. Случаи бывают разные. А истина всегда ускользает. Продолжаем оптимизировать код.... Каждый по своему....

[simson43]

Попробуйте использовать вот этот код:
Код - C# [Выбрать]
if (objId.ObjectClass.IsDerivedFrom(RXObject.GetClass(typeof(MText))))

Проблема!
внешняя ссылка для него это блок и он залазит в нее (если я ищу блоки).

[Александр Ривилис]

Проблема!
внешняя ссылка для него это блок и он залазит в нее (если я ищу блоки).

Кто куда залазит???

[Владимир Шу]

Проблема!
внешняя ссылка для него это блок и он залазит в нее (если я ищу блоки).

Почему проблема? Внешняя ссылка и есть блок, внутри автокада очень многое можно привести к блоку (таблицы, размеры, выноски и т.д.).
Если Вам не нужны XRef, то и фильтруйте их: BlockTableRecord.IsFromExternalReference

Все еще не понимаю где проблема...

[simson43]

Я просто не знал что так много всего подходит под определение блока. Придется фильтровать все что не нужно.
Спасибо

[simson43]

Если Вам не нужны XRef, то и фильтруйте их: BlockTableRecord.IsFromExternalReference

т е для проверки придется открыть блок через транзакцию и дополнительно получать так же через транзакцию BlockTableRecord полученного блока?

[Александр Ривилис]

Если Вам не нужны XRef, то и фильтруйте их: BlockTableRecord.IsFromExternalReference

т е для проверки придется открыть блок через транзакцию и дополнительно получать так же через транзакцию BlockTableRecord полученного блока?

Угу.

[Владимир Шу]

т е для проверки придется открыть блок через транзакцию и дополнительно получать так же через транзакцию BlockTableRecord полученного блока?

Можно и так, но ИМХО, проще написать метод расширения для ObjectId и проверять сразу в условии.

Что то вроде:

Код - C# [Выбрать]

1.     public static class BlockReferenceHelper
2.     {
3.         public static bool IsXRef(this Db.ObjectId id)
4.         {
5.             bool ret = false;
6.             if (id.ObjectClass.IsDerivedFrom(Rtm.RXClass.GetClass(typeof(Db.BlockReference))))
7.             {
8.                 Db.BlockReference br = id.Open(Db.OpenMode.ForRead) as Db.BlockReference;
9.                 Db.BlockTableRecord btr = br.BlockTableRecord.Open(Db.OpenMode.ForRead) as Db.BlockTableRecord;
10.                 ret = btr.IsFromExternalReference;
11.  
12.                 br.Close();
13.                 br.Dispose();
14.  
15.                 btr.Close();
16.                 btr.Dispose();
17.             }
18.             return ret;
19.         }
20.     }

И далее уже в своем коде:

Код - C# [Выбрать]

1. if (Id.IsXRef())
2. {
3.  // тут какой то свой код
4. }

PS.
За код, особо сильно не пинайте, "чукча не писатель, чукча читатель"

[Александр Ривилис]

PS.
За код, особо сильно не пинайте, "чукча не писатель, чукча читатель"

Всё-таки слегка "пну" по поводу Close и Dispose. Если !ObjectId.IsNull (а это наш случай), то Dispose вызывает Close. Так что вызывать и Close и Dispose не следует. Если ObjectId.IsNull то Close вызвать нельзя.
Лучше всего в этом коде использовать using:

Код - C# [Выбрать]

1. using (Db.BlockReference br = id.Open(Db.OpenMode.ForRead) as Db.BlockReference)
2. using (Db.BlockTableRecord btr = br.BlockTableRecord.Open(Db.OpenMode.ForRead) as Db.BlockTableRecord)
3. {
4.     ret = btr.IsFromExternalReference;
5. }