Отправлено: Александр Пекшев aka Modis от 26-05-2015, 12:52:44
Для копирования таблицы из другого файла использую метод WblockCloneObjects. Вот примерно так это выглядит:
Код - C# [Выбрать]
- var tableStyle = "bla_bla_bla";
- // Имя файла из которого берем таблицу
- var sourceFileName = Path.Combine(dir, "Tables.dwg");
- // Read the DWG into a side database
- sourceDb.ReadDwgFile(sourceFileName, FileShare.Read, true, string.Empty);
- var tblIds = new ObjectIdCollection();
- // Создаем пустую таблицу
- var tbl = new Table();
- using (var myT = sourceDb.TransactionManager.StartTransaction())
- {
- var sourceBtr = (BlockTableRecord) myT.GetObject(sourceDb.CurrentSpaceId, OpenMode.ForRead, false);
- foreach (var obj in sourceBtr)
- {
- var ent = (Entity)myT.GetObject(obj, OpenMode.ForRead);
- if (ent is Table)
- {
- var tblsty = (Table)myT.GetObject(obj, OpenMode.ForRead);
- if (tblsty.TableStyleName.Equals(tableStyle))
- {
- tblIds.Add(tblsty.ObjectId);
- var im = new IdMapping();
- sourceDb.WblockCloneObjects(tblIds, db.CurrentSpaceId, im, DuplicateRecordCloning.Replace, false);
- tbl = (Table) tr.GetObject(im.Lookup(tblsty.ObjectId).Value, OpenMode.ForWrite);
- break;
- }
- }
- }
- myT.Commit();
- }
Допустим в файле из, которого я копирую, есть текстовый стиль Standard у которого шрифт стоит Arial и степень сжатия 1.0. И этот текстовый стиль применен к таблице. В моем текущем документе тоже есть текстовый стиль Standard, но с другим шрифтом и степенью сжатия (например Simplex.shx и 0.7). Так вот - после работы выше написанного кода текстовый стиль в текущем документе тоже переопределится (шрифт станет Arial, степень сжатия 1.0).
Предположил, что дело в самом методе WblockCloneObjects и попробовал заменить DuplicateRecordCloning.Replace на DuplicateRecordCloning.Ignore. И вроде все сработало (т.е. текстовый стиль не переопределился, а таблица скопировалась), но при попытке дальнейших манипуляций с таблице автокад вылетает с FatalError
Как быть?
Отправлено: Александр Ривилис от 26-05-2015, 12:55:40
2) Попробуй заменить DuplicateRecordCloning.Replace на DuplicateRecordCloning.MangleName
Отправлено: Александр Ривилис от 26-05-2015, 13:00:35
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 13:31:34
P.S.: Вообще-то в этом коде ты клонируешь не таблицу, а табличный стиль.Разве?
Я "беру" таблицу: var tblsty = (Table)myT.GetObject(obj, OpenMode.ForRead);
проверяю, что у нее табличный стиль нужный мне: if (tblsty.TableStyleName.Equals(tableStyle))
и делаю "клон" по ObjectId таблицы: tblIds.Add(tblsty.ObjectId);
Просто имена переменных корявые
Отправлено: Александр Ривилис от 26-05-2015, 13:37:24
Отправлено: Андрей Бушман от 26-05-2015, 13:49:10
Код - C# [Выбрать]
- var tbl = new Table();
- ...
- tbl = (Table) tr.GetObject(im.Lookup(tblsty.ObjectId).Value, OpenMode.ForWrite);
Код - C# [Выбрать]
- var tbl = null;
Получать Entity для каждого ObjectId лишь для того, чтобы проверить тип - это затратно и нецелесообразно. Имхо лучше так:
Код - C# [Выбрать]
- // за рамками цикла
- var targetType = RXClass.GetClass(typeof(Entity));
- ..
- // в цикле
- if (id.ObjectClass.IsDerivedFrom(targetType)){
- }
А это что за ужасы?
Код - C# [Выбрать]
- var ent = (Entity)myT.GetObject(obj, OpenMode.ForRead);
- if (ent is Table)
- {
- var tblsty = (Table)myT.GetObject(obj, OpenMode.ForRead);
А этой строчкой кода ты что хотел сказать?
Код - C# [Выбрать]
- tbl = (Table) tr.GetObject(im.Lookup(tblsty.ObjectId).Value, OpenMode.ForWrite);
Код - C# [Выбрать]
- if (tblsty.TableStyleName.Equals(tableStyle))
Цитировать
Просто имена переменных корявыеКорявые имена способствуют соответствующему восприятию, путанице и появлению ошибок.
Честно говоря, мне даже "больно" читать твой код... :(
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 14:06:44
Код - C# [Выбрать]
- var doc = AcApp.DocumentManager.MdiActiveDocument;
- var currentDb = doc.Database;
- var tableStyle = "bla_bla_bla";
- // Read the DWG into a side database
- sourceDb.ReadDwgFile(sourceFileName, FileShare.Read, true, string.Empty);
- var tblIds = new ObjectIdCollection();
- // Создаем пустую таблицу
- Table tbl = null;
- using (var sourceTr = sourceDb.TransactionManager.StartTransaction())
- {
- var sourceBtr = (BlockTableRecord)sourceTr.GetObject(sourceDb.CurrentSpaceId, OpenMode.ForRead, false);
- foreach (var objectId in sourceBtr)
- {
- if (sourceTr.GetObject(objectId, OpenMode.ForRead).GetType() == typeof(Table))
- {
- var sourceTable = (Table)sourceTr.GetObject(objectId, OpenMode.ForRead);
- if (sourceTable.TableStyleName.Equals(tableStyle)) // Если стиль таблицы совпадает - это искомая таблица
- {
- tblIds.Add(sourceTable.ObjectId);
- var im = new IdMapping();
- sourceDb.WblockCloneObjects(tblIds, currentDb.CurrentSpaceId, im, DuplicateRecordCloning.Replace, false);
- tbl = (Table)tr.GetObject(im.Lookup(sourceTable.ObjectId).Value, OpenMode.ForWrite);
- break;
- }
- }
- }
- sourceTr.Commit();
- }
- sourceDb.Dispose();
- if(tbl == null) return;
- // Перемещаем
- var entJig = new TableJig(tbl, pointAligin, tbl.Width, tbl.Height);
- var pr = ed.Drag(entJig);
- if (pr.Status != PromptStatus.OK) return;
- doc.TransactionManager.QueueForGraphicsFlush();
- // Дальше идут другие действия с таблицей...
Обычные шаги:
1. Открыл базу данных документа
2. Нашел подходящий объект
3. Скопировал в текущую базу данных
Естественно таблица копируется с табличным стилем. А он за собой тянет текстовый стиль, который в свою очередь перезаписывается в текущем документе (если есть такой-же)
Отправлено: Андрей Бушман от 26-05-2015, 14:08:22
Отправлено: Андрей Бушман от 26-05-2015, 14:12:31
Этот кода - самый, что ни есть, стандартный из всех возможных примеров, что можно найти.Допущенные тобой ошибки - твои личные. По обозначенным ссылкам они отсутствуют, так что не нужно пенять на других.
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 14:16:55
Ты пробовал сделать то, что тебе советовали (http://adn-cis.org/forum/index.php?topic=2736.msg9589#msg9589) (п.2)?Да. Если выставить DuplicateRecordCloning.MangleName, то стили (табличный и текстовый) копируются в текущий документ с приставками к имени символов $0$. Конечно текстовый стиль не перезаписывается, но при каждом использовании кода текущие списки стилей будут, простите за мой французский, засераться новыми копиями стилей
Допущенные мной ошибки ни коем случаем не являются причиной вопроса, описанного в самом начале)))Этот кода - самый, что ни есть, стандартный из всех возможных примеров, что можно найти.Допущенные тобой ошибки - твои личные. По обозначенным ссылкам они отсутствуют, так что не нужно пенять на других.
Отправлено: Андрей Бушман от 26-05-2015, 14:28:27
Цитировать
If a duplicate is found, ignore the clone, and continue to use the existing record in the destination database. This is how the INSERT command and Database.insert() operate.Или DuplicateRecordCloning.UnmangleName:
Цитировать
Primarily used by RefEdit when it checks records back into the origin database. Any mangling done by MangleName is undone, and then it defaults to Ignore. In other words, if a duplicate is found after the mangling is removed, AutoCAD continues to use the existing record, and ignores the cloned one.Их описание в справке не совсем ясно для меня, поэтому лучше проверить эти варианты на примере.
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 14:51:29
Попробуй вариант DuplicateRecordCloning.Ignore:Вот тут и начинается самое интересное)ЦитироватьIf a duplicate is found, ignore the clone, and continue to use the existing record in the destination database. This is how the INSERT command and Database.insert() operate.Или DuplicateRecordCloning.UnmangleName:ЦитироватьPrimarily used by RefEdit when it checks records back into the origin database. Any mangling done by MangleName is undone, and then it defaults to Ignore. In other words, if a duplicate is found after the mangling is removed, AutoCAD continues to use the existing record, and ignores the cloned one.Их описание в справке не совсем ясно для меня, поэтому лучше проверить эти варианты на примере.
Сразу говорю - да, я согласен заранее, что код может быть "печальным"
Дальше по коду при действиях с этой таблицей есть динамическая вставка строк.
Запускается так (этот код идет прям сразу после приведенного выше)
Код - C# [Выбрать]
- // Динамическая вставка строк
- if (this.ChkDynRowsStandard.IsChecked.Value)
- {
- var jig = new TableAddCellsJig
- {
- FPt = tbl.Position,
- RowH = double.Parse(this.TbRowHeight.Text) * Scale(this.CbScales.SelectedItem.ToString()),
- StopRows = int.Parse(xmltbl.Attribute("DataRow").Value),
- TbH = tbl.GeometricExtents.MaxPoint.Y - tbl.GeometricExtents.MinPoint.Y
- };
- jig.StartJig(tbl);
- }
Извините, вам запрещён просмотр содержимого спойлеров.
Дело в том, что в чистом чертеже все работает. Но если я пытаюсь запустить код в существующем чертеже, то автокад вылетает с FatalError. Причем вот именно в тот момент, когда пытается сработать Jig
Отправлено: Александр Ривилис от 26-05-2015, 15:11:43
1) Использовать DuplicateRecordCloning.Replace и тогда заменяются стили в чертеже-приёмнике.
2) Использовать DuplicateRecordCloning.Ignore и тогда используются стили, которые есть в чертеже-приёмнике. Кстати, это не только текстовый стиль, но и табличный стиль. То, что у тебя при этом возникает Fatal Error мы пока рассматривать не будем.
3) Использовать DuplicateRecordCloning.MangleName и тогда вопросов нет - свои текстовый и табличные стили.
Так какой вариант тебе нужен?
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 15:17:08
Если резюмировать, то у тебя есть такие варианты:Естественно идеальный вариант п.2. Кстати DuplicateRecordCloning.UnmangleName с виду делает почти тоже самое, что и DuplicateRecordCloning.Ignore
1) Использовать DuplicateRecordCloning.Replace и тогда заменяются стили в чертеже-приёмнике.
2) Использовать DuplicateRecordCloning.Ignore и тогда используются стили, которые есть в чертеже-приёмнике. Кстати, это не только текстовый стиль, но и табличный стиль. То, что у тебя при этом возникает Fatal Error мы пока рассматривать не будем.
3) Использовать DuplicateRecordCloning.MangleName и тогда вопросов нет - свои текстовый и табличные стили.
Так какой вариант тебе нужен?
Остается вопрос - почему Fatal Error?
Пытался отладкой найти место где именно случается ошибка, но у меня какие-то проблемы - при отладке автокад открывается вообще без текстовых стилей
Вот я и пошел на форум - возможно проблема очевидна, но я ее не вижу
Отправлено: Александр Ривилис от 26-05-2015, 15:26:48
Код - C# [Выбрать]
- sourceDb.WblockCloneObjects(tblIds, currentDb.CurrentSpaceId, im, DuplicateRecordCloning.Ignore, false);
Код - C# [Выбрать]
- currentDb.WblockCloneObjects(tblIds, currentDb.CurrentSpaceId, im, DuplicateRecordCloning.Ignore, false);
Отправлено: Александр Ривилис от 26-05-2015, 15:28:37
Естественно идеальный вариант п.2.Для меня этот вариант совсем не выглядит идеальным, т.к. при копировании таблицы я бы не хотел, чтобы у меня все элементы ячеек выглядели бы не так, как в исходном чертеже.
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 15:43:43
Как оказалось - совсем в другом месте нужно было искать))
Одно из действий с таблицей было ее масштабирование. Этого куска кода я даже не стал писать, т.к. не подумал на него
После кучи тестов я заметил, что fatal error выпадает только при динамической вставке строк в таблицу, у которой был изменен масштаб
после замены этого
Код - C# [Выбрать]
- var mat = Matrix3d.Scaling(Scale(this.CbScales.SelectedItem.ToString()), tbl.Position);
- tbl.TransformBy(mat);
Код - C# [Выбрать]
- tbl.UpgradeOpen();
- var mat = Matrix3d.Scaling(Scale(this.CbScales.SelectedItem.ToString()), tbl.Position);
- tbl.TransformBy(mat);
- tbl.DowngradeOpen();
В данном случае это все обусловлено тем, что файл из которого копируется таблица - это просто как исходник. А таблица как раз не должна по содержимому получится как в исходникеЕстественно идеальный вариант п.2.Для меня этот вариант совсем не выглядит идеальным, т.к. при копировании таблицы я бы не хотел, чтобы у меня все элементы ячеек выглядели бы не так, как в исходном чертеже.
Отправлено: Александр Ривилис от 26-05-2015, 15:48:51
проблема исчезлаНесколько странно, т.к. в этом случае должен был быть не Fatal Error, а Exception с кодом NotOpenForWrite
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 15:53:58
Проверил на 2010, 2013 и 2016 автокаде - работает)проблема исчезлаНесколько странно, т.к. в этом случае должен был быть не Fatal Error, а Exception с кодом NotOpenForWrite
Отправлено: Александр Ривилис от 26-05-2015, 17:10:42
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 17:21:45
А каким местом на это влияло Replace и Ignore? Или при Replace тоже были Fatal Error, но ты на них не попадал?В том-то и юмор - при Replace все работало без проблем. Просто я столкнулся с проблемой переопределения текстового стиля и начал "копать" причину
Отправлено: Андрей Бушман от 26-05-2015, 17:27:57
modplus.orgСимпатично сайт заделал, молодец! :) Правда "очепяток" в тексте многовато...
Отправлено: Александр Ривилис от 26-05-2015, 17:32:50
Правда "очепяток" в тексте многовато...
Off-Topic: показать
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 17:37:19
Ну если сидеть и ждать, когда получится идеальная программа, чтобы поделиться ее с пользователем, то так и будешь сидеть... :-)Правда "очепяток" в тексте многовато...Off-Topic: показать
А так - модуль растет, развивается и уже получает своих поклонников
Отправлено: Андрей Бушман от 26-05-2015, 17:40:22
Похоже и в программе тоже.Имхо: если серьёзно за дело возьмётся + освоит модульное тестирование для своего кода, то софт получится хороший (за который деньги платить не жалко). С хорошими идеями у Модиса проблем нет. У него, как я заметил, в основном качество реализации хромает (к сожалению). Но повторюсь - если он со временем всё же сможет сделать акцент на качестве, то это будет "бомба". :)
Отправлено: Александр Ривилис от 26-05-2015, 17:42:04
С хорошими идеями у Модиса проблем нетВот за это я ему плюсик и поставил.
Отправлено: Андрей Бушман от 26-05-2015, 17:47:16
Ну если сидеть и ждать, когда получится идеальная программа, чтобы поделиться ее с пользователем, то так и будешь сидеть... :-)Есть и обратная сторона медали: пользователь может так обжечься о качестве реализации того или иного продукта, что его потом и калачом не заманишь на повторную попытку использования (даже если очередная версия имеет намного более качественный код). Это тот случай, когда "ложечки нашлись, а осадочек остался". Психология, так сказать. Каждый разработчик для себя сам решает, на чём делать акцент: на фантиках, на скорости разработки или же на качестве. Подобрать оптимальное соотношение этих параметров - искусство (имхо).
Отправлено: Александр Пекшев aka Modis от 26-05-2015, 18:21:14
Ну, Андрей, тут все понятно и логично. У программы всегда будут постоянные пользователи, всегда будут те, кто обжегся, кому просто не понравилась или кому вообще не нужна) Это оправданный риск. К тому-же сейчас у модуля нет никакого пиара, кроме раздачи на рутрекере, потому-что есть еще места, которые нужно улучшатьНу если сидеть и ждать, когда получится идеальная программа, чтобы поделиться ее с пользователем, то так и будешь сидеть... :-)Есть и обратная сторона медали: пользователь может так обжечься о качестве реализации того или иного продукта, что его потом и калачом не заманишь на повторную попытку использования (даже если очередная версия имеет намного более качественный код). Это тот случай, когда "ложечки нашлись, а осадочек остался". Психология, так сказать. Каждый разработчик для себя сам решает, на чём делать акцент: на фантиках, на скорости разработки или же на качестве. Подобрать оптимальное соотношение этих параметров - искусство (имхо).
Ну и по поводу улучшения - я не знаю ни одной программы, которая не выпускает обновлений. И всегда в этих обновлениях есть исправление ошибок. То же можно сказать и об операционных системах, да и о самом автокаде. Например сейчас у меня стоит Windows 10 и естественно в ней куча багов, но это не значит, что я не буду ей пользоваться. Однако до сих пор есть люди, которые кричат, что лучше всего была Windows XP) Всем не угодишь
Отправлено: Дима_ от 26-05-2015, 22:44:20
Off-Topic: показать
Я еще один раз убедился, что лучший метод разобраться с ошибкой - это написать "хелло ворд" с ней.
Отправлено: Александр Пекшев aka Modis от 27-05-2015, 00:25:20
Лучший метод - отвлечься на другую проблему ) Или попробовать объяснить проблему кому-нибудь. Обычно в процессе объяснения сам понимаешь в чем делоOff-Topic: показать
Я еще один раз убедился, что лучший метод разобраться с ошибкой - это написать "хелло ворд" с ней.
Отправлено: Андрей Бушман от 27-05-2015, 10:53:14
лучший метод
Лучший методЛучший метод - TDD (https://ru.wikipedia.org/wiki/%D0%A0%D0%B0%D0%B7%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D0%BA%D0%B0_%D1%87%D0%B5%D1%80%D0%B5%D0%B7_%D1%82%D0%B5%D1%81%D1%82%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D0%B5) (имхо). В противном случае* ошибки чаще будет обнаруживаться не разработчиком, но пользователями его ПО, что существенно будет понижать карму как ПО, так и разработчика (имхо).
* - что, впрочем, зависит так же от внимательности и степени концентрации разработчика. Например, А.Н. Ривилис тесты не пишет, но качество его ПО вполне приемлемое (ошибки какие-то, помню выявлялись, но не существенные и не частые).
Отправлено: Дима_ от 27-05-2015, 21:01:47
з.ы. но по сути все "вылетающие" ошибки - это ерунда - не приятно конечно, но не смертельно - гораздо хуже когда программа что-то не так насчитает (ошибка в логике) - вот здесь могут быть сильные неприятности - тесты на данный вид ошибок "обязательная" процедура - причем желательно на объеме проверенных данных побольше (хорошо если эти данные есть) - конечно тоже не гарантирует, что ошибок не будет - но спится гораздо спокойней - когда по твоему "свежему расчетнику" объект на десяток другой "лямов" посчтитали.
Отправлено: Андрей Бушман от 27-05-2015, 21:10:26
Я писал про "разобраться с ошибкой" - то есть "вылечить" ее, а не способ найти.А что, ошибки можно "лечить" не находя их? :)
Отправлено: Александр Ривилис от 27-05-2015, 21:10:42
И не понимаю тех формучан (а этим грешат и достаточно опытные) которые выкладывают кода на строк эдак 800 и говорят - у меня тут где-то "фаталит" - то есть - я тут "панописал" - но мне самому разбираться лень - а Вам все равно делать нечего.Браво! Я каждый раз "намекаю", что если хотите, чтобы помогли с поиском ошибки и/или отправкой бага в Autodesk - минимизируйте код, создайте отдельный проект, которого достаточно для воспроизведения ошибки. Увы, но не всегда "намёк" воспринимается. Создаётся впечатление, что создать новый проект - это самое сложная процедура...
Отправлено: Александр Пекшев aka Modis от 27-05-2015, 21:23:44
Ошибки есть и будут всегда - код пишет человек, который может и отвлечься и устать, и прочее. И очень часто бывает, что "смотришь в книгу - видишь фигу" и это никоем случаем не зависит от твоего уровня умений и знаний - просто так бывает
И спросить кого-то - это всегда только плюс. Когда начинаешь обсуждать вопрос - бывает сам и "доходишь" до проблемы
И зачем делать форум и постоянно на нем отвечать, если этого не хочешь. А если хочешь - то зачем "направлять" людей на то, чтобы они "не ленились, делали новые проекты и сами находили ошибки"? Тогда все станут такими молодцами, что и форум не нужен будет) И общение)
И вообще - вопрос, заданный в топике, нашел решение! Закрывайте тему и хватит тут флудить! Для этого обычно есть специальные ветки!
Отправлено: Александр Ривилис от 27-05-2015, 21:51:11
Ошибки есть и будут всегда - код пишет человек, который может и отвлечься и устать, и прочее.Согласен.
И спросить кого-то - это всегда только плюс.Тоже согласен.
А если хочешь - то зачем "направлять" людей на то, чтобы они "не ленились, делали новые проекты и сами находили ошибки"?А вот тут ты не прав. Во-первых, лично у меня вызывает отторжение и нежелание помогать человеку, которому лень пошевелится для решения собственной проблемы. Хочешь чтобы тебе помогли? Пожалуйста. Но и ты не ленись принять в этом участие. В данном конкретном случае я думал, что ты проверил все варианты и вышел на баг в AutoCAD .NET API. А если это баг, то есть и достаточно четкие и понятные правила для оформления его и отсылки в Autodesk, которые нужно соблюсти. В первую очередь это проект с максимально простым кодом, который этот баг воспроизводит. Вот об этом я не устаю повторять. ;)
И вообще - вопрос, заданный в топике, нашел решение! Закрывайте тему и хватит тут флудить! Для этого обычно есть специальные ветки!Не нужно учить администратора тому, как управлять форумом. А то заставлю учить правила и сдавать экзамен! :D
Отправлено: Дима_ от 27-05-2015, 21:52:32
Создаётся впечатление, что создать новый проект - это самое сложная процедура...Которая не редко еще и показывает что ошибка "не здесь".
То Андрей - можно видеть что ошибка есть (например фатал - он немножко заметен) но не знать как с ней бороться и собственно где она.