Notes |
|
(0014694)
|
Garl
|
15-10-2014 12:29
|
|
вопрос: нормально ли будет передавать в интерфейс значения всех чек-боксов как Byte? |
|
|
|
Выбранные типы меток одним байтом, а признак удаления только видимых отдельным параметром типа boolean. |
|
|
|
И еще. Не "Удалить файлы карты", а "Удалить тайлы карты" ибо они могут и не в файлах хранится. Или вообще оставить только слово "Карты" |
|
|
(0014697)
|
zed
|
15-10-2014 14:27
|
|
Если уж вторая закладка называется "Метки", то первая должна называться "Тайлы". Карты там не удаляются ж. |
|
|
|
Согласен. "Тайлы" оптимально. |
|
|
(0014699)
|
Garl
|
16-10-2014 07:50
|
|
вышесказанное принято, добрался чуток дальше в реализации и вот ещё вопросы:
GetMarkSubsetByCategoryInRect - как я понимаю ищет в квадрате.
а если у нас выделение произвольное?
Игнорировать его удалять только при квадратном выделении
или пытаться реализовавать чтото типа GetMarkSubsetByCategoryInPolygon ? |
|
|
|
Получи из полигона MBR, по нем получи список меток, отбери только те, которые пересекаются с полигоном. Окончательно сформированный список удали. |
|
|
|
Глянул в исходники. Для отбора меток попадающих в полигон, придется проектировать геометрии меток в текущую проекцию и проверять пересечение со спроектированным полигоном. Но первичный отбор все равно по Bounds |
|
|
|
Хотя методов проверки пересечения полигона с путем или другим полигоном у нас все равно нет. Но делать его все равно стоит в спроецированном виде в текущей активной проекции. |
|
|
(0014703)
|
Garl
|
16-10-2014 09:53
|
|
ну удаляться будут только те пути которые поностью попадают в полигон выделения, так оно логичнее и правильнее |
|
|
|
Ну вперед. Ждем пулл реквеста. |
|
|
(0014709)
|
Garl
|
17-10-2014 07:57
|
|
я вот тоже глянул:
есть ли смысл делать _AddMarksToListByRect, если он циклом перебирает все метки?
может проще сделать свой цикл и тут же их перероецировать и проверять за 1 проход? |
|
|
|
Проще, но нельзя. Базе меток не положено знать ничего о проекциях, тем более о текущей. |
|