Сделать константы, описывающие данные об узлах, константами класса node
Originally reported on Google Code with ID 7 ``` Необходимо убрать константы с данными об узлах из глобального пространства имен, сделав их константами класса node.
У многих из них достаточно общие имена, просто чудо что до сих пор ни одна из них не вступила в конфликт с другими константами Moodle (или, возможно, вступила - но мы еще не заметили тонких ошибок, которые это вызывает при определенных обстоятельствах).
В глобальном пространстве имен могут оставаться настроечные константы типа максимального количества узлов и переходов автомата, но они должны иметь префикс QTYPE_PREG_....
```
Reported by `oasychev` on 2010-09-20 09:38:32
- Blocked on: #3
Comments (5)
-
reporter -
reporter ``` Надо бы наконец переименовать глобальные константы типа MAX_STATE_COUNT и MAX_PASSAGE_COUNT в QTYPE_PREG_DFA_MAX_STATE_COUNT и т.д.
Да закрыть это дело, чего такую мелочь держать открытой? ```
Reported by `oasychev` on 2011-05-06 23:03:02
-
reporter ``` Ну вы про это не забыли? Тоже ведь практически часть рефакторинга.... ```
Reported by `oasychev` on 2011-05-12 22:28:25 - Status changed: `InProgress`
-
reporter ``` Дело не сделано, но задача перенесена в рефакторинг (номер 31) и константы все же не переименовывать, а приписать к классу, для отдельного issue она маловата- их и так слишком много. А кто-то говорил что не видел ни разу.... ```
Reported by `oasychev` on 2011-09-09 13:09:47 - Status changed: `Verified`
-
reporter - changed status to closed
Давно сделано.
- Log in to comment
``` И переименуйте его уже наконец в preg_node, node - слишком общее название чтобы нарваться на конфликт в конце концов.
Этот класс может послужить прототипом для базового класса всех узлов при рефакторинге. Тогда, возможно, большая часть констант вообще не понадобятся - надо будет просто создавать объект нужного типа (при необходимости - проверять тип...). Поэтому константы оставляем до рефакторинга, но с ним не тянуть... ```
Reported by `oasychev` on 2010-09-21 20:01:15 - Labels added: Priority-Medium - Labels removed: Priority-High - Blocked on: #