|
Notes |
|
|
|
|
ИМХО вполне нормально. Отметь эти ексепшены один раз как игнорируемые и забудь о них. |
|
|
|
(0007421)
|
|
zed
|
|
17-06-2012 13:37
|
|
|
Вопрос - для кого эти исключения. Для разработчика, который сидит за отладчиком или для пользователя? По-моему, Garl хотел, чтобы их увидел именно пользователь. |
|
|
|
(0007422)
|
|
zed
|
|
17-06-2012 13:45
|
|
|
По-моему, создавать геокодер нужно вне зависимости от наличия какой-то там папки. А вот ошибку отдавать (и показывать пользователю, естественно) надо только тогда, когда пользователь пытается что-то искать этим геокодерем и не предоставил для этого никаких данных (и нужно подробно расписать в сообщении с ошибкой, где, собственно, эти данные можно взять и куда покласть). |
|
|
|
(0007423)
|
|
vdemidov
|
17-06-2012 13:55
(edited on: 17-06-2012 14:00) |
|
Нет. Он хотел, что бы если данных нету, не захламлялся список геокодеров.
PS: Перед этим он вообще вставил проверку наличия этих папок в u_GeoCoderListSimple, а я ему сказал, что бы запихнул проверки в сами геокодеры и кидать ексепшены.
|
|
|
|
(0007424)
|
|
zed
|
|
17-06-2012 14:12
|
|
>не захламлялся список геокодеров
Добавить геокодерам свойство Enabled и пихать в список только разрешённые кодеры.
Если эти геокодеры не создавать и тихо помирать, то пользователи, не читающие доки (а это, процентов 90 от общего числа), так и не узнают, что оказывается есть ещё и оффлайн поиск. |
|
|
|
|
>Добавить геокодерам свойство Enabled и пихать в список только разрешённые кодеры.
Добавляй. |
|
|
|
(0007426)
|
|
zed
|
|
17-06-2012 14:26
|
|
>Добавляй.
Ответственный пускай добавит. Или перенесёт эксепшен в .DoSearch. |
|
|
|
|
|
Я против. Мне нерабочие геокодеры в списке не нужны. |
|
|
|
(0007428)
|
|
zed
|
|
17-06-2012 14:44
|
|
|
Он рабочий, но с условием. Так же, как и склейка в jpeg/png/ecw, так же как и кэши Беркли/GE/GC. Тут условие - наличие папки с данными, там - наличие dll. |
|
|
|
|
|
Ну так и склейки не должны маячить если нет нужной dll. И кэши Беркли, GE и тд не должны отсвечивать если не могут функционировать. |
|
|
|
(0007430)
|
|
zed
|
|
17-06-2012 15:07
|
|
|
Я к тому, что получается разрыв шаблона: в одном месте сделано так, что и при отсутствии dll пункты маячат, а в другом (геокодеры) - пункты не маячат. А разрыв шаблонов - нехорошо. |
|
|
|
|
|
Если во многих местах сделано хреново, это не повод делать хреново везде. |
|
|
|
(0007433)
|
|
Garl
|
|
17-06-2012 17:56
|
|
таки как лучше сделать?
а в wiki я на неделе постараюсь осветить этот пункт. |
|
|
|
|
|
Я за сохранение текущего состояния. |
|
|
|
(0007441)
|
|
zed
|
|
18-06-2012 04:23
|
|
>таки как лучше сделать?
Сделай пропертю Enabled и используй её вместо исключения. |
|
|
|
(0007443)
|
|
Garl
|
|
18-06-2012 05:00
|
|
|
тогда получается по ошибке - дисейблить геокодер и не маячить ексепшионами ? |
|
|
|
(0007459)
|
|
zed
|
|
18-06-2012 09:45
|
|
|