SASGIS - SAS.Планета
View Issue Details
0001941SAS.ПланетаРефакторингpublic29-05-2013 17:1001-02-2015 09:11
zed 
zed 
normalminorN/A
resolvedfixed 
121010 
150915150915 
0001941: Сделать универсальный фрейм выбора типа кэша
Параметры, которые нужны фрейму
- добавлять ли тип кэша "По-умолчанию"
- ID кэша который нужно установить при старте

Возвращать должен ID выбранного кэша (в том числе, фрейм должен возвращать правильный ID для типа кэша "По-умолчанию")

Сейчас в нескольких местах повторяются почти идентичные функции вроде этой. Фрейм же должен помочь избавиться от дублирования кода и облегчить жизнь при добавлении нового (а вдруг?) типа кэша.
No tags attached.
related to 0002014resolved vdemidov Перейти на ITileStorageType вместо прямого создания объектов классов 
related to 0002623confirmed  В фрейме выбора типа тайлохранилища добавить отбор допустимых типов 
Issue History
29-05-2013 17:10zedNew Issue
29-05-2013 17:15zedDescription Updatedbug_revision_view_page.php?rev_id=5404#r5404
30-05-2013 10:47vdemidovNote Added: 0011464
30-05-2013 10:47vdemidovStatusnew => confirmed
10-06-2013 13:20vasketsovNote Added: 0011623
11-07-2013 15:03vdemidovRelationship addedrelated to 0002014
31-01-2015 15:58zedAssigned To => zed
31-01-2015 15:58zedStatusconfirmed => assigned
31-01-2015 16:11zedNote Added: 0015194
31-01-2015 16:11zedTarget Version24xxxx => 150915
31-01-2015 17:07vdemidovNote Added: 0015195
31-01-2015 17:14zedNote Added: 0015196
31-01-2015 17:30vdemidovNote Added: 0015197
31-01-2015 17:40zedNote Added: 0015198
31-01-2015 20:29vdemidovNote Added: 0015207
01-02-2015 09:11zedStatusassigned => resolved
01-02-2015 09:11zedFixed in Version => 150915
01-02-2015 09:11zedResolutionopen => fixed
02-02-2015 13:34vdemidovRelationship addedrelated to 0002623

Notes
(0011464)
vdemidov   
30-05-2013 10:47   
А еще нужно допилить таки фабрики для тайлохранилищ. Это еще более упростит добавление новых типов кэша. Достаточно будет зарегистрировать новую фабрику в нужных списках и новый тип появится там где нужно.
(0011623)
vasketsov   
10-06-2013 13:20   
Попробую временно сделать не через фрейм, а просто в отдельном модуле процедурки для комбобокса. Всё равно добавлять тип кэша, заодно и поглядим, где чего затронется. Логика будет идентичной, за исключением автовозврата реального ID для DEFAULT.
(0015194)
zed   
31-01-2015 16:11   
Сделал фрейм и заюзал его в формах frm_Settings и frm_MapTypeEdit. Ещё осталось облагородить frm_CacheManager и fr_TilesCopy.
(0015195)
vdemidov   
31-01-2015 17:07   
> Ещё осталось облагородить frm_CacheManager и fr_TilesCopy.
Там перед тем как облагораживать их нужно сделать write-only тайлохранилища для заархивированных файловых тайлохранилищ и соответсвующие фабрики для этих типов тайлохранилищ. Плюс в фрейм добавить параметры отбора допустимых для конкретного случая типов тайлохранилищ.
(0015196)
zed   
31-01-2015 17:14   
> перед тем как облагораживать
Это можно сделать и после.
(0015197)
vdemidov   
31-01-2015 17:30   
Ну смотри, но тогда бы оно гораздо красивее ложилось на идею отдельного фрейма:
Для выбора в параметрах карты ставим, что нужны типы тайлохранилищ с произвольным доступом на чтение, можно без записи.
Для выбора дефолтного тайлохранилища - с произвольным доступом и обязательно записью
Для копирования тайлов - любые с возможностью записи
Для исходного в менеджере кэша - любые с возможностю чтения
Для целевого в менеджере кэша - как и для копирования - любые с записью.

А без этого все опять сведется к развесистым case.
(0015198)
zed   
31-01-2015 17:40   
Не спорю, есть куда улучшать, но я хочу сделать минимум и унифицировать выбор типа кэша. И на этой базе уже можно будет рефакторить и дальше, при желании.
(0015207)
vdemidov   
31-01-2015 20:29   
Ну, как знаешь. Занимайся.