|
•Отв: Отв: Отв: критика - Artlav, 23.05.2010 20:56:26 |
|
|
>public enum Event { Build, Transfer, Move, EndTurn ... Можно так сделать, перечислимые типы есть. Преимущество однако не существенно, так что основным аргументом становится что нравится как есть. Сгруппированы они по именам, вроде всё понятно, числа проставить невелика хитрость.
>1. curs_calc - короче писать >2. CalculateCursorType - легче понимать :) Согласен, в эту сторону изменения идут.
>В терминах ООП можно описать все, даже мы состоим из органов/клеток/атомов. Можно. Также можно и в декларативных. Опять таки упирается в что кому милее.
>Думаю и так понятно что легче читать и понимать, хотя остается загадкой что такое .unu, pType и цифра 5. Согласен, константы типов проходимости давно надо проименовать. С комканьем строки не согласен - у меня широкий монитор, так что как есть удобнее. Как переименовать unu не разжирневая код пока не представляю. Вполне наверное можно внести в словарь проекта.
refmod - refuel_mode navmod - not_available_mode Эти выравниваются и группируются размером и окончанием, почему-то переименовывать не тянет - становится не видно о чём речь.
ptyp и ptype - разницы не улавливаю совсем, passability_type или что-то более английское.
Префикс mg_ указывает на предметы сервера, убирать его - вносить путаницу.
>Большинство процедур из этого файла относится непосредственно к командам <сделай что-то там>_cmd, так что все остальное логичнее вынести в другие файлы. Потом будет легче разобраться что и где.
По смыслу там 3 события ввода: -щёлкнуть мышью тут - mouse_event -отладочный щелчёк мышию туда - dbg_place_unit -навести курсор сюда - curs_calc Вроде логично, куда ещё можно последнее сунуть?
|
|
| |