Гейзенбаг - после очистки всех кешей, на некоторых машинах повисает процесс httpd

Originally reported on Google Code with ID 195

На первое время очевидные задачи.

1. Убить switch-case в dotstyleprovider, по возможности вообще избавиться от этого
класса. Сделать это путем создания новых dst узлов для рисования дерева (аналогично
другим dst узлам, более-менее адекватный пример см. в nfa_matcher). В них перенести
функцию dot_script из обычных узлов, и скорее всего определить функции типа label(),
shape(), color() и т. д.

2. Избавиться от некоторых switch-case в explaining graph. Там есть dst узлы для листа
и оператора, но не для конкретных операторов\листов. Их следует создать - это убьет
значительную часть switch-case'ов.

3. Обратить внимание на то, что конструкции типа

if ($this->pregnode->subtype == qtype_preg_leaf_assert::SUBTYPE_CIRCUMFLEX)
    return get_string('circumflex_leaf_assert', 'qtype_preg');

можно заменить на

return get_string($this->pregnode->subtype);

если же ключ в lang файле не совпадает с subtype, можно сделать функцию, вычисляющую
его на основе subtype - это гораздо проще такого копипаста, который там сейчас; код
будет проще читаться и будет некоторая "дисциплина" именования классов\строк в lang
файле.

Reported by vostreltsov on 2013-04-26 14:58:50

Comments (38)