SASGIS

Веб-картография и навигация
All Projects | SAS.Планета | Домен, сайт, форум, багтрекер | Доработка карты (ZMP) | Переводы и локализации | Прочее

My View | View Issues | Change Log | Roadmap | Search
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001338SACS.ПланетаРефакторингpublic17-06-2012 11:4628-02-2014 10:52
Reporterzed 
Assigned Tovasketsov 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformWindowsOSXPOS VersionSP3
Product Version 
Target VersionFixed in Version130803 
Summary0001338: Исключения в конструкторах TGeoCoderByTXT и TGeoCoderByPolishMap
DescriptionПри запуске SAS под отладчиком, вываливаются два исключения:

First chance exception at $7C812AFB. Exception class EDirNotExist with message 'not found .\userdata\mp\! skip GeoCoderByPolishMap'. Process SASPlanet.exe (2852)
First chance exception at $7C812AFB. Exception class EDirNotExist with message 'not found .\userdata\txt\! skip GeoCoderByTXT'. Process SASPlanet.exe (2852)

нужно эту ситуацию разрулить более красиво.
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0007420)
vdemidov (manager)
17-06-2012 12:58

 ИМХО вполне нормально. Отметь эти ексепшены один раз как игнорируемые и забудь о них.
(0007421)
zed (manager)
17-06-2012 13:37

 Вопрос - для кого эти исключения. Для разработчика, который сидит за отладчиком или для пользователя? По-моему, Garl хотел, чтобы их увидел именно пользователь.
(0007422)
zed (manager)
17-06-2012 13:45

 По-моему, создавать геокодер нужно вне зависимости от наличия какой-то там папки. А вот ошибку отдавать (и показывать пользователю, естественно) надо только тогда, когда пользователь пытается что-то искать этим геокодерем и не предоставил для этого никаких данных (и нужно подробно расписать в сообщении с ошибкой, где, собственно, эти данные можно взять и куда покласть).
(0007423)
vdemidov (manager)
17-06-2012 13:55
edited on: 17-06-2012 14:00

 Нет. Он хотел, что бы если данных нету, не захламлялся список геокодеров.
PS: Перед этим он вообще вставил проверку наличия этих папок в u_GeoCoderListSimple, а я ему сказал, что бы запихнул проверки в сами геокодеры и кидать ексепшены.
(0007424)
zed (manager)
17-06-2012 14:12

 >не захламлялся список геокодеров
Добавить геокодерам свойство Enabled и пихать в список только разрешённые кодеры.

Если эти геокодеры не создавать и тихо помирать, то пользователи, не читающие доки (а это, процентов 90 от общего числа), так и не узнают, что оказывается есть ещё и оффлайн поиск.
(0007425)
vdemidov (manager)
17-06-2012 14:23

 >Добавить геокодерам свойство Enabled и пихать в список только разрешённые кодеры.
Добавляй.
(0007426)
zed (manager)
17-06-2012 14:26

 >Добавляй.
Ответственный пускай добавит. Или перенесёт эксепшен в .DoSearch.
(0007427)
vdemidov (manager)
17-06-2012 14:31

 Я против. Мне нерабочие геокодеры в списке не нужны.
(0007428)
zed (manager)
17-06-2012 14:44

 Он рабочий, но с условием. Так же, как и склейка в jpeg/png/ecw, так же как и кэши Беркли/GE/GC. Тут условие - наличие папки с данными, там - наличие dll.
(0007429)
vdemidov (manager)
17-06-2012 14:59

 Ну так и склейки не должны маячить если нет нужной dll. И кэши Беркли, GE и тд не должны отсвечивать если не могут функционировать.
(0007430)
zed (manager)
17-06-2012 15:07

 Я к тому, что получается разрыв шаблона: в одном месте сделано так, что и при отсутствии dll пункты маячат, а в другом (геокодеры) - пункты не маячат. А разрыв шаблонов - нехорошо.
(0007431)
vdemidov (manager)
17-06-2012 15:33

 Если во многих местах сделано хреново, это не повод делать хреново везде.
(0007433)
Garl (reporter)
17-06-2012 17:56

 таки как лучше сделать?
а в wiki я на неделе постараюсь осветить этот пункт.
(0007434)
vdemidov (manager)
17-06-2012 18:30

 Я за сохранение текущего состояния.
(0007441)
zed (manager)
18-06-2012 04:23

 >таки как лучше сделать?
Сделай пропертю Enabled и используй её вместо исключения.
(0007443)
Garl (reporter)
18-06-2012 05:00

 тогда получается по ошибке - дисейблить геокодер и не маячить ексепшионами ?
(0007459)
zed (manager)
18-06-2012 09:45

 Да.

- Users who viewed this issue
User List Anonymous (2104x)
Total Views 2104
Last View 25-04-2024 01:45

- Issue History
Date Modified Username Field Change
17-06-2012 11:46 zed New Issue
17-06-2012 11:46 zed Description Updated View Revisions
17-06-2012 12:58 vdemidov Note Added: 0007420
17-06-2012 13:37 zed Note Added: 0007421
17-06-2012 13:45 zed Note Added: 0007422
17-06-2012 13:55 vdemidov Note Added: 0007423
17-06-2012 14:00 vdemidov Note Edited: 0007423 View Revisions
17-06-2012 14:12 zed Note Added: 0007424
17-06-2012 14:23 vdemidov Note Added: 0007425
17-06-2012 14:26 zed Note Added: 0007426
17-06-2012 14:31 vdemidov Note Added: 0007427
17-06-2012 14:44 zed Note Added: 0007428
17-06-2012 14:59 vdemidov Note Added: 0007429
17-06-2012 15:07 zed Note Added: 0007430
17-06-2012 15:33 vdemidov Note Added: 0007431
17-06-2012 17:56 Garl Note Added: 0007433
17-06-2012 18:30 vdemidov Note Added: 0007434
18-06-2012 04:23 zed Note Added: 0007441
18-06-2012 05:00 Garl Note Added: 0007443
18-06-2012 09:45 zed Note Added: 0007459
09-08-2012 06:56 vdemidov Product Version .Nightly => 120808
29-12-2012 11:34 vdemidov Status new => confirmed
29-12-2012 11:34 vdemidov Target Version => 20xxxx
14-02-2013 19:37 vasketsov Assigned To => vasketsov
14-02-2013 19:37 vasketsov Status confirmed => assigned
14-02-2013 19:37 vasketsov Project SAS.Планета => SACS.Планета
14-02-2013 19:42 vasketsov Status assigned => resolved
14-02-2013 19:42 vasketsov Resolution open => fixed
09-08-2013 15:02 vasketsov Fixed in Version => 130803
09-08-2013 15:13 vasketsov Status resolved => closed

My View | View Issues | Change Log | Roadmap | Search
  

Copyright © 2007 - 2024 SAS.Planet Team