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

[simson43]

Добрый день!

выполняю перебор всех объектов чертежа. в чертеже есть внешние ссылки.
когда она проверяется и натыкается на эту строчку

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

1. if (myTransaction.GetObject(objId, OpenMode.ForWrite) is MText mt)

выскакивает ошибка
видимо потому что ее нельзя получить внешнюю ссылку для записи. получать два раза как то не хочется: для проверки а потом для изменения
как быть?
Спасибо

[avc]

Открывать все подряд объекты на запись - плохая практика. Если есть сомнения, что объект надо будет редактировать, то открывайте на чтение. А прямо перед модификацией вызывайте UpgradeOpen

[simson43]

ну да логично.
Спасибо.

[simson43]

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

[avc]

да

[simson43]

Спасибо большое!

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

simson43, если я правильно понял, то Ваш код проверяет, методом открытия и приведения объекта, является ли он MText
Попробуйте использовать вот этот код:

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

1. if (objId.ObjectClass.IsDerivedFrom(RXObject.GetClass(typeof(MText))))

он не открывает объект (если я правильно помню) и работает несколько быстрей, да и транзакция не нужна...

[avc]

Владимир Шу, cудя по тому что оператор is объявляет переменную mt - я бы предположил, что она где-то дальше используется. Вот если не используется, тогда да - однозначно надо проверять тип объекта по ObjectId.

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

avc, конечно используется, иначе какой смысл создавать эту перемененную.
Я просто имел ввиду, что более правильно сначала определить класс объекта, а потом, если он подходит, его открывать...

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

1. if (objId.ObjectClass.IsDerivedFrom(RXObject.GetClass(typeof(MText))))
2. {
3.   MText mt = myTransaction.GetObject(objId, OpenMode.ForWrite) as MText;
4.   //и далее какой-то код
5.  }

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

Открывать все подряд объекты на запись - плохая практика. Если есть сомнения, что объект надо будет редактировать, то открывайте на чтение. А прямо перед модификацией вызывайте UpgradeOpen

Всё так, кроме того, что вместо UpgradeOpen нужно снова вызвать повторно Transaction.GetObject(...), но с OpenMode.ForWrite. UpgradeOpen - это для Open/Close (или OpenCloseTransaction) и может конфликтовать с обычной транзакцией.

[avc]

моджет конфликтовать с обычной транзакцией

Да что вы?! Точно? Я всегда так делаю и в куче примеров видел и у объекта IsWriteEnabled становится true... Неужели я столько лет косячил... шок...

[avc]

Полез на блог Kean Walmsley - во всех примерах он берет объекты через транзакции, и потом вызывает UpgradeOpen. Вот от него я и набрался. Вот например тут http://www.keanw.com/2010/01/creating-an-autocad-block-using-net.html

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

1. BlockTable bt =
2.           (BlockTable)tr.GetObject(
3.             db.BlockTableId,
4.             OpenMode.ForRead
5.           );

и следом

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

1.         bt.UpgradeOpen();
2.         ObjectId btrId = bt.Add(btr);
3.         tr.AddNewlyCreatedDBObject(btr, true);

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

моджет конфликтовать с обычной транзакцией

Да что вы?! Точно? Я всегда так делаю и в куче примеров видел и у объекта IsWriteEnabled становится true... Неужели я столько лет косячил... шок...

Ты был не один. И я тоже такие примеры приводил с использованием UpgradeOpen. Но один старший товарищ (Art Cooney - один из руководителей разработки AutoCAD) объяснил мне, что я не прав. Это вылезло в одном из обновлений AutoCAD 2018. Тут подробности: https://forums.autodesk.com/t5/net/api-bug-2018-1-causes-crash-using-upgradeopen-on-dependent/m-p/7272262/highlight/true#M54471

[simson43]

т е или заменить обычную транзакция на openclosetransaction или еще раз получать объект для записи?

а как же вот с оф сайта пример?
https://knowledge.autodesk.com/search-result/caas/CloudHelp/cloudhelp/2015/ENU/AutoCAD-NET/files/GUID-CC5CC229-B122-4897-A8DA-5C5ADADB0F38-htm.html?_ga=2.51307242.2126543432.1539105496-1602586928.1535484681

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

т е или заменить обычную транзакция на openclosetransaction или еще раз получать объект для записи?

Менять ничего не надо. Просто вместо UpgradeOpen вызываешь снова myTransaction.GetObject, но уже с OpenMode.ForWrite.

а как же вот с оф сайта пример?
https://knowledge.autodesk.com/search-result/caas/CloudHelp/cloudhelp/2015/ENU/AutoCAD-NET/files/GUID-CC5CC229-B122-4897-A8DA-5C5ADADB0F38-htm.html?_ga=2.51307242.2126543432.1539105496-1602586928.1535484681

Он ошибочен. И по той ссылке, которую я дал выше, это обсуждается.