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

[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. И открывай по мере надобности.