Сообщество программистов Autodesk в СНГ

ADN Club => ObjectARX => Тема начата: Николай Горлов от 20-12-2017, 16:21:27

Название: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 16:21:27
Задача такая: рисовать точки в определенной области. Но перед отрисовкой проверить, есть ли уже в указанных координатах нарисованная точка (из условия - все нужные слои включены). Ну и изначально участок, на котором планируется рисовать много-много :) точек может лежать за пределами видимости экрана.
Вот исходя из этих начальных условий, решить есть ли в указанных координатах точка или нет можно двумя (вот больше не придумал) способами:
1. последовательный перебор всех объектов чертежа в поисках AcDbPoint, ну и если нашли, то проверить координаты.
2. программный выбор рамкой (формируется массив точек вокруг исходной точки и acedSSGet(_T("_cp")... в помощь)

У первого варианта есть один существенный недостаток - чем больше объектов в чертеже, тем больше тратится времени на поиск одной точки (даже если в первый пробег по базе сформировать массив всех известных AcDbPoint, а потом искать чисто в нём)
Второй вариант работает на порядок быстрей (где-то раз в 10-15 на чертеже с большим количеством объектов). НО... не всегда работает, как показала практика. А тестовый пример всегда НЕ работает :)

Итак по поводу тестового примера. Первоначально идет зуммирование рамкой, чтоб вся требуемая область была видна на экране, а потом идет в цикле acedSSGet
Так вот, если зуммирование идет внутри моей команды, причем не важно, вызывается оно через acedCommand или с использованием автокадовского ActiveX, acedSSGet всегда возвращает RTERROR. Если же чертеж отзуммировать как нужно через меню автокада, а потом вызвать мою команду (в которой убрать строчки, отвечающие за зуммирование), то acedSSGet всегда работает правильно.

И вот тут вопрос: это лечится? :) Нужен обязательный зум внутри команды.

PS: тестовый проект прилагается. Тестировал в 2014x64 и в 2017x64. Результат одинаковый. В коде есть оба варианта зума для текстирования
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 16:37:38
Цитата: Николай Горлов от 20-12-2017, 16:21:27
Так вот, если зуммирование идет внутри моей команды, причем не важно, вызывается оно через acedCommand или с использованием автокадовского ActiveX, acedSSGet всегда возвращает RTERROR.
А зуммирование при этом происходит или нет?
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 16:45:20
Цитировать
А зуммирование при этом происходит или нет?
естественно.

Как я тестировал.
вариант 1. вариант с activex (в проекте уже настроен этот вариант)
рисует быстро. зуммирование происходит. под дебагом возврат acedSSGet - RTERROR. команду запускал дважды. получал два набора точек

вариант 2. вариант с acedCommand (в проекте раскоментарить и закоментировать activex вариант)
рисует быстро. зуммирование происходит. под дебагом возврат acedSSGet - RTERROR. команду запускал дважды. получал два набора точек

вариант 3. сохранил чертеж с варианта1 или варианта2, не важно. главное, что область рисования уже отзумирована. удалил оттуда все точки. в проекте закоментировал оба варианта зума. также два запуска команды. рисует медленней (минуты полторы). после двух запусков получил 1 набор точек. под дебагом acedSSGet выдает RTNORM
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 16:51:03
И привязки отключал?
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 16:54:08
"узел" выключен. чертеж новый. сейчас отключил все. не помогло.
я даже пыль с монитора стряхнул, но на результат это не повлияло :)
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 16:55:20
Может тогда сделать однократно _Zoom _All и не мучаться?
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 17:00:21
так а зум и делается однократно перед циклом.
Код - C++ [Выбрать]
  1. acedCommandS(RTSTR, _T("_zoom"), RTSTR, _T("_w"), RTSTR, _T("7367820.0,5376960.0"), RTSTR, _T("7371560.0,5371400.0"), 0);
  2. acedCommandS(RTSTR, _T("_zoom"), RTSTR, _T("_s"), RTSTR, _T("0.7X"), 0);

просто потом пользователю б еще и увидеть, что оно нарисовало. тем более, что в новом чертеже _Zoom _All точно не поможет. а в старом чертеже рисование может проходить не внутри уже нарисованных объектов, а где-то за пределами.
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 17:00:43
Вот еще один вариант для ZoomWindow: http://jprdintprev.autodesk.com/adn/servlet/devnote?siteID=4814862&id=5409301&preview=1&linkID=4900509
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 17:05:39
попробую, но не в зуме проблема, кажись. ведь даже используя вызов автокадовских команд для зуммирования SSGet не работает.
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 17:06:58
Цитата: Николай Горлов от 20-12-2017, 17:05:39
попробую, но не в зуме проблема, кажись. ведь даже используя вызов автокадовских команд для зуммирования SSGet не работает.
Ну а если "_CP" заменить на "_C"?
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 20-12-2017, 17:30:34
Цитировать
Ну а если "_CP" заменить на "_C"?
проверил с "_W" и с "_С"
Код - C++ [Выбрать]
  1. /*AcGePoint3dArray pts1 = SelectCrossingPolygonLEps(pt, eps); // восьмиугольник вокруг точки. точки в UCS
  2. if (!pts1.first().isEqualTo(pts1.last())) pts1.append(pts1.first()); // замыкаем первую область
  3. struct resbuf *pointlist1;
  4. pointlist1 = pointsToResbuf(pts1);
  5. int resFind = acedSSGet(_T("_CP"), pointlist1, NULL, &lookAt, ent1); //RTNORM - не было зумма, RTERROR - был зум внутри команды
  6. acutRelRb(pointlist1); // чистим набор точек. больше не нужен
  7. */
  8.        
  9. ads_point pt1, pt2;
  10. pt1[X] = pt.x - eps; pt1[Y] = pt.x - eps; pt1[Z] = 0.0;
  11. pt2[X] = pt.x + eps; pt2[Y] = pt.x + eps; pt2[Z] = 0.0;
  12. acedSSGet(_T("_C"), pt1, pt2, &lookAt, ent1);
  13.  

работают оба варианта. но работают как-то странно, нет постоянства в заполнении прогресса. С каждой новой итерацией это происходит всё медленней и медленней. и под конец практически засыпает прогресс.
вариант с "_W" на холостом ходу в новом чертеже отрабатывает 3 мин. 18 сек.
вариант с "_С" на холостом ходу в новом чертеже отрабатывает 5 мин. 30 сек.

для сравнения, вариант с "_CP", если убрать вызов зума из моей команды отрабатывает за 1мин. 24сек. и прогресс перемещается с завидным постоянством.
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 17:46:22
Цитата: Николай Горлов от 20-12-2017, 17:30:34
для сравнения, вариант с "_CP", если убрать вызов зума из моей команды отрабатывает за 1мин. 24сек. и прогресс перемещается с завидным постоянством.
Похоже он вообще неправильно работает.
Цитата: Николай Горлов от 20-12-2017, 17:30:34
ads_point pt1, pt2;
pt1[X] = pt.x - eps; pt1[Y] = pt.x - eps; pt1[Z] = 0.0;
pt2[X] = pt.x + eps; pt2[Y] = pt.x + eps; pt2[Z] = 0.0;
acedSSGet(_T("_C"), pt1, pt2, &lookAt, ent1);
А тут у тебя просто ошибка в коде как мне кажется.  Должно быть как-то так:
Код - C++ [Выбрать]
  1. ads_point pt1, pt2;
  2. pt1[X] = pt.x - eps; pt1[Y] = pt.y - eps; pt1[Z] = 0.0;
  3. pt2[X] = pt.x + eps; pt2[Y] = pt.y + eps; pt2[Z] = 0.0;
  4. acedSSGet(_T("_C"), pt1, pt2, &lookAt, ent1);
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 20-12-2017, 17:57:21
Что же касается странностей с прогрессом, тот тут то как раз всё понятно. AutoCAD не успевает обработать свою очередь сообщений. Нужно вызвать вот такую функцию перед очередным вызовом прогрессбара:
Код - C++ [Выбрать]
  1. //////////////////////////////////////////////////////////////////////////
  2. // Эта функция при длительных вычислениях
  3. // позволяет AutoCAD обработать свою очередь сообщений (например, обновить
  4. // содержимое окна редактора, отреагировать на движения мыши и т.д.)
  5. //////////////////////////////////////////////////////////////////////////
  6.  
  7. int OnIdleAcadInternal(void) {
  8.   CWinApp *app = acedGetAcadWinApp();
  9.   // Это главное окно приложения AutoCAD
  10.   CWnd *wnd = app->GetMainWnd ();
  11.   // Создадим свой собственный диспетчерский цикл для обработки
  12.   // сообщений, переданных в AutoCAD. Если появились новые собщения -
  13.   // заставим AutoCAD их обработать.
  14.   MSG msg;  
  15.   // Выкачиваем сообщения из очереди сообщений AutoCAD
  16.   while (::PeekMessage(&msg, wnd->m_hWnd, 0, 0, PM_NOREMOVE))  {
  17.     // Если в очереди сообщений был запрос
  18.     // на завершение AutoCAD - выполним его.
  19.     if (!app->PumpMessage())  {
  20.       ::PostQuitMessage(0);
  21.       break;
  22.     }
  23.   }
  24.  
  25.   LONG lIdle = 0;
  26.   // Позволим MFC выполнить обработку состояния ожидания
  27.   //while (app->OnIdle(lIdle++)&& lIdle < 1000);
  28.   return RSRSLT;
  29. }
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 21-12-2017, 12:50:52
Цитировать
Нужно вызвать вот такую функцию перед очередным вызовом прогрессбара
у меня такая есть, только с наворотами :). оно еще и прерывать цикл умеет по Esc :):):)
Код - C++ [Выбрать]
  1. int acad_abortCycle(const ACHAR * abortString)
  2. {
  3.         int res = 0;
  4.         CWinApp *app = acedGetAcadWinApp();
  5.         CWnd *wnd = app->GetMainWnd();
  6.         MSG msg;
  7.         while (::PeekMessage(&msg, wnd->m_hWnd, 0, 0, PM_NOREMOVE))
  8.         {
  9.                 if (!app->PumpMessage())
  10.                 {
  11.                         ::PostQuitMessage(0);
  12.                         break;
  13.                 }
  14.         }
  15.         LONG lIdle = 0;
  16.         while (app->OnIdle(lIdle++));
  17.  
  18.         res = acedUsrBrk();
  19.         if (res == 1 && abortString != NULL) acutPrintf(abortString);
  20.         return res;
  21. }
  22.  
а пользоваться так:
Код - C++ [Выбрать]
  1. if (acad_abortCycle(_T("\nИНФО: Отрисовка прервана пользователем...")) == 1)
  2. {
  3.         acedRestoreStatusBar();
  4.         return;
  5. }
  6.  

Цитировать
А тут у тебя просто ошибка в коде как мне кажется
угу, действительно недоглядел.  итак, провел сравнительный анализ. вот табличка
(https://s9.postimg.org/hgbljt55n/image.png) (https://postimg.org/image/hgbljt55n/)

из нее видно, что в новом чертеже автокада acedSSGet  с _С, _CP, _W возвращает RTERROR и время работы для _W и _C увеличивается в разы. но если взять старый чертеж, то варианты _W и _C работают отлично.

из таблички: там где тестировался вариант без зума, значит, что требуемая область уже была видимой на экране.
еще в табличке есть "*". ситуация с ней такая. если в чертеже провести ряд махинаций перед вторым вызовом команды (что-нибудь нарисовать, дважды клацнуть колесиком мышки, командами автокада поменять область видимости... в общем закономерности я не нашел), то команда может сработать, а может и НЕ сработать. Всё зависит от настроения :)

Да, и время работы у всех приблизительно одинаковое. "_CP", как мне кажется, работает чуток дольше чисто из-за того, что приходится строить восьмиугольник, перегонять его в resbuf, а потом еще и чиститься за собой.
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 21-12-2017, 15:35:25
хм... новая серия тестов. уже только с вариантом "_С"
слегка модернизировал команду. если за проход что-то находит, точка рисуется красным, если не находит - черным. ну и вначале команды идет запрос - нужен зум или нет.

итак. вот такие варианты. выводы будут в конце.
1. новый чертеж. средствами автокада создаю отрезок по координатам 7367820.0,5376960.0 и 7371560.0,5371400.0
 запуск команды. зум нужен. работает 6м 10с. обнаружено 124 объекта, т.е. 124 точки нарисовало красным

2. новый чертеж. копирую отрезок в него с сохранением координат. zoom _e. слегка колесиком мышки уменьшаю масштаб отображения и смещаю отрезок вправо. отрезок полностью виден. запуск команды. зум не нужен. время работы - 0м 39с. обнаружено 222 объекта.

3. новый чертеж. копирую отрезок в него с сохранением координат. zoom _e. слегка колесиком мышки уменьшаю масштаб отображения и прячу линию за пределы экрана. запуск команды. зум нужен. время работы - 0м 33с. обнаружено 124 объекта.

итого в сухом остатке. RTERROR - это нормальный ответ функции acedSSGet (чисто моё умозаключение. в хелпе вообще этот вопрос обошли стороной), если она ничего не находит. обнаружить на самом деле должно было 2 объекта, т.к. сканируется рамкой 0.002x0.002 вокруг исходной точки. но, как показала практика - рамку автокад еще и на текущий масштаб отображения перемножил и мои 0.002x0.002 превращаются.... превращаются 0.002x0.002... короче превращаются они где-то в 3x3 или даже 5x5. и такая рамка захватывает уже лишние точки. дальше, почему в варианте 2 получается больше точек. всё просто - масштаб отображения другой. и 0.002x0.002 превратились, например, в 15x15.

непонятно одно. с какого перепуга время в 1 и 3 настолько сильно отличается. неужто животворящий zoom _e способен привести новый чертеж в нормальное состояние. и почему тогда по умолчанию новый чертеж не находится в этом самом нормальном состоянии? и еще не понятно, с какого перепуга входные точки рамки преобразовались таким диким образом, что рамка увеличилась в разы от заданного размера?


-----------------------------------------------------
добавлено.
переварив эти приколы, решил проверить, как ведет себя "_CP"
вариант1: работает 5м 19с, найдено 2 объекта
вариант2: работает 1м 6с, найдено 2 объекта
вариант3: работает 1м 13с, найдено 2 объекта
т.е. единственный работоспособный вариант в этом случае без танцев с бубном по пересчету размеров окна - _CP, хотя по прежнему время работы в новом чертеже со вставленным отрезком и без zoom _e сильно отличается.


Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 21-12-2017, 17:16:27
Не забывай, что точка вообще очень хитрый и нестандартный примитив, у которой обычно размер зависит от масштаба изображения.
Мой тебе совет - прекрати играться с acedSSGet со всеми опциями в видимой области. Никакой стабильности не будет. Особенно в больших координатах (как у тебя). Думаю, что R-Tree в твоём случае могло бы помочь.
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 21-12-2017, 18:01:59
Цитата: Николай Горлов от 21-12-2017, 15:35:25
итого в сухом остатке. RTERROR - это нормальный ответ функции acedSSGet (чисто моё умозаключение. в хелпе вообще этот вопрос обошли стороной), если она ничего не находит
Это для меня вполне очевидно. Я думал, что для тебя тоже, хотя смотрел на вот эти "пассажи" с недоумением:

Код - C++ [Выбрать]
  1. //-> выборка по первой точке
  2. Adesk::Int32 len = 0L;
  3. int res = acedSSLength(ent1, &len);
  4. if (res != RTNORM || len == 0)  ////// <- тут то чего ловить????
  5. {// возможно объект 1 а не несколько
  6.  
  7.         if (acdbGetObjectId(eId, ent1) == Acad::eOk)
  8.         {// а вдруг в выборке все-таки есть 1 объект
  9.                 if (!eId.isNull())
  10.                 {//объект есть
  11.                         if (!resIds.contains(eId)) // проверяем нет ли его уже в нашем выходном массиве
  12.                                 resIds.append(eId);
  13.                 }
  14.         }
  15. }
  16. else
  17. {
  18.         for (int i = 0; i < len; i++)
  19.         {
  20.                 if (ads_ssname(ent1, i, one_ent) != RTNORM)continue;
  21.                 if (acdbGetObjectId(eId, one_ent) != Acad::eOk)continue;
  22.                 if (!eId.isNull())
  23.                 {
  24.                         if (!resIds.contains(eId)) // проверяем нет ли его уже в нашем выходном массиве
  25.                                 resIds.append(eId);
  26.                 }
  27.         }
  28. }
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 22-12-2017, 11:33:29
Цитировать
Я думал, что для тебя тоже, хотя смотрел на вот эти "пассажи" с недоумением
та дално, знакомьтесь, это затычка :):):). и там проверяется возврат не acedSSGet, а acedSSLength. в каких-то старых автокадах была проблема, если в выборку (ну например был вызов ssget для выбора рамкой любых объектов, и пользователь выбирал не рамкой, а просто ткнул в один объект и успокоился) попадал 1 объект, то начинались чудеса. в современных версиях не проверял, но спокойней, когда эта затычка есть. в лучшем случае внутрь программа никогда не зайдет :)

Цитата: Александр Ривилис от 21-12-2017, 17:16:27
Не забывай, что точка вообще очень хитрый и нестандартный примитив, у которой обычно размер зависит от масштаба изображения.
а это тут при чем? я на вход подаю две точки рамки (XYZ а не AcDbPoint) и этой рамкой, с жестко мной заданными точками ищу захват куска отрезка рамкой. а сама рамка каким-то хитрожо странным образом увеличивается в размерах, что позволяет получить вхождение куска отрезка в контур большее количество раз (а не только тем точкам, рамки которых в действительности могут захватить отрезок)

объясню на примере.
пусть строю рамку 1x1 вокруг точки (0,0). и я хочу получить все объекты, которые попадают в контур({-1,-1},{-1,1},{1,1},{1,-1},{-1,-1}). есть отрезок, проходящий через координату (10,10). так вот в зависимости от масштаба отображения моя рамка способна захватить этот отрезок. я не знаю, что они там употребляют, но я тоже такое хочу :):):)
и так работает опция "_C". путем умозаключений можно предположить, что опция "_W" работает так же (не проверял).

опция "_CP" работает правильно только в том случае, если extents базы включает в себя максимально возможные координаты будущей отрисовки. и это я проверил лично. вчера вечером программно увеличил extents базы текущего (нового) чертежа программно перед рисованием, выставил вид по extents тоже программно (не командой автокада а кодом на С++) а потом нарисовал точки в пустом чертеже. итого 5м 25с.
то же самое рисование занимает полторы минуты если объект есть ФИЗИЧЕСКИ по координатам, где будет проходить поиск рамкой.
вот не похоже это на "ожидаемое поведение", даже с учетом запрещенных препаратов :)

PS: может автодеск напрячь ))) 3 года уже не напрягал )))
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 22-12-2017, 12:38:55
Цитата: Николай Горлов от 22-12-2017, 11:33:29
в лучшем случае внутрь программа никогда не зайдет :)
Как раз таки зайдёт и почему-то будет считать, что ent1 это не набор, а отдельный примитив.
Название: Re: программный выбор рамкой
Отправлено: Николай Горлов от 22-12-2017, 16:08:53
Цитата: Александр Ривилис от 22-12-2017, 12:38:55
Как раз таки зайдёт и почему-то будет считать, что ent1 это не набор, а отдельный примитив.
:) убрал вообще, чтоб не смущать. в конкретном случае пользы от этой затычки нет, т.к. пользователя лишили возможности тыкать по экрану. "не заходит" - это я имел ввиду, что в resIds не добавляется. ну да ладно.

по поводу R-Tree. это оно? https://ru.wikipedia.org/wiki/R-%D0%B4%D0%B5%D1%80%D0%B5%D0%B2%D0%BE_(%D1%81%D1%82%D1%80%D1%83%D0%BA%D1%82%D1%83%D1%80%D0%B0_%D0%B4%D0%B0%D0%BD%D0%BD%D1%8B%D1%85) (https://ru.wikipedia.org/wiki/R-%D0%B4%D0%B5%D1%80%D0%B5%D0%B2%D0%BE_(%D1%81%D1%82%D1%80%D1%83%D0%BA%D1%82%D1%83%D1%80%D0%B0_%D0%B4%D0%B0%D0%BD%D0%BD%D1%8B%D1%85)) тануна ))), для команды, которая всего-то затягивает точки из текстового файла по координатам - это круто. тем более, что для формирования дерева мне все-равно нужен один пробег по БД. Уж лучше тогда std::map использовать. так сказать, дешево и сердито. но хочется использовать автокад хотя б для таких простых заданий. а то получается, что объемы посчитать внутри 3D сетей - автокад не тянет, т.к. сети создает криво (сам создал, сам сказал что кривая и в расчет ее брать нельзя), объекты выбрать - тоже не к нему, блоки между чертежами тягать - лучше своими методами а не командами автокада...


так что вот новый вариант тестового проекта со всеми изменениями. даже на английский перевел :) на всякий случай :):):)
Название: Re: программный выбор рамкой
Отправлено: Александр Ривилис от 22-12-2017, 18:12:37
Цитата: Николай Горлов от 22-12-2017, 16:08:53
что для формирования дерева мне все-равно нужен один пробег по БД.
Он у тебя займёт максимум несколько секунд. А на том файле, что ты генерировал в предыдущей версии программы, доли секунды.