Сделать константы, описывающие данные об узлах, константами класса node

Issue #7 closed
Oleg Sychev repo owner created an issue

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)

  1. Oleg Sychev reporter

    ``` И переименуйте его уже наконец в preg_node, node - слишком общее название чтобы нарваться на конфликт в конце концов.

    Этот класс может послужить прототипом для базового класса всех узлов при рефакторинге. Тогда, возможно, большая часть констант вообще не понадобятся - надо будет просто создавать объект нужного типа (при необходимости - проверять тип...). Поэтому константы оставляем до рефакторинга, но с ним не тянуть... ```

    Reported by `oasychev` on 2010-09-21 20:01:15 - Labels added: Priority-Medium - Labels removed: Priority-High - Blocked on: #

  2. Oleg Sychev reporter

    ``` Надо бы наконец переименовать глобальные константы типа MAX_STATE_COUNT и MAX_PASSAGE_COUNT в QTYPE_PREG_DFA_MAX_STATE_COUNT и т.д.

    Да закрыть это дело, чего такую мелочь держать открытой? ```

    Reported by `oasychev` on 2011-05-06 23:03:02

  3. Oleg Sychev reporter

    ``` Ну вы про это не забыли? Тоже ведь практически часть рефакторинга.... ```

    Reported by `oasychev` on 2011-05-12 22:28:25 - Status changed: `InProgress`

  4. Oleg Sychev reporter

    ``` Дело не сделано, но задача перенесена в рефакторинг (номер 31) и константы все же не переименовывать, а приписать к классу, для отдельного issue она маловата- их и так слишком много. А кто-то говорил что не видел ни разу.... ```

    Reported by `oasychev` on 2011-09-09 13:09:47 - Status changed: `Verified`

  5. Log in to comment