Anonymous | Login | Signup for a new account | 04-05-24 10:08 UTC |
All Projects | SAS.Планета | Домен, сайт, форум, багтрекер | Доработка карты (ZMP) | Переводы и локализации | Прочее |
My View | View Issues | Change Log | Roadmap | Search |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0001338 | SACS.Планета | Рефакторинг | public | 17-06-2012 11:46 | 28-02-2014 10:52 | ||||
Reporter | zed | ||||||||
Assigned To | vasketsov | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | Windows | OS | XP | OS Version | SP3 | ||||
Product Version | |||||||||
Target Version | Fixed in Version | 130803 | |||||||
Summary | 0001338: Исключения в конструкторах 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) нужно эту ситуацию разрулить более красиво. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
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 |
Да. |
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 |