Как поймать ошибку?

[Luca]

Здравствуйте.
"Падает" приложение, притом Violation Access, те я даже StackTrace посмотреть не могу...
Как в такой ситуации найти ошибку...
Пробовал отлаживать (точки останова), вроде норм всё, до конца доходит приложение и на выходе падает.
Не соображу в чём дело.

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

Общие рекомендации:
Смотрите окно "Output" ("Вывод") в Visual Studio при отладке. Есть шанс, что там будет полезная информация.
В настройках отладки добавить тип исключения "AccessViolationException" в список исключений, при которых останавливается отладка. Или ставить точку останова в начале команды и выполнять пошагово до тех пор, пока не упадёт. Это, как минимум, поможет понять в каком месте возникает исключение.

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

Luca,
На этом форуме не гадают на кофейной гуще - нужен исходный код.

[trir]

обычно это обращение к null-объекту

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

обычно это обращение к null-объекту

Или к объекту, который открыт в транзакции, за пределами этой транзакции... Тут вариантов множество...

[Luca]

Автокад падает при команде  AUTO_ELECTRO_NUMBER_BLOCKS_
Решение разделено на части:
Общие вещи собраны в пространстве Common

[Luca]

обычно это обращение к null-объекту

из документации следует, что это грубая ошибка, сродни NullException, но только обращение за границы, а не к Null

[trir]

нельзя так делать

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

1.             public static List<BlockReference> getListOfBlocks(SelectionSet selectionSet, string layerName)
2.             {
3.                 List<BlockReference> listOfBlocks = new List<BlockReference>();
4.                 Transaction transaction = startTransaction();
5.                 using (transaction)
6.                 {
7.                     foreach (SelectedObject selectionSetObject in selectionSet)
8.                     {
9.                         if (selectionSetObject != null)
10.                         {
11.                             Entity entity = transaction.GetObject(selectionSetObject.ObjectId, OpenMode.ForRead) as Entity;
12.                             if (entity is BlockReference block)
13.                             {
14.                                 if (block.Layer == layerName)
15.                                 {
16.                                     listOfBlocks.Add(block);
17.                                 }
18.                             }
19.                         }
20.                     }
21.                     transaction.Commit();
22.                 }
23.                 if (listOfBlocks.Count != 0)
24.                 {
25.                     return listOfBlocks;
26.                 }
27.                 else
28.                 {
29.                     Application.ShowAlertDialog("[listOfBlocks:] НЕТ БЛОКОВ. КОЛИЧЕСТВО БЛОКОВ = " + listOfBlocks.Count);
30.                     return null;
31.                 }
32.             }

Посмотри как у меня реализовано - функции CrTbl и GetSelect https://github.com/triroakenshield/tblPrs/blob/master/TableParser/MyTable.vb

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

нельзя так делать

Именно то, о чем я и писал выше.

[Luca]

обычно это обращение к null-объекту

Или к объекту, который открыт в транзакции, за пределами этой транзакции... Тут вариантов множество...

Так и есть!
У меня было ощущения, судя по отладке. что с транзакцией что-то не то, но понять что именно я не мог.
Как только Вы сказали, то я сразу нашел transaction.Commit, который не в тех операторных скобках.
Ох, спасибо...

[Luca]

нельзя так делать

Именно то, о чем я и писал выше.

да, я уже понял ... нашел. Спасибо.

[trir]

то я сразу нашел transaction.Commit, который не в тех операторных скобках.

он то как раз там где надо

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

Как только Вы сказали, то я сразу нашел transaction.Commit, который не в тех операторных скобках.

У тебя в Common.cs  полный бред, который даже сложно комментировать.
Например,

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

1.             public static BlockTableRecord getBlockByNameFromDocument(string blockName)
2.             {
3.                 Transaction transaction = startTransaction();
4.  
5.                 BlockTable blockTable = (BlockTable)transaction.GetObject(database.BlockTableId, OpenMode.ForRead);
6.                 foreach (ObjectId btrID in blockTable)
7.                 {
8.                     BlockTableRecord btr = (BlockTableRecord)transaction.GetObject(btrID, OpenMode.ForWrite);
9.                     if (btr.Name.ToUpper() == blockName.ToUpper())
10.                     {
11.                         return btr;
12.                     }
13.                 }
14.                 return null;
15.             }

1. Запускается транзакция, но нет ни transaction.Dispose() ни transaction.Commit()
2. Из метода возвращается BlockTableRecord, открытый в транзакции.

Т.е. нужно переделывать всё логику методов.

[Luca]

Как только Вы сказали, то я сразу нашел transaction.Commit, который не в тех операторных скобках.

У тебя в Common.cs  полный бред, который даже сложно комментировать.
Например,

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

1.             public static BlockTableRecord getBlockByNameFromDocument(string blockName)
2.             {
3.                 Transaction transaction = startTransaction();
4.  
5.                 BlockTable blockTable = (BlockTable)transaction.GetObject(database.BlockTableId, OpenMode.ForRead);
6.                 foreach (ObjectId btrID in blockTable)
7.                 {
8.                     BlockTableRecord btr = (BlockTableRecord)transaction.GetObject(btrID, OpenMode.ForWrite);
9.                     if (btr.Name.ToUpper() == blockName.ToUpper())
10.                     {
11.                         return btr;
12.                     }
13.                 }
14.                 return null;
15.             }

1. Запускается транзакция, но нет ни transaction.Dispose() ни transaction.Commit()
2. Из метода возвращается BlockTableRecord, открытый в транзакции.

Т.е. нужно переделывать всё логику методов.

метод  getBlockByNameFromDocument пока просто лежит, на будущее, я его не использую.
прям всё и переделывать?
в Common общие для методы и констаны для всех операций, типа получение транзакции, получение предварительного выбора и тд
теперь в следующей команде  AUTO_ELECTRO_CREATE_CABLE_LOG_  тоже Violation Access

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

теперь в следующей команде  AUTO_ELECTRO_CREATE_CABLE_LOG_  тоже Violation Access

Никогда не храни открытые объекты/примитивы - храни их ObjectId. И открывай по мере надобности.

[Luca]

У тебя в Common.cs  полный бред, который даже сложно комментировать.

обсуждалось здесь
https://adn-cis.org/forum/index.php?topic=9705.0

нужно переделывать всё логику методов.

зачем всю логику переделывать всех методов?

[trir]

обсуждалось здесь

ага

для получения ObjectId того BlockTableRecord

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

зачем всю логику переделывать всех методов?

Я не знаю как мне еще нужно объяснять про открытые примитивы/объекты вне транзакций, а у тебя это на каждом шагу. Не должно быть глобальных List<Polyline>, List<BlockReference> и т.д.

[Luca]

Никогда не храни открытые объекты/примитивы - храни их ObjectId. И открывай по мере надобности.

не совсем понял о чем речь

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

не совсем понял о чем речь

Ты хранишь между запусками команд вот такое: List<BlockReference>, List<Polyline> - этого делать категорически нельзя. Вместо этого храни List<ObjectId> для вставок блоков и полилиний и если нужны их данные - открывай их по мере надобности.

[trir]

этого делать категорически нельзя

а я делаю 

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

1.         public void DrawLayer(Data.DataLayer wl)
2.         {
3.             //List<Entity> wEntlist = new List<Entity>();
4.             Autodesk.Gis.Map.ObjectData.Table wTbl = null;
5.             if (wl.HasAttributes()) wTbl = CreateODTable(wl._Rule);
6.             ObjectId wid;
7.             Entity wEnt;
8.             Geometry.TGeometry g;
9.             foreach (Data.MyRecord wr in wl.Table)
10.             {
11.                 g = wr.GetGeometry();
12.                 if (g != null)
13.                 {
14.                     wEnt = MakeGeometry(g);
15.                     wid = DrawEntity(wEnt);
16.                     if (wl.HasAttributes()) AddAttr(wid, wr, wTbl);
17.                 }
18.                 //wEntlist.Add(wEnt);
19.             }
20.             //DrawEntity(wEntlist);
21.         }

https://github.com/triroakenshield/RosReestrImpLib/blob/master/RosReestrImp/MyCommands.cs

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

а я делаю 

wEnt у тебя содержится в Database (т.е. ObjectId у него не null) или только в памяти?

[trir]

Только в памяти  , а DrawEntity добавляет его в Database и возвращает ObjectId - с которым дальше и работаю!

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

Только в памяти  , а DrawEntity добавляет его в Database и возвращает ObjectId - с которым дальше и работаю!

Ну так совсем другое дело...

[Luca]

Вместо этого храни List<ObjectId> для вставок блоков и полилиний и если нужны их данные - открывай их по мере надобности.

c этого момента подробнее, пожалуйста... если можно, конечно
что значит открываешь? просто работа очень много и читать исходники, пока, времени просто нет...
не совсем понимаю пока в каких абстрактных типах данных (АТД) всё это хранится, поэтому так и поступаю.
по-идее, как я это понимаю, BlockReference - ссылка ( разыменованный указатель), те указатель, но при обращении к нему получаем АТД. ObjectID, исходя из названия, это какой-то ключ для получения Object из Dictonary, например или какой-то древовидной структуры.
Можно пример, как имея BlockReference получить ObjectID и обратно. ObjectId это ключ для получения BlockReference из какой-то структуры, АТД - что это за структура?
Честно сказать, у меня мозги "заплелись" немного ))

[trir]

Код - vb.net [Выбрать]

1.     Public Shared Function CrTbl(acDoc As MyAcAs.Document) As MyTable
2.         'Создаём таблицу
3.         Dim ed As Editor = acDoc.Editor
4.         Dim objIdArray() As ObjectId = MyTable.GetSelect(ed) 'Получаем от пользователя набор данных для парсинга
5.         If objIdArray IsNot Nothing Then
6.             Dim dbObj As DBObject
7.             Dim wList As New List(Of Line)
8.             Dim wTList As New List(Of DBText)
9.             Dim wMTList As New List(Of MText)
10.             Using tr As Transaction = acDoc.Database.TransactionManager.StartTransaction
11.                 Try
12.                     For Each objId As ObjectId In objIdArray
13.                         dbObj = tr.GetObject(objId, OpenMode.ForRead)
14.                         'Сортируем полученные объекты
15.                         Select Case True
16.                             Case TypeOf dbObj Is Line
17.                                 wList.Add(dbObj)
18.                             Case TypeOf dbObj Is Polyline
19.                                 wList.AddRange(MyTable.PolyToLine(dbObj))
20.                             Case TypeOf dbObj Is DBText
21.                                 wTList.Add(dbObj)
22.                             Case TypeOf dbObj Is MText
23.                                 wMTList.Add(dbObj)
24.                         End Select
25.                     Next
26.                     tr.Commit()
27.                 Catch ex As Exception
28.                     ed.WriteMessage(ex.ToString())
29.                     tr.Abort()
30.                 End Try
31.             End Using
32.             '
33.             Dim wMTbl As MyTable = MyTable.CrTbl(wList)
34.             'Заполняем текстом
35.             For Each wt In wTList
36.                 wMTbl.SetValue(wt)
37.             Next
38.             For Each wmt In wMTList
39.                 wMTbl.SetValue(wmt)
40.             Next
41.             Return wMTbl
42.         Else
43.             Return Nothing
44.         End If
45.     End Function

блин, неправильный пример... но ведь работает

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

Честно сказать, у меня мозги "заплелись" немного ))

Извини, но это единственно что я вынес из твоего сообщения. Не лезь в абстракции. Делай то, что я тебе подсказываю, иначе ты потратишь уйму времени на написание кода, который будет работать через раз и ты будешь создавать постоянно темы, аналогичные этой.
Открытие объекта это или tr.GetObject(id,...) или id.Open(...) - в результате из ObjectId получаем одного из наследников DBObject (например, Entity или BlockTableRecord). Закрытие объекта, открытого через id.Open(...) - это метод Close, а для открытого в транзакции - это или tr.Commit() или tr.Dispose()
Если объект открыт (например, BlockReference) в одной из твоих команд и не закрыт в ней (хранится в static переменной), то это значит, что ни сам AutoCAD, ни другие приложения к нему обратиться уже не смогут.

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

блин, неправильный пример... но ведь работает

Для объектов, открытых ForRead такое иногда работает. Но зачем наступать на грабли???

[Lemieux]

Прошу прощения за некропостинг, но может кто-то объяснит почему автор везде пихает static?

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

Прошу прощения за некропостинг, но может кто-то объяснит почему автор везде пихает static?

1. Это не некропостинг - этой теме всего-то четыре месяца.
2. static бывает достаточно удобен для утилитных функций, которые можно вызвать без создания самого объекта класса.
Но почему автор решил использовать static наверное у автора и нужно спрашивать. Впрочем подозреваю, что с его стороны это просто "копипастинг" готового метода, найденного на просторах интернет.