Подчистка кода графа

Issue #242 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 242

1. Добавить поля со смысловыми значениями в узлы (возможно и дуги) графа  - например
isassert, isvoid - и перевести все условия на них; условий по цвету, форме узла и т.д.
оставаться не должно ни одного!
2. $neighborl и подобная однобуквица плохо читается, меняем по типу $leftneighbor
3. Жду описания мне основных случаев расстановки точек по ассертам на бумаге. И поэкспериментировать
с принадлежностью дуг - нельзя ли ее упростить....

Reported by oasychev on 2013-10-30 22:39:37

Comments (3)

  1. Oleg Sychev reporter
    Я не понял изменений типа $graph = $this->dst_root->create_graph(); в tool в первом
    коммите и почему это все работает...
    
    После релиза в полях (но не одноименных функциях!) базовых классов были убраны подчеркивания
    для приведения их в соответствие со стандартом, а вы зачем-то у себя в классе унаследованное
    поле назад исправляете...
    

    Reported by oasychev on 2013-11-27 21:05:39

  2. Oleg Sychev reporter
    1) https://code.google.com/r/zlumyo-preg-26/source/browse/question/type/preg/authoring_tools/preg_explaining_graph_misc.php#234
     последняя часть условия $neighbor->style == $tmpdnode->style должна теперь использовать
    типы, а не стили?
    
    2) https://code.google.com/r/zlumyo-preg-26/source/browse/question/type/preg/authoring_tools/preg_explaining_graph_misc.php#429
    смотрите $this->color != 'darkgreen' - очевидно, надо для выделения тоже отдельное
    поле смысловое создать, чтобы убрать проверку по цвету
    
    P.S. Ну и комментарии типа // Assert should has a red color. наверное надо теперь поправить....
    

    Reported by oasychev on 2013-11-27 21:25:56

  3. Log in to comment